Skip to content

🌐 [translation-sync] TEST: Sub-subsection added (####) (10 - lecture) - #384

Closed
mmcky wants to merge 2 commits into
mainfrom
translation-sync-2026-10-09T04-37-01-pr-986
Closed

mmcky wants to merge 2 commits into
mainfrom
translation-sync-2026-10-09T04-37-01-pr-986

Conversation

@mmcky

@mmcky mmcky commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Automated Translation Sync

This PR contains automated translations from QuantEcon/test-translation-sync.

Source PR

#986 - TEST: Sub-subsection added (####) (10 - lecture)

Files Added

  • ✅ .translate/state/lecture.md.yml

Files Updated

  • ✏️ lecture.md

Details

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

This PR was created automatically by the translation action.

Copilot AI balanced review requested due to automatic review settings October 9, 2026 04:37
@mmcky mmcky added action-translation PRs created by QuantEcon/action-translation automated Automated sync PR opened by action-translation labels Oct 9, 2026
@github-actions

github-actions Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

✅ Translation Quality Review

Verdict: PASS | Model: claude-sonnet-5 | Date: 2026-10-09
Routing: editor — accuracy 8 below floor 9; fluency 7 below floor 8


📝 Translation Quality

Criterion Score
Accuracy 8/10
Fluency 7/10
Terminology 9/10
Formatting 10/10
Overall 8.3/10

Summary: The translation of the modified and added sections (frontmatter, Applications in Economics, and new Closure Property section) is accurate and preserves all formatting and math correctly. Minor fluency issues exist in a couple of sentences where word order feels slightly unnatural, but overall meaning and terminology are solid and consistent with the glossary. New 'Closure Property' section is translated with correct mathematical notation and preserved LaTeX/MyST blocks Terminology is consistent with glossary entries (e.g., 'ویژگی بسته بودن' as a clear rendering of 'Closure Property') Frontmatter translation metadata correctly added without disrupting YAML structure

Suggestions:

  • [minor · fluency] lecture.md — #### ویژگی بسته بودن: The phrase 'ترکیب بسته‌های مصرفی یا برنامه‌های تولیدی امکان‌پذیر، به نتیجه‌ای امکان‌پذیر دیگر منجر می‌شود' places 'امکان‌پذیر' awkwardly after the nouns it modifies, making the sentence harder to parse than the English original ('combining feasible consumption bundles or production plans results in another feasible outcome'). → ترکیب بسته‌های مصرفی امکان‌پذیر یا برنامه‌های تولیدی امکان‌پذیر، نتیجه‌ای امکان‌پذیر دیگر را به همراه دارد.
  • [minor · fluency] lecture.md — #### کاربردها در اقتصاد: 'ترکیب‌های تخصیص‌های امکان‌پذیر، امکان‌پذیر باقی می‌مانند' is a slightly repetitive and stiff rendering of 'combinations of feasible allocations remain feasible'; could be smoothed for better readability. → ترکیب تخصیص‌های امکان‌پذیر، امکان‌پذیر باقی می‌ماند
  • [nit · fluency] lecture.md — #### ویژگی بسته بودن: 'برای هر بردارهای $\mathbf{u}, \mathbf{v} \in V$' mixes singular 'هر' with plural 'بردارهای' which is grammatically inconsistent in Persian; typically 'هر' is used with singular nouns. → برای هر بردار $\mathbf{u}, \mathbf{v} \in V$ و اسکالرهای $\alpha, \beta \in \mathbb{R}$

🔍 Diff Quality

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

Summary: The new 'Closure Property' subsection was correctly inserted in the matching location with proper heading level and corresponding translation metadata entry.


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

The automated translation and its sync-state metadata are internally consistent and exactly match the verified English source change in PR #986.

0 open findings

What changed in this PR

This is an automated translation sync PR that propagates a documentation change from the upstream English repository (QuantEcon/test-translation-sync#986) into the Farsi (fa) translation. The source PR added a new #### Closure Property sub-subsection to the linear algebra lecture, and this PR adds the corresponding translated section plus its heading mapping and sync-state metadata.

Changes:

  • Added the Vector Spaces::Basic Properties::Closure Property: ویژگی بسته بودن entry to the translation.headings front matter and the translated #### ویژگی بسته بودن section body in lecture.md.
  • Added a new .translate/state/lecture.md.yml tracking the sync (source-sha, mode UPDATE, section-count 3, etc.).
File Description
lecture.md Adds the heading-map entry and the translated sub-subsection matching the English source PR #986 (content, math blocks, and heading hierarchy preserved).
.translate/​state/​lecture.md.yml New per-file sync state; source-sha matches the PR metadata, section-count: 3 matches the 3 top-level headings, synced-at is today's date.

I verified the translation maps 1:1 to the English source diff in PR #986, the added heading-map value (ویژگی بسته بودن) matches the in-body #### heading, the source-sha matches the PR's sourceCommitSha, section-count: 3 correctly reflects the three top-level heading keys, and synced-at is not a future date. No issues found.

🧠 Review effort: Balanced


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

@mmcky

mmcky commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

Closing for test reset

@mmcky mmcky closed this Oct 9, 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