Skip to content

Restore the repointed data URLs reverted by the #154 regeneration - #155

Merged
mmcky merged 1 commit into
mainfrom
restore-repointed-urls
Aug 18, 2026
Merged

Restore the repointed data URLs reverted by the #154 regeneration#155
mmcky merged 1 commit into
mainfrom
restore-repointed-urls

Conversation

@mmcky

@mmcky mmcky commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

#154 was regenerated from its source PR's merge-time snapshot, which predates the test_pwt repoint this repo synced in #153 — so merging it reverted two polars.md reads to lecture-python-programming's old own-repo URL (a path scheduled for deletion, which would 404 these cells at runtime) and rewrote three pandas_panel.md reads (realwage/countries/employ) from the canonical github.com/QuantEcon/data-lectures/raw/main/… form to the raw.githubusercontent.com host form.

This restores all five reads to match the English source on lecture-python-programming main. URL-only — no translated text is touched, so there is nothing for the translation pipeline to re-review. The underlying regeneration defect is recorded in QuantEcon/action-translation#276.

🤖 Generated with Claude Code

The #154 sync was regenerated from its source PR's merge-time snapshot,
which predates the test_pwt repoint synced here in #153 — so it
reverted two polars.md reads to the source repo's old own-repo URL (a
path scheduled for deletion) and rewrote three pandas_panel.md reads
to the raw.githubusercontent host form. This restores all five reads
to the canonical data-lectures form the English source uses. URL-only;
no translated text changes. The underlying regeneration defect is
recorded in QuantEcon/action-translation#276.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 18, 2026 03:54
@netlify

netlify Bot commented Aug 18, 2026

Copy link
Copy Markdown

Deploy Preview for majestic-griffin-10b166 ready!

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

Restores dataset download URLs in the Persian lecture markdown files so they match the canonical locations used by the upstream English source (avoiding reverted URLs that could 404 after scheduled deletions, and standardizing the host form for data-lectures reads).

Changes:

  • Repoints two test_pwt.csv reads in lectures/polars.md back to https://github.com/QuantEcon/data-lectures/raw/main/lectures/test_pwt.csv.
  • Rewrites three data-lectures CSV reads in lectures/pandas_panel.md to use the github.com/.../raw/... canonical form (realwage / countries / employ).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
lectures/polars.md Restores test_pwt.csv reads to data-lectures to avoid old in-repo URLs that may be removed.
lectures/pandas_panel.md Normalizes data-lectures CSV URLs to the canonical github.com/.../raw/... form.

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

@mmcky
mmcky merged commit 76022e8 into main Aug 18, 2026
6 of 7 checks passed
@mmcky
mmcky deleted the restore-repointed-urls branch August 18, 2026 04:00
mmcky added a commit that referenced this pull request Aug 19, 2026
…URL form (#158)

Restore .translate/state/{pandas,polars}.md.yml to the versions #153 wrote (source-sha 55c87c9f, the Track D repoint) — #154 overwrote them with the stale snapshot's SHA (d35eb831), making isSourceChanged over-report both lectures as pending. Also revert the pandas_panel.md half of #155: the three data reads return to the raw.githubusercontent form, which is what source main itself serves at its last touch of that file — the regeneration's rewrite there was correct sync content, not stale damage.

Measurement and ledger: QuantEcon/action-translation#276.

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
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