Skip to content

🌐 [translation-sync] FIX: lecture cleanup batch from the 2026-08 tracker audit - #34

Merged
mmcky merged 12 commits into
mainfrom
translation-sync-2026-08-18T03-42-26-pr-595
Aug 18, 2026
Merged

🌐 [translation-sync] FIX: lecture cleanup batch from the 2026-08 tracker audit#34
mmcky merged 12 commits into
mainfrom
translation-sync-2026-08-18T03-42-26-pr-595

Conversation

@mmcky

@mmcky mmcky commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Automated Translation Sync

This PR contains automated translations from QuantEcon/lecture-python-programming.

Source PR

#595 - FIX: lecture cleanup batch from the 2026-08 tracker audit

Files Updated

  • ✏ lectures/about_py.md
  • ✏ .translate/state/about_py.md.yml
  • ✏ lectures/autodiff.md
  • ✏ .translate/state/autodiff.md.yml
  • ✏ lectures/pandas.md
  • ✏ .translate/state/pandas.md.yml
  • ✏ lectures/pandas_panel.md
  • ✏ .translate/state/pandas_panel.md.yml
  • ✏ lectures/polars.md
  • ✏ .translate/state/polars.md.yml
  • ✏ lectures/python_by_example.md
  • ✏ .translate/state/python_by_example.md.yml

Details

  • Source Language: en
  • Target Language: fr
  • Model: claude-sonnet-5

This PR was created automatically by the translation action.

@mmcky
mmcky requested a lite review from Copilot August 18, 2026 03:42
@mmcky mmcky added action-translation PRs created by QuantEcon/action-translation automated Automated sync PR opened by action-translation labels Aug 18, 2026
@github-actions

Copy link
Copy Markdown

✅ Translation Quality Review

Verdict: PASS | Model: claude-sonnet-5 | Date: 2026-08-18
Routing: editor — 1 minor finding(s) in gating categories (accuracy/terminology/syntax/diff-check/other)
Shadow gate: would NOT auto-merge (recorded only; no action taken)


📝 Translation Quality

Criterion Score
Accuracy 9/10
Fluency 9/10
Terminology 9/10
Formatting 9/10
Overall 9/10

Summary: This is a high-quality, professional translation across all six changed documents. Terminology is consistent with the glossary, French typography rules are correctly applied, and MyST/Markdown structure is fully preserved with no syntax errors. The main issues are a handful of places where translators appear to have made small code edits (e.g., isinstance vs type()==str, added future_stack=True parameters) rather than pure translation — these should be reviewed since they alter code behavior/version compatibility beyond simple translation, though they do not affect the readability or accuracy of the prose. Excellent adherence to the glossary and French typographic conventions (guillemets, non-breaking spaces before high punctuation) Consistent and idiomatic rendering of technical vocabulary (autodiff, descente de gradient, Ă©valuation paresseuse, etc.) Code blocks, math, and MyST directives are preserved correctly across all six files with no structural breakage Comments within code cells are naturally translated while preserving functional code (variable names, function calls) intact in the vast majority of cases

Suggestions:

  • [minor · accuracy] lectures/pandas.md — ### Apply Method / replace_nan function: The English source uses type(x)!=str in both the .map() lambda and the replace_nan function, but the French translation changes this to not isinstance(x, str) and pd.isna(x) respectively. This is a code change not present in the English source (which is otherwise identical code), going beyond translation into code modification. → Keep the code identical to the English source (type(x)!=str, np.isnan(x)) unless the PR intends a deliberate code fix; if intentional, it should be noted separately from translation quality.
  • [minor · fluency] lectures/pandas_panel.md — ## DĂ©coupage et remodelage des donnĂ©es / .stack() explanation: The French translation adds future_stack=True to several .stack() calls and includes a parenthetical comment '(future_stack=True est requis jusqu'Ă  pandas>3.0)' that does not appear in the English source, which uses plain .stack(). This is a substantive code/content addition beyond translation. → If this is an intentional fix to keep the code working with newer pandas versions, it should be flagged as a content change outside the scope of translation rather than presented as an equivalent translation.
  • [minor · fluency] lectures/polars.md — ### Performance comparison: "Sur quelques lignes, la diffĂ©rence de vitesse est nĂ©gligeable --- utilisez celle des API que vous trouvez la plus pratique." is a slightly awkward rendering of "use whichever API you find more convenient"; 'celle des API' reads a bit unnatural. → Consider: "Sur quelques lignes, la diffĂ©rence de vitesse est nĂ©gligeable --- utilisez l'API que vous trouvez la plus pratique."
  • [nit · terminology] lectures/about_py.md — ### Relative Popularity: "especially those connected to AI" is translated as "en particulier celle liĂ©e Ă  l'IA" with an added final period not present in the English (English sentence has no closing punctuation after 'AI'), which is a minor formatting nicety rather than an issue, but worth confirming intent. → No change needed; this is a stylistic improvement.

