🌐 [translation-sync] Read test_pwt.csv from data-lectures (Track D) - #33
🌐 [translation-sync] Read test_pwt.csv from data-lectures (Track D)#33mmcky wants to merge 4 commits into
Conversation
✅ Translation Quality ReviewVerdict: PASS | Model: claude-sonnet-5 | Date: 2026-08-18 📝 Translation Quality
Summary: The translation of the modified sections is fluent and terminologically consistent with the glossary, but contains a few accuracy-critical code deviations from the English source (reverted rng usage in pandas.md and polars.md Series sections, a fabricated/incorrect URL in the requests section, and a missing Suggestions:
🔍 Diff Quality
Summary: The translation sync correctly applied all source changes (CSV URL updates, np.random to rng migration in the big-data benchmark, isinstance/pd.isna refactors, and drop_nulls removal) in the same positions within both target files. This review was generated automatically by action-translation review mode. |
There was a problem hiding this comment.
Pull request overview
This automated translation-sync PR updates the French lecture content to match upstream changes from QuantEcon/lecture-python-programming PR #612, primarily switching the test_pwt.csv dataset source to the QuantEcon/data-lectures repository and aligning related example code.
Changes:
- Update
test_pwt.csvloading URLs in both the pandas and polars lectures to usehttps://github.com/QuantEcon/data-lectures/raw/main/lectures/test_pwt.csv. - Modernize NumPy random generation in the polars lecture to use
np.random.default_rng(42). - Adjust pandas examples to use more robust idioms (
df.where(... )defaulting toNaN,isinstance,pd.isna).
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| lectures/pandas.md | Switch test_pwt.csv source URL to data-lectures and refine a few pandas examples for correctness/robustness. |
| lectures/polars.md | Switch test_pwt.csv source URL to data-lectures, update RNG usage, and minor formatting normalization. |
| .translate/state/pandas.md.yml | Update translation-sync state metadata (source SHA, synced date, model, mode, tool version). |
| .translate/state/polars.md.yml | Update translation-sync state metadata (source SHA, synced date, model, mode, tool version). |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
0f90967 to
a6a46ee
Compare
|
♻️ Automatically rebased after #27 was merged. Overlapping files: The translation content is preserved; only unchanged sections were updated to match the current |
|
Superseded: conflicts with fr#29 and the automatic rebase failed. A regenerated sync PR from current main replaces this one. |
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.ymllectures/polars.md.translate/state/polars.md.ymlDetails
This PR was created automatically by the translation action.