🌐 [translation-sync] Fix broken raw GitHub URLs for CSV data files#71
🌐 [translation-sync] Fix broken raw GitHub URLs for CSV data files#71
Conversation
✅ Deploy Preview for majestic-griffin-10b166 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
❌ Translation Quality ReviewVerdict: FAIL | Model: claude-sonnet-4-6 | Date: 2026-03-24 📝 Translation Quality
Summary: The translation quality for the changed sections is generally good in terms of accuracy, fluency, and terminology. However, there is a critical structural error in the python_advanced_features document: the entire Decorators and Descriptors section is duplicated — appearing both in its correct location and again where the Generators section (anchored by '(paf_generators)=') should begin. This duplication is a serious formatting/structural bug that must be fixed. The Generators section content does eventually appear but with the wrong anchor placement. Apart from this critical issue, the translation of the modified sections (Overview, DataFrames, Exercises for pandas; Overview, Iterables and Iterators, * and ** Operators, Decorators and Descriptors, Generators, Exercises for python_advanced_features) is competent and readable. Technical terminology is consistently and correctly translated throughout both documents, following the reference glossary (e.g., 'تولید ناخالص داخلی' for GDP, 'سری زمانی' for time series). Persian text reads naturally with appropriate academic register; sentence structures are well-adapted for RTL reading. Code blocks, LaTeX math, MyST directives, and figure references are all properly preserved without modification. The heading-map in the YAML frontmatter is correctly structured and comprehensive, covering all major sections. Inline code terms (Series, DataFrame, iterator, decorator, generator, etc.) are appropriately kept in English as is standard practice in Persian technical writing.
|
| Check | Status |
|---|---|
| Scope Correct | ✅ |
| Position Correct | ❌ |
| Structure Preserved | ❌ |
| Heading-map Correct | ❌ |
| Overall | 2.5/10 |
Summary: The translation sync has critical structural failures: the python_advanced_features.md target duplicates the entire Decorators and Descriptors section where the Generators section should be, the heading-map uses stale/incorrect heading keys that don't match the updated source headings, and a URL change in pandas.md was not propagated to the target.
Issues:
- python_advanced_features.md target 'After' contains a massively duplicated 'Decorators and Descriptors' section: after the Generators anchor
(paf_generators)=, the entire Decorators and Descriptors section (including all subsections, code cells, and content) is repeated verbatim instead of the Generators section content appearing there. This is a critical structural error. - The heading-map in python_advanced_features.md 'After' is incorrect: it still maps old English source headings (e.g., 'Iterables and iterators', '
*and**operators', 'Decorators and descriptors', 'Generators', etc. with lowercase variants) instead of updating to the new English headings from the source 'After' (e.g., 'Iterables and Iterators', '*and**Operators', 'Decorators and Descriptors', 'Generator Expressions', 'Generator Functions', 'Advantages of Iterators'). The source changed heading capitalizations throughout. - The heading-map also incorrectly maps 'Type hints', 'Type hints::Basic syntax', 'Type hints::Common types' to Decorator/Descriptor translations, which is wrong — the 'Type hints' section was removed from the source 'After', so these entries should be removed, not remapped.
- The heading-map adds 'More Language Features: ویژگیهای بیشتر زبان' which is the h1 title — this is unusual and may be incorrect since the h1 is handled separately from section headings.
- pandas.md heading-map 'After' adds
'{index}\Pandas <single: Pandas>`': '{index}`Pandas <single: Pandas>`'` which maps the h1 title to itself (untranslated) — this is suspicious and likely incorrect since h1 titles are not typically in the heading-map. - The source 'After' for python_advanced_features.md removed the 'Type hints' section entirely, but the target 'After' heading-map still contains 'Type hints' related entries (remapped incorrectly to decorator/descriptor translations).
- The url change in pandas.md (from 'master/source/_static' to 'main/lectures/_static') was NOT reflected in the target — the target still uses the old URL. This is a missed content sync.
This review was generated automatically by action-translation review mode.
Automated Translation Sync
This PR contains automated translations from QuantEcon/lecture-python-programming.
Source PR
#491 - Fix broken raw GitHub URLs for CSV data files
Files Updated
lectures/pandas.md.translate/state/pandas.md.ymllectures/python_advanced_features.md.translate/state/python_advanced_features.md.ymlDetails
This PR was created automatically by the translation action.