🔍 Diff Quality

Check Status
Scope Correct ✅
Position Correct ✅
Structure Preserved ✅
Heading-map Correct ✅
Overall 10/10

Summary: The translation correctly mirrors all source changes (rng refactors, PyTorch capitalization, pandas.stack future_stack fixes, JAX key splitting, Polars random generator updates, URL updates) in the same files, sections, and order, with headings and translation metadata intact.

Issues:

  • In about_py.md, blank lines between some sections (e.g., before/after H3 subsections like 'Can't I Just Use LLMs?', 'Introducing Python', 'NumPy Alternatives', 'Networks and Graphs') were removed compared to the source's spacing pattern, though this doesn't affect rendered structure.
  • In pandas.md and pandas_panel.md, translated prose text ('Here you can imagine...', 'we set up', etc.) also had minor incidental wording tweaks (e.g., 'nous' -> 'on') alongside the rng-related code changes, but these are consistent with the source's applied changes and don't break sync.

This review was generated automatically by action-translation review mode.

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

Synchronisation automatisĂ©e des traductions FR avec les derniĂšres mises Ă  jour du dĂ©pĂŽt source (PR QuantEcon/lecture-python-programming#595), incluant des ajustements de code d’exemples (gĂ©nĂ©ration alĂ©atoire) et quelques corrections de contenu/formatage dans plusieurs leçons.

Changes:

  • Remplacement d’appels legacy Ă  np.random.* par l’API np.random.default_rng() dans plusieurs exemples.
  • Ajustement de la gĂ©nĂ©ration alĂ©atoire JAX via jax.random.split pour sĂ©parer les clĂ©s.
  • Mise Ă  jour de certaines URLs de donnĂ©es vers raw.githubusercontent.com + rafraĂźchissement des mĂ©tadonnĂ©es de synchronisation .translate/state/*.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
lectures/about_py.md Nettoyage de blancs + correction de casse “PyTorch” et petits ajustements de texte.
lectures/autodiff.md Ajustements de clés aléatoires JAX + corrections mineures de formatage.
lectures/pandas.md Passage à default_rng() dans l’exemple d’introduction + retouches de formulation.
lectures/pandas_panel.md Mise à jour des URLs de téléchargement des CSV vers raw.githubusercontent.com.
lectures/polars.md Passage à default_rng() et correction d’un calcul de variation % avec gestion des nulls.
lectures/python_by_example.md Mise Ă  jour de l’exemple “bruit blanc” pour utiliser default_rng() + texte associĂ©.
.translate/state/about_py.md.yml Mise à jour des métadonnées de sync (source SHA, date, modÚle, version outil).
.translate/state/autodiff.md.yml Mise à jour des métadonnées de sync (source SHA, date, modÚle, version outil).
.translate/state/pandas.md.yml Mise à jour des métadonnées de sync (source SHA, date, version outil).
.translate/state/pandas_panel.md.yml Mise à jour des métadonnées de sync (source SHA, date, version outil).
.translate/state/polars.md.yml Mise à jour des métadonnées de sync (source SHA, date, version outil).
.translate/state/python_by_example.md.yml Mise à jour des métadonnées de sync (source SHA, date, version outil).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread lectures/polars.md
Comment on lines 705 to 709
price_change = ticker.select([
((pl.col(tick).last() / pl.col(tick).first() - 1) * 100)
((pl.col(tick).drop_nulls().last() / pl.col(tick).drop_nulls().first() - 1) * 100)
.alias(tick)
for tick in ticker_list.keys()
]).transpose(
@mmcky
mmcky merged commit 888561c into main Aug 18, 2026
4 of 7 checks passed
@mmcky
mmcky deleted the translation-sync-2026-08-18T03-42-26-pr-595 branch August 18, 2026 03:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action-translation PRs created by QuantEcon/action-translation automated Automated sync PR opened by action-translation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants