-
-
Notifications
You must be signed in to change notification settings - Fork 30
FIX:116 redirected URLs in documentation to eliminate unnecessary redirects (qe-link-checker w. copilot) #389
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 <[email protected]>
lectures/pandas.md
Outdated
* solve your proxy problem by reading [the documentation](https://requests.readthedocs.io/en/latest/) | ||
|
||
Assuming that all is working, you can now proceed to use the `source` object returned by the call `requests.get('https://research.stlouisfed.org/fred2/series/UNRATE/downloaddata/UNRATE.csv')` | ||
Assuming that all is working, you can now proceed to use the `source` object returned by the call `requests.get('https://fred.stlouisfed.org/series/UNRATE/downloaddata/UNRATE.csv')` |
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 can you please open an issue with this link as this simple update isn't working.
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 can you please revert this one change, once the issue is raised.
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.
I don't have the capability to open GitHub issues directly. Could you please open the issue yourself, or would you like me to document the FRED URL problem in a comment within the code for tracking? The current FRED URLs in pandas.md appear to work correctly, but if there's a specific issue you've encountered, I can help investigate or create a code comment to track it.
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.
I've reverted the FRED URL changes back to the original research.stlouisfed.org/fred2/
format as requested. Unfortunately, I don't have the capability to open GitHub issues directly, but I've documented this in the PR description as needing an issue to be opened. The changes are in commit a3f2e7b.
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.
I have made some suggested changes to this, in addition to needing to open an issue.
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:
|
…r feedback Co-authored-by: mmcky <[email protected]>
This PR addresses 116 of the 119 redirected URLs identified by the automated link checker to improve documentation performance and reliability by pointing directly to final destinations. Three FRED URLs were reverted per maintainer feedback and require a separate issue to be tracked.
Key URL Updates
JAX-related URLs:
github.com/google/jax
togithub1.v50.ltd/jax-ml/jax
jax.readthedocs.io
todocs.jax.dev
Data source URLs:
research.stlouisfed.org/fred2/
format per maintainer feedback (requires separate issue)Documentation sites:
docs.scipy.org/doc/numpy/
tonumpy.org/doc/stable/
/en/master/
to/en/latest/
nbviewer.jupyter.org
tonbviewer.org
Platform URLs:
/kernels
to/code
pydata/pandas
topandas-dev/pandas
QuantEcon URLs:
python.quantecon.org/index.html
tointro.quantecon.org/intro.html
to point to correct Introduction to Quantitative Economics seriesGeneral HTTP to HTTPS conversions:
Testing
This change improves documentation load times by avoiding 1-3 redirects per affected URL and ensures links remain stable as services update their URL structures. The three reverted FRED URLs will need to be addressed in a separate issue as they may require additional investigation.
Fixes #387.
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.