Skip to content

🌐 [translation-sync] [pandas_panel] Describe the single .replace() call, not a loop - #170

Open
mmcky wants to merge 2 commits into
mainfrom
translation-sync-2026-10-12T01-34-50-pr-645
Open

mmcky wants to merge 2 commits into
mainfrom
translation-sync-2026-10-12T01-34-50-pr-645

Conversation

@mmcky

@mmcky mmcky commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

Automated Translation Sync

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

Source PR

#645 - [pandas_panel] Describe the single .replace() call, not a loop

Files Updated

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

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 12, 2026 01:34
@mmcky mmcky added action-translation PRs created by QuantEcon/action-translation automated Automated sync PR opened by action-translation labels Oct 12, 2026
@netlify

netlify Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

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

Name Link
🔨 Latest commit 50f989a
🔍 Latest deploy log https://app.netlify.com/projects/majestic-griffin-10b166/deploys/6acc393fe46b8000085ffd83
😎 Deploy Preview https://deploy-preview-170--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

Copy link
Copy Markdown

✅ Translation Quality Review

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


📝 Translation Quality

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

Summary: ترجمه بخش «ادغام Dataframe ها و پر کردن NaN ها» به‌طور کلی دقیق، روان و از نظر اصطلاحات فنی مطابق با واژه‌نامه مرجع است. چند مورد جزئی در روانی جملات (به‌ویژه ساختارهای پیچیده انگلیسی که به‌صورت نزدیک به کلمه‌به‌کلمه ترجمه شده‌اند) و یک نکتهٔ کوچک دربارهٔ ترجمه 'Americas' وجود دارد، اما هیچ خطای ساختاری یا مشکل جدی در معنا یافت نشد. اصطلاحات فنی مانند DataFrame، MultiIndex، merge، groupby و xs به‌درستی حفظ شده و در سراسر بخش یکدست هستند ساختار کد، بلوک‌های کد و دستورات MyST بدون خطا و کاملاً مطابق با نسخه اصلی حفظ شده‌اند توضیحات مربوط به left/right/inner/outer join به‌دقت و روشن ترجمه شده‌اند

Suggestions:

  • [minor · fluency] lectures/pandas_panel.md — ## ادغام Dataframe ها و پر کردن NaN ها: عبارت 'ما قاره آمریکا را در یک قاره واحد ترکیب خواهیم کرد' کمی تحت‌اللفظی است و از متن اصلی 'combine the Americas into a single continent' دور شده است؛ 'Americas' به معنای چند قاره آمریکا (شمالی، جنوبی، مرکزی) است نه یک قاره واحد به نام آمریکا. → همچنین قاره‌های آمریکا (شمالی، مرکزی و جنوبی) را در یک قاره واحد ترکیب خواهیم کرد
  • [minor · terminology] lectures/pandas_panel.md — ## ادغام Dataframe ها و پر کردن NaN ها: در جمله «dataframe های ما با استفاده از نام کشورها ادغام خواهند شد، که نیاز داریم از transpose...» ساختار جمله کمی نامتعارف و گنگ است و روانی جمله را کاهش می‌دهد. → dataframe های ما با استفاده از نام کشورها ادغام خواهند شد، که این امر مستلزم استفاده از transpose مربوط به realwage_f است تا ردیف‌ها در هر دو dataframe با نام کشورها مطابقت داشته باشند
  • [nit · fluency] lectures/pandas_panel.md — ## ادغام Dataframe ها و پر کردن NaN ها: عبارت «که key خواهد بود که برای ادغام dataframe ها 'on' آن استفاده می‌شود» کمی دست‌وپاگیر و نزدیک به ترجمه کلمه‌به‌کلمه است. → که همان key است که برای ادغام dataframe ها بر اساس آن استفاده می‌شود

🔍 Diff Quality

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

Summary: The single-sentence edit in the 'Merging Dataframes' section was correctly translated and positioned in the same location without affecting scope, structure, or metadata.


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 one-sentence translation update accurately matches the source code's single .replace() call, and the metadata (SHA, date, tool version) is consistent with the PR details.

0 open findings

What changed in this PR

This is an automated translation-sync PR that propagates upstream English change QuantEcon/lecture-python-programming#645 into the Farsi translation. It updates one sentence in the pandas_panel.md lecture so the prose correctly describes a single .replace() call (passing a list of continent values) instead of looping through them, matching the actual code at line 282 (merged['Continent'].replace(to_replace=replace, value='America')).

Changes:

  • Reworded the Persian description of the .replace() usage to reflect a single call rather than a loop.
  • Bumped translation state metadata (source-sha, synced-at, tool-version).
File Description
lectures/​pandas_panel.md Updates one sentence so the Farsi text matches the source intent: a single .replace() call with a list, not a loop.
.translate/​state/​pandas_panel.md.yml Updates sync state (source SHA, synced-at date, tool version) to record this translation sync.

🧠 Review effort: Balanced


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

@github-actions

Copy link
Copy Markdown

This branch was successfully deployed

1 active deployment
pull request — 50f989a3 Deployed Oct 12, 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