Skip to content

🌐 [translation-sync] Retire the last references to the retired lecture-python repo#8

Closed
mmcky wants to merge 2 commits into
mainfrom
translation-sync-2026-07-16T04-04-05-pr-576
Closed

🌐 [translation-sync] Retire the last references to the retired lecture-python repo#8
mmcky wants to merge 2 commits into
mainfrom
translation-sync-2026-07-16T04-04-05-pr-576

Conversation

@mmcky

@mmcky mmcky commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Automated Translation Sync

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

Source PR

#576 - Retire the last references to the retired lecture-python repo

Files Updated

  • ✏ lectures/pandas_panel.md
  • ✏ .translate/state/pandas_panel.md.yml

Details

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

This PR was created automatically by the translation action.

Copilot AI review requested due to automatic review settings July 16, 2026 04:04
@mmcky mmcky added action-translation PRs created by QuantEcon/action-translation automated Automated sync PR opened by action-translation labels Jul 16, 2026
@github-actions

Copy link
Copy Markdown

✅ Translation Quality Review

Verdict: PASS | Model: claude-sonnet-5 | Date: 2026-07-16


📝 Translation Quality

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

Summary: The translation of the modified sections (Slicing and Reshaping Data, Merging Dataframes and Filling NaNs, Exercises) is accurate, fluent, and well-formatted, with correct handling of technical terms and code blocks. Minor issues include an unauthorized addition of 'future_stack=True' annotations not in the source, a slight mistranslation of 'removed' as 'déprécié', and small phrasing choices that could be refined for better precision and fluency. Consistent and accurate rendering of technical pandas/statistical terminology (MultiIndex, DataFrame, pivot_table, groupby, etc.) left appropriately untranslated as code identifiers Faithful preservation of code cells, markdown structure, and MyST directives across all modified sections Natural French phrasing for explanatory prose, e.g., 'diviser-appliquer-combiner' for 'split-apply-combine' fits French academic register well

Suggestions:

  • Slicing and Reshaping Data section: The translation adds 'future_stack=True' to .stack() calls and inline comments explaining it, which are not present in the English source. While this reflects a real pandas API change, it constitutes added content not authorized by the source text and should be flagged/verified against the intended scope of this translation task.
  • Grouping and Summarizing Data section (technically unchanged, but check consistency): 'as pandas has removed support for axis=1' was translated as 'car pandas a dĂ©prĂ©ciĂ© l'utilisation de axis=1' - 'dĂ©prĂ©ciĂ©' (deprecated) is not equivalent to 'removed' (supprimĂ©/retirĂ©); this changes the technical meaning since the English states the feature was removed, not merely deprecated.
  • Merging Dataframes and Filling NaNs section: 'the key that is used to merge the dataframes on' is translated as 'la key utilisĂ©e pour fusionner les dataframes (« on »)' - the parenthetical rendering of 'on' as a standalone quoted term is awkward; consider 'la clĂ© sur laquelle les dataframes seront fusionnĂ©s' for better fluency.
  • Slicing and Reshaping Data section: 'hourly real minimum wages across countries and time' translated as 'salaires minimums rĂ©els horaires par pays et par pĂ©riode' - 'par pĂ©riode' loses the specific meaning of 'time' as the temporal dimension matching the panel structure; 'dans le temps' or 'au fil du temps' would align better with dataset structure terminology used elsewhere.

🔍 Diff Quality

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

Summary: The stack()/future_stack additions were correctly synced in position and structure, but the source's URL path updates (lecture-python -> lecture-python-programming) were not applied to the target document's three code cells.

Issues:

  • The URLs (url1, url2, url3) in the target document were NOT updated to match the source's new repo path (lecture-python-programming/main/lectures/...); they still point to the old lecture-python/master/source/... path. This means the translation sync missed updating these three code-cell URL changes, even though the +9/-9 diff count suggests something else was changed instead (likely the .stack() calls) rather than the URL updates.

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 lecture content to remove remaining references to the retired QuantEcon/lecture-python repository by switching dataset download URLs to QuantEcon/lecture-python-programming.

Changes:

  • Updated three raw GitHub URLs in lectures/pandas_panel.md to point at QuantEcon/lecture-python-programming instead of the retired repo.
  • Updated translation-sync state metadata (source-sha, synced-at, model, mode, tool-version) for pandas_panel.md.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
lectures/pandas_panel.md Repoints dataset URLs to the active upstream repository and applies minor punctuation/spacing edits.
.translate/state/pandas_panel.md.yml Records the new sync provenance (source SHA/date/model/tool version).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lectures/pandas_panel.md
pays et l'assigner Ă  `realwage`.

L'ensemble de données est accessible via le lien suivant :
L'ensemble de données est accessible via le lien suivant :
Comment thread lectures/pandas_panel.md
le continent de chaque pays Ă  `realwage_f` avec la fonction `merge`.

L'ensemble de données est accessible via le lien suivant :
L'ensemble de données est accessible via le lien suivant :
Comment thread lectures/pandas_panel.md

Nous pouvons utiliser une jointure left, right, inner ou outer pour fusionner nos
ensembles de données :
ensembles de données :
Comment thread lectures/pandas_panel.md
```

Nous avons trois valeurs manquantes !
Nous avons trois valeurs manquantes !
Comment thread lectures/pandas_panel.md
d'emploi en Europe par Ăąge et par sexe provenant d'[Eurostat](https://ec.europa.eu/eurostat/data/database).

L'ensemble de données est accessible via le lien suivant :
L'ensemble de données est accessible via le lien suivant :
@mmcky

mmcky commented Jul 18, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #12, now merged. Upstream, source PR QuantEcon/lecture-python-programming#578 built on QuantEcon/lecture-python-programming#576 and moved the same three dataset URLs on to their final data-lectures form, so #12 carries the end state of the lines this PR touches, and its sync state records the newer source-sha (4cacd9a). Merging this PR after #12 would conflict and regress the URLs, so it is closed unmerged — no content is lost. (Both PRs also carry the v0.16.1 typography stripping on five prose lines — action-translation#97 — which will be restored in a follow-up in the style of #9.)

@mmcky mmcky closed this Jul 18, 2026
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