Skip to content

🌐 [translation-sync] Correct two typos in exercise text (functions, python_by_example) - #176

Merged
mmcky merged 4 commits into
mainfrom
translation-sync-2026-11-02T00-34-22-pr-638
Nov 2, 2026
Merged

mmcky merged 4 commits into
mainfrom
translation-sync-2026-11-02T00-34-22-pr-638

Conversation

@mmcky

@mmcky mmcky commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Automated Translation Sync

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

Source PR

#638 - Correct two typos in exercise text (functions, python_by_example)

Files Updated

  • ✏️ lectures/functions.md
  • ✏️ .translate/state/functions.md.yml
  • ✏️ lectures/python_by_example.md
  • ✏️ .translate/state/python_by_example.md.yml

Details

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

This PR was created automatically by the translation action.

@mmcky mmcky added action-translation PRs created by QuantEcon/action-translation automated Automated sync PR opened by action-translation labels Nov 2, 2026
Copilot AI balanced review requested due to automatic review settings November 2, 2026 00:34
@netlify

netlify Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for majestic-griffin-10b166 ready!

Name Link
🔨 Latest commit 1352b1d
🔍 Latest deploy log https://app.netlify.com/projects/majestic-griffin-10b166/deploys/6ae7da96e3c5295ef0c2792f
😎 Deploy Preview https://deploy-preview-176--majestic-griffin-10b166.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown

✅ Translation Quality Review

Verdict: PASS | Model: claude-sonnet-5 | Date: 2026-11-02
Routing: editor — 2 minor finding(s) in gating categories (accuracy/terminology/syntax/diff-check/other); accuracy 8 below floor 9; terminology 8 below floor 9; fluency 7 below floor 8
Shadow gate: would NOT auto-merge (recorded only; no action taken)


📝 Translation Quality

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

Summary: The Exercises section translation is accurate and well-structured, with all code, math, and MyST syntax intact. Minor fluency issues exist in a few sentences (redundant phrasing, awkward word order, an unconventional transliteration) and one formatting inconsistency where Persian words were embedded inside a LaTeX-style math expression. These are minor to nit-level issues that do not impede comprehension but could be polished for better readability and consistency. Exercise and solution sections preserve all code blocks, LaTeX, and MyST directive structure correctly Technical vocabulary (e.g., مونت‌کارلو, توزیع یکنواخت دومتغیره) is consistent with the glossary Overall meaning of each exercise and hint is accurately conveyed despite minor phrasing issues

Suggestions:

  • [minor · fluency] lectures/python_by_example.md — pbe_ex1 exercise description: The phrase 'دستورات import را تنها به موارد زیر محدود و تنها از کتابخانه های زیر استفاده کنید' is redundant and awkward — it repeats 'تنها' (only) twice and combines two clauses clumsily, unlike the clean English 'restrict your import statements to'. → در راه حل خود، دستورات import را تنها به موارد زیر محدود کنید:
  • [minor · accuracy] lectures/python_by_example.md — pbe_ex3 exercise text: 'همانند تمرینات قبلی، سریزمانی را رسم کنید.' ends with a period instead of a colon before introducing the equation, and 'سریزمانی' is missing a space (should be 'سری‌زمانی' or 'سری زمانی'). This is a minor typo inherited from source formatting but affects readability. → همانند تمرینات قبلی، سری زمانی زیر را رسم کنید:
  • [minor · accuracy] lectures/python_by_example.md — pbe_ex3, 'Search online for a function...': 'آنلاین برای تابعی که می‌تواند برای محاسبه قدر مطلق $|x_t|$ استفاده شود جستجو کنید.' is a slightly awkward/unnatural word order for 'Search online for...'; 'آنلاین' placed at the very start reads unnaturally in Persian academic style. → برای یافتن تابعی که بتوان با آن قدر مطلق $|x_t|$ را محاسبه کرد، به صورت آنلاین جستجو کنید.
  • [minor · formatting] lectures/python_by_example.md — pbe_ex5 hint block, 'مساحت = \pi * شعاع^2': Mixing Persian words directly into a LaTeX math expression (مساحت = \pi * شعاع^2) breaks the math rendering convention used in the rest of the document, where variable names stay in Latin script inside math mode. → Use: $area = \pi \times radius^2$ (کلمات فارسی را از داخل عبارت ریاضی حذف کنید و در صورت نیاز در متن توضیح دهید)
  • [nit · fluency] lectures/python_by_example.md — pbe_ex4, 'برنچینگ (branching) و شرط ها (conditions)': The transliteration 'برنچینگ' for 'branching' is unusual and less standard than a more descriptive Persian equivalent or at least consistent formatting with other parenthetical English terms used elsewhere in the document. → شاخه‌بندی (branching) و شرط‌ها (conditions)

🔍 Diff Quality

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

Summary: The single-line text correction inside the pbe_ex2 hint block was correctly applied in the same location in the target document without any structural or scope issues.


This review was generated automatically by action-translation review mode.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 sync accurately mirrors two upstream typo fixes with internally consistent, current metadata and no defects surfaced in review.

No open issues 🎉

What changed in this PR

This PR is an automated translation sync that propagates two upstream typo corrections (from source PR QuantEcon/lecture-python-programming#638) into the Farsi (fa) translations. The source changes fixed English grammar in two lecture exercises; this PR updates the corresponding Persian text and the translation-state metadata.

Changes:

  • Refined the python_by_example.md legend hint to drop the redundant "متغیر" (variable), matching the source fix "note that if var = 42".
  • Updated .translate/state/*.yml metadata (source-sha, synced-at, tool-version) for both synced lectures; functions.md's existing translation already reflected the source fix.
File Description
lectures/​python_by_example.md Removed redundant "متغیر" so the legend hint aligns with the corrected English ("if var = 42").
.translate/​state/​python_by_example.md.yml Bumped source-sha to cadd911…, synced-at to 2026-11-02, tool-version 0.26.0 → 0.30.0.
.translate/​state/​functions.md.yml Bumped source-sha to cadd911…, synced-at to 2026-11-02, tool-version 0.25.0 → 0.30.0.

Both translation edits accurately reflect the upstream typo corrections, the metadata is internally consistent (matching the PR's declared sourceCommitSha), and the synced-at date is current rather than in the future. No defects found.

Review effort: Balanced

@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown

@github-actions
github-actions Bot temporarily deployed to pull request November 2, 2026 00:38 Inactive
@mmcky
mmcky merged commit db10c77 into main Nov 2, 2026
10 checks passed
@mmcky
mmcky deleted the translation-sync-2026-11-02T00-34-22-pr-638 branch November 2, 2026 01:22

This branch was previously deployed

1 inactive deployment
pull request — 1352b1da Deployed Nov 2, 2026 by github-actions[bot]
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