Skip to content

Add translated _admonition/gpu.md (referenced but missing)#172

Merged
mmcky merged 1 commit into
mainfrom
phase0/gpu-admonition
Jul 18, 2026
Merged

Add translated _admonition/gpu.md (referenced but missing)#172
mmcky merged 1 commit into
mainfrom
phase0/gpu-admonition

Conversation

@mmcky

@mmcky mmcky commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

mix_model.md on main and several resync-wave files include _admonition/gpu.md, but the directory never existed in this repo. Adds the Chinese translation of the source's 12-line admonition. Found during the 7-PR merge review.

🤖 Generated with Claude Code

mix_model.md on main and several resync branches (e.g. ifp_egm) include
_admonition/gpu.md, which never existed in this repo. Translated from
the English source's lectures/_admonition/gpu.md.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 18, 2026 11:22
@netlify

netlify Bot commented Jul 18, 2026

Copy link
Copy Markdown

Deploy Preview for astonishing-narwhal-a8fc64 ready!

Name Link
🔨 Latest commit 5173978
🔍 Latest deploy log https://app.netlify.com/projects/astonishing-narwhal-a8fc64/deploys/6a5b6200119ec00008a217d7
😎 Deploy Preview https://deploy-preview-172--astonishing-narwhal-a8fc64.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds the missing _admonition/gpu.md include file (Chinese translation) that is referenced by existing lecture content, preventing broken {include} directives in the JupyterBook/MyST build.

Changes:

  • Add lectures/_admonition/gpu.md containing a GPU warning admonition in Chinese.
  • Restore validity of existing ```{include} _admonition/gpu.md ``` usage (e.g., in lectures/mix_model.md).
Comments suppressed due to low confidence (1)

lectures/_admonition/gpu.md:12

  • The file ends with a whitespace-only line after the closing code fence. This can introduce trailing-whitespace noise and may trip markdown linters; it’s best to end the file right after the closing ``` fence (with a normal newline).
</details>




---

💡 <a href="/QuantEcon/lecture-python.zh-cn/new/main?filename=.github/instructions/*.instructions.md" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Add Copilot custom instructions</a> for smarter, more guided reviews. <a href="https://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Learn how to get started</a>.

@github-actions

Copy link
Copy Markdown

@github-actions
github-actions Bot temporarily deployed to pull request July 18, 2026 11:34 Inactive
@mmcky
mmcky merged commit 8b4c90b into main Jul 18, 2026
8 checks passed
@mmcky
mmcky deleted the phase0/gpu-admonition branch July 18, 2026 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants