Skip to content

Advance about_py state to d35eb831 — the content was already current - #160

Merged
mmcky merged 1 commit into
mainfrom
recover/about-py-state
Aug 20, 2026
Merged

Advance about_py state to d35eb831 — the content was already current#160
mmcky merged 1 commit into
mainfrom
recover/about-py-state

Conversation

@mmcky

@mmcky mmcky commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Recovers the sync failures recorded as lecture-python-programming#610 and #614without re-translating anything.

What was actually wrong

Only the state file. Source PR #595 changed exactly two lines of about_py.md:

-Pytorch is just one of several Python libraries for deep learning and AI.
+PyTorch is just one of several Python libraries for deep learning and AI.

a capitalisation fix in two places. That content is already present here, applied by hand in #124 on 2026-06-19 — this edition carries PyTorch three times and Pytorch zero times, matching source. And source has not touched the lecture since #595. So the lecture was current and only its source-sha was stale, at 3213613a (2026-05-09).

This PR advances it to d35eb831 and stamps synced-at. model, tool-version and section-count are deliberately left as they were, since they record how the text was actually produced and nothing was regenerated.

Why not translate forward

It was attempted first, and discarded. Two reasons:

  1. It rewrote 266 lines to deliver a two-character casing fix that was already applied.
  2. It dropped translator content. The tool's own triage flagged it before writing: "TARGET has content not in SOURCE — the resync will remove it … The Persian translation adds a sentence not in source ('نمودار نشان می دهد که با رشد سریع PyTorch محبوبیت MATLAB در حال کمرنگ شدن است…') and elaborates on the PyTorch/MATLAB chart description beyond the source text." After the run it reported TARGET has additions that were lost during resync, and the sentence was gone.

git log traces that content to Zahra's 2025-10-30 commit (#14, "update about_py and python_by_example + image fixes") — translator work, not machine output. Per the 2026-08-20 ruling on action-translation#276, a natively-translated lecture is not regenerated when a deterministic repair will do; the same reasoning that had numpy.md hand-ported on the French edition in lecture-python-programming.fr#38.

Worth noting for the engine

Both #610 and #614 are structural-parity failures on this lecture, and #614 was filed by the \translate-resync retry of the run that filed #610 — the two bodies differ only by every output line number shifting by one. The retry reproduced the failure rather than clearing it, which is part of why resync is no longer the documented recovery route.

about_py.md is also the file that repeatedly fails the rebase automation with structural parity check failed, leaving sibling PRs un-rebasable. This PR does not address that; it only removes the false "behind" signal.

Recovers the sync failures recorded as lecture-python-programming#610 and
#614, without re-translating anything.

Source PR #595 changed exactly two lines of about_py.md — the capitalisation
`Pytorch` -> `PyTorch` in two places. That fix is already present here,
applied by hand in #124 on 2026-06-19: the target carries `PyTorch` three
times and `Pytorch` zero times, matching source. The source has not moved on
this lecture since #595, so only the state file was stale.

A `translate forward` regeneration was attempted first and deliberately
discarded. It rewrote 266 lines to deliver a two-character casing fix that
was already applied, and it dropped translator content — the tool's own
triage flagged an added PyTorch/MATLAB sentence and a Java code comment as
"TARGET has additions that were lost during resync", and `git log` traces
that content to Zahra's 2025-10-30 commit (#14). Per the 2026-08-20 ruling on
QuantEcon/action-translation#276, a natively-translated lecture is not
regenerated when a deterministic repair will do.

`model`, `tool-version` and `section-count` are deliberately left recording
how the text was actually produced, since nothing was regenerated.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 20, 2026 02:12
@netlify

netlify Bot commented Aug 20, 2026

Copy link
Copy Markdown

Deploy Preview for majestic-griffin-10b166 ready!

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

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.

Pull request overview

Updates the translation-sync state metadata for about_py.md so this Persian edition no longer appears “behind” the source when the lecture content is already up to date.

Changes:

  • Advance source-sha for about_py.md to d35eb831….
  • Update synced-at to 2026-08-20 while leaving the generation metadata (model, mode, section-count, tool-version) unchanged.

💡 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

@github-actions
github-actions Bot temporarily deployed to pull request August 20, 2026 02:16 Inactive
@mmcky
mmcky merged commit 71e4c66 into main Aug 20, 2026
8 checks passed
@mmcky
mmcky deleted the recover/about-py-state branch August 20, 2026 03:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants