Skip to content

Fix broken raw GitHub URLs for CSV data files#491

Merged
mmcky merged 1 commit intomainfrom
fix/broken-csv-urls
Mar 23, 2026
Merged

Fix broken raw GitHub URLs for CSV data files#491
mmcky merged 1 commit intomainfrom
fix/broken-csv-urls

Conversation

@mmcky
Copy link
Contributor

@mmcky mmcky commented Mar 23, 2026

Problem

The cache build is failing because pandas.md tries to fetch test_pwt.csv from a URL that no longer exists:

https://raw.githubusercontent.com/QuantEcon/lecture-python-programming/master/source/_static/...

This returns 404 because the repo was restructured (the .myst suffix was dropped and master became main, source/_static became lectures/_static).

Failed run: https://github.com/QuantEcon/lecture-python-programming/actions/runs/23420771220/job/68125163471

Fix

Updated two broken raw.githubusercontent.com URLs from the old path to the current one:

File Description
lectures/pandas.md test_pwt.csv URL — this causes the cache build failure
lectures/python_advanced_features.md test_table.csv URL — also broken (404)

Old pattern: master/source/_static/lecture_specific/...
New pattern: main/lectures/_static/lecture_specific/...

Both updated URLs have been verified to return HTTP 200.

The repo restructure (drop .myst suffix, master→main) broke two
raw.githubusercontent.com URLs used in code cells:

- pandas.md: test_pwt.csv → 404 (causes cache build failure)
- python_advanced_features.md: test_table.csv → 404

Updated both from the old path:
  master/source/_static/lecture_specific/...
to the current path:
  main/lectures/_static/lecture_specific/...
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes broken raw.githubusercontent.com links in QuantEcon’s lecture content so that examples referencing CSV data files resolve correctly after the repository restructure (mastermain, source/_staticlectures/_static).

Changes:

  • Update the test_table.csv raw GitHub URL in lectures/python_advanced_features.md to the current branch/path.
  • Update the test_pwt.csv raw GitHub URL in lectures/pandas.md to the current branch/path (addresses the reported cache build failure).

Reviewed changes

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

File Description
lectures/python_advanced_features.md Fixes the raw GitHub download URL for test_table.csv used in an exercise.
lectures/pandas.md Fixes the raw GitHub URL used by pd.read_csv(...) to fetch test_pwt.csv.

@github-actions
Copy link

@github-actions github-actions bot temporarily deployed to pull request March 23, 2026 04:42 Inactive
@mmcky mmcky merged commit 02e57a5 into main Mar 23, 2026
9 checks passed
@mmcky mmcky deleted the fix/broken-csv-urls branch March 23, 2026 05:09
@mmcky
Copy link
Contributor Author

mmcky commented Mar 24, 2026

\translate-resync

@mmcky
Copy link
Contributor Author

mmcky commented Mar 24, 2026

✅ Translation sync completed (zh-cn)

Target repo: QuantEcon/lecture-python-programming.zh-cn
Translation PR: QuantEcon/lecture-python-programming.zh-cn#4
Files synced (2):

  • lectures/pandas.md
  • lectures/python_advanced_features.md

@mmcky
Copy link
Contributor Author

mmcky commented Mar 24, 2026

✅ Translation sync completed (fa)

Target repo: QuantEcon/lecture-python-programming.fa
Translation PR: QuantEcon/lecture-python-programming.fa#71
Files synced (2):

  • lectures/pandas.md
  • lectures/python_advanced_features.md

@mmcky
Copy link
Contributor Author

mmcky commented Mar 24, 2026

\translate-resync

@mmcky
Copy link
Contributor Author

mmcky commented Mar 24, 2026

✅ Translation sync completed (zh-cn)

Target repo: QuantEcon/lecture-python-programming.zh-cn
Translation PR: QuantEcon/lecture-python-programming.zh-cn#5
Files synced (2):

  • lectures/pandas.md
  • lectures/python_advanced_features.md

@mmcky
Copy link
Contributor Author

mmcky commented Mar 24, 2026

✅ Translation sync completed (fa)

Target repo: QuantEcon/lecture-python-programming.fa
Translation PR: QuantEcon/lecture-python-programming.fa#72
Files synced (2):

  • lectures/pandas.md
  • lectures/python_advanced_features.md

@mmcky
Copy link
Contributor Author

mmcky commented Mar 24, 2026

\translate-resync

@mmcky
Copy link
Contributor Author

mmcky commented Mar 24, 2026

✅ Translation sync completed (zh-cn)

Target repo: QuantEcon/lecture-python-programming.zh-cn
Translation PR: QuantEcon/lecture-python-programming.zh-cn#9
Files synced (2):

  • lectures/pandas.md
  • lectures/python_advanced_features.md

@mmcky
Copy link
Contributor Author

mmcky commented Mar 24, 2026

✅ Translation sync completed (fa)

Target repo: QuantEcon/lecture-python-programming.fa
Translation PR: QuantEcon/lecture-python-programming.fa#76
Files synced (2):

  • lectures/pandas.md
  • lectures/python_advanced_features.md

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