Skip to content

🌐 [translation-sync] chore: take theme v3.0.0, migrating the launch and footer config - #79

Open
mmcky wants to merge 4 commits into
mainfrom
translation-sync-2026-09-12T05-08-52-pr-629
Open

🌐 [translation-sync] chore: take theme v3.0.0, migrating the launch and footer config#79
mmcky wants to merge 4 commits into
mainfrom
translation-sync-2026-09-12T05-08-52-pr-629

Conversation

@mmcky

@mmcky mmcky commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Automated Translation Sync

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

Source PR

#629 - chore: take theme v3.0.0, migrating the launch and footer config

Files Updated

  • ✏️ lectures/need_for_speed.md
  • ✏️ .translate/state/need_for_speed.md.yml
  • ✏️ lectures/polars.md
  • ✏️ .translate/state/polars.md.yml

Details

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

This PR was created automatically by the translation action.

Copilot AI lite review requested due to automatic review settings September 12, 2026 05:08
@mmcky mmcky added action-translation PRs created by QuantEcon/action-translation automated Automated sync PR opened by action-translation labels Sep 12, 2026
@github-actions

Copy link
Copy Markdown

✅ Translation Quality Review

Verdict: PASS | Model: claude-sonnet-5 | Date: 2026-09-12
Routing: auto-merge — no gating findings; floors met
Shadow gate: would auto-merge (recorded only; no action taken)


📝 Translation Quality

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

Summary: The translation of the modified sections (frontmatter, Overview, and Accessing GPU Resources) is accurate, fluent, and terminologically consistent with the glossary and style rules. No syntax errors were found, and formatting/MyST structures are fully preserved. Only very minor stylistic nuances could be tightened, but nothing rises above a nit-level observation. Accurate and idiomatic rendering of technical vocabulary (GPU, TPU, cœurs, parallélisation) consistent with the glossary Correct French typography applied throughout changed sections (guillemets, non-breaking spaces before ':' and '?') Frontmatter translation metadata correctly added without disrupting YAML/MyST structure Natural restructuring of English sentences into idiomatic French rather than literal calques

Suggestions:

  • [minor · fluency] lectures/need_for_speed.md — ### Accessing GPU Resources — 'largement abordée': The English 'discussed extensively' modifies 'JAX' as a general statement, and the French 'largement abordée' agrees correctly with 'JAX' (feminine), which is fine, but the phrase reads slightly awkwardly compared to a more natural construction like 'dont nous parlerons longuement'. → que nous étudierons en détail tout au long de cette série de cours
  • [nit · terminology] lectures/need_for_speed.md — ## Overview — 'Cela s'explique par': Minor stylistic point: 'Cela s'explique par' is a good natural rendering of 'This is due to', no issue, listed only as a strength note rather than a problem.

🔍 Diff Quality

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

Summary: The translation sync correctly removed the raw jupyter header blocks in both files matching the source changes, with correct positioning, preserved structure, and unchanged (still valid) translation heading maps, though minor extra blank-line removals in need_for_speed.md exceed what the source diff shows.

Issues:

  • Target need_for_speed.md removed additional blank lines between some heading sections (e.g., before '### Pourquoi en avons-nous besoin ?', '#### Types statiques', '#### Sommation en Python pur', '### Résumé', '## Accélérer Python', etc.) that were not removed in the source, causing extra line-count changes (+1/-31) beyond the raw jupyter block removal (-8 expected). This is a minor formatting inconsistency, not a content/structure error.

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.

🟢 Approval recommended

Changes are limited to deterministic documentation/layout cleanup and translation sync metadata updates with no functional code impact.

Pull request overview

This PR syncs French lecture translations with upstream theme v3.0.0 changes, removing the legacy notebook header HTML and updating the translation sync state metadata to match the new upstream commit.

Changes:

  • Remove the embedded QuantEcon notebook header {raw} jupyter HTML block from two lecture pages.
  • Normalize markdown spacing/blank lines in lectures/need_for_speed.md (no content changes beyond formatting/structure).
  • Update .translate/state/*.yml to the new source SHA, sync date, model, mode, and tool version.
File summaries
File Description
lectures/polars.md Removes the legacy raw-Jupyter HTML notebook header block.
lectures/need_for_speed.md Removes the legacy raw-Jupyter HTML notebook header block and cleans up extra blank lines/indentation.
.translate/state/polars.md.yml Updates translation sync metadata to the new upstream commit/date/tooling.
.translate/state/need_for_speed.md.yml Updates translation sync metadata to the new upstream commit/date/tooling.
Review details
  • Files reviewed: 4/4 changed files
  • Comments generated: 0
  • Review effort level: Lite

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

@github-actions

Copy link
Copy Markdown

📖 Netlify Preview Ready!

Preview URL: https://pr-79--verdant-toffee-3261a8.netlify.app

Commit: e066ed8

📚 Changed Lectures


Build Info

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