[action-translation] resync: ak_aiyagari.md#103
Merged
Conversation
mmcky
added
action-translation-sync
✅ Deploy Preview for astonishing-narwhal-a8fc64 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
This PR is a forward resync of lectures/ak_aiyagari.md to match the current upstream source, intended to restore missing structural/content elements (notably the GPU admonition include) and update translation sync state.
Changes:
- Added translation metadata and heading mappings to the document front matter.
- Inserted the GPU admonition include directive after the main title.
- Updated translation sync state in
.translate/state/ak_aiyagari.md.yml.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| lectures/ak_aiyagari.md | Resync of the translated lecture, adds GPU admonition include and new metadata (but currently introduces parsing-breaking fences / duplicate front matter). |
| .translate/state/ak_aiyagari.md.yml | Records the source SHA, sync date, and resync metadata for the translation tooling. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+38
to
42
| ``` | ||
| --- | ||
| jupytext: | ||
| text_representation: | ||
| extension: .md |
Comment on lines
+55
to
+56
| ```{include} _admonition/gpu.md | ||
| ``` |
| plt.show() | ||
| ``` | ||
|
|
||
| ``` No newline at end of file |
The resynced file keeps Chinese plot labels but reverted the import cell to the source's exact form, losing the Source Han Serif setup - Chinese in figures would render as missing glyphs. Residual #107 class found in the merge review; applied wave-wide. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…anslation#116) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Forward Resync: ak_aiyagari.md
Source: QuantEcon/lecture-python.myst — lectures/ak_aiyagari.md
Source commit:
ce54324This PR resyncs the translation to match the current source document.
Reason: The Chinese translation omits the '{include} _admonition/gpu.md' directive present in the source. It also omits a blank line/paragraph break structure in a few places but more importantly drops the admonition include entirely, which is a substantive content element (GPU notice) missing from the target. Aside from that, the rest of the document is a faithful and complete translation.
Changes
Whole-file resync applied. The entire document was resynced in a single pass.
Created by action-translation forward resync