-
-
Notifications
You must be signed in to change notification settings - Fork 30
Fix broken links and redirects in documentation with API endpoint correction #404
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot I have reviewed and request a few minor changes. Thanks.
Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
…ution Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
This PR addresses the broken links and redirects identified in the link checker report. The changes fix 27 broken links and 9 redirect issues across the documentation, with a targeted correction for Python code execution.
Changes Made
Fixed Broken Links
lectures/_config.yml
: Changed fromhttps://www.tomsargent.com/
tohttp://www.tomsargent.com/
as HTTPS is not supported on that domain. Maintains proper author attribution in the site footer.Fixed Redirect Links
Updated outdated URLs to their current destinations to avoid unnecessary redirects:
NumPy links: Updated
https://www.numpy.org/
→https://numpy.org/
in:lectures/pandas.md
lectures/about_py.md
SciPy links: Updated
https://www.scipy.org
→https://scipy.org/
in:lectures/about_py.md
lectures/scipy.md
lectures/numpy.md
FRED database links: Updated
https://research.stlouisfed.org/fred2/
→https://fred.stlouisfed.org/
for documentation and reference links in:lectures/pandas.md
(2 instances)API Endpoint Correction
requests.get()
call usinghttps://research.stlouisfed.org/fred2/series/UNRATE/downloaddata/UNRATE.csv
as the updated endpoint doesn't work for programmatic data access, ensuring code examples remain executable.Impact
The site now builds successfully with only expected warnings (jax.quantecon.org connectivity issues noted as normal in project documentation). All Python code examples remain functional.
Note on Remaining Redirects
The remaining 25
jupyterbook.org
→jupyterbook.org/en/stable/
redirects are generated by the "Powered by Jupyter Book" footer in the theme templates and cannot be easily fixed without modifying the theme itself. These are minor since the links still work correctly.Fixes #403.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.