Skip to content

🌐 [translation-sync] Read test_pwt.csv from data-lectures (Track D) - #35

Merged
mmcky merged 4 commits into
mainfrom
translation-sync-2026-08-18T03-53-18-pr-612
Aug 18, 2026
Merged

🌐 [translation-sync] Read test_pwt.csv from data-lectures (Track D)#35
mmcky merged 4 commits into
mainfrom
translation-sync-2026-08-18T03-53-18-pr-612

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

#612 - Read test_pwt.csv from data-lectures (Track D)

Files Updated

  • ✏️ lectures/pandas.md
  • ✏️ .translate/state/pandas.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 August 18, 2026 03:53
@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 10/10
Overall 9.2/10

Summary: The translation of the modified sections in both pandas.md and polars.md is highly accurate, fluent, and terminologically consistent with the provided glossary and French typographic rules. No blocking or major issues were found; only a few minor stylistic and consistency nits remain, primarily around table syntax whitespace and code-comment localization consistency. Excellent, idiomatic French rendering of technical Pandas/Polars vocabulary (e.g., 'évaluation paresseuse', 'descente des prédicats', 'centrée réduite' concepts correctly avoided/adapted where not applicable) with strong consistency across both files French typographic conventions (non-breaking spaces before ; : ! ?, guillemets where applicable) are correctly applied throughout the changed sections Long English sentences are frequently and skillfully split into more natural French sentence structures without losing technical precision Code blocks, LaTeX, and MyST directive syntax are fully preserved with no structural breakage

Suggestions:

  • [minor · terminology] lectures/pandas.md — Table headers under DataFrames section: Table alignment markers were altered from :-: to :- : (with an inserted space), which is unnecessary and inconsistent with standard Markdown table syntax, though it does not break rendering in most parsers. → Use :-: instead of :- : for table column alignment markers.
  • [minor · fluency] lectures/polars.md — ### Performance comparison, 'Sur quelques lignes, la différence de vitesse est négligeable': The phrase "utilisez celle des API que vous trouvez la plus pratique" is a slightly awkward rendering of "use whichever API you find more convenient"; the split across the code fence makes the sentence read oddly. → "utilisez celle des deux API qui vous convient le mieux."
  • [minor · fluency] lectures/polars.md — ### Eager vs lazy, comment '# Rechargement du jeu de données': The code comment was localized, which is acceptable, but note that other code comments in the same file (e.g., in read_data_polars docstring) were left in English, creating inconsistency in whether comments are translated throughout the Polars lecture. → Apply a consistent policy: either translate all code comments/docstrings in changed sections or leave all as-is.
  • [nit · terminology] lectures/pandas.md — ## Overview / ## Vue d'ensemble: "munging" is translated as "traitement général des données" in the body text but the footnote uses "munging" transliterated as « munging » in quotes — consistent handling is good, but the in-body translation choice (traitement général vs. nettoyage) differs slightly from the footnote's "nettoyage", which is a minor internal inconsistency. → Consider aligning terms between body text and footnote (e.g., both using "nettoyage/traitement des données").

🔍 Diff Quality

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

Summary: The URL changes in both pandas.md and polars.md were correctly applied in the same positions in the French translation, with structure and translation metadata intact.


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

This automated translation-sync PR updates the French pandas and polars lecture content to load the test_pwt.csv Penn World Tables sample dataset from the central QuantEcon/data-lectures repository, aligning with upstream source PR #612.

Changes:

  • Updated read_csv / pl.read_csv URL(s) to https://github.com/QuantEcon/data-lectures/raw/main/lectures/test_pwt.csv in both pandas and polars lectures.
  • Adjusted the pandas lecture text to refer to test_pwt.csv (instead of the previous pandas/data/test_pwt.csv path).
  • Updated translation sync state files to the new upstream source-sha.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
lectures/pandas.md Switches the dataset loading example to read test_pwt.csv from data-lectures and updates the surrounding sentence accordingly.
lectures/polars.md Switches all occurrences of the dataset URL to the data-lectures location.
.translate/state/pandas.md.yml Updates translation-sync metadata (source-sha, synced-at) for pandas lecture.
.translate/state/polars.md.yml Updates translation-sync metadata (source-sha, synced-at) for polars lecture.

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

@mmcky
mmcky merged commit 7db6a23 into main Aug 18, 2026
3 of 5 checks passed
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