-
-
Notifications
You must be signed in to change notification settings - Fork 432
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
Broken link #617
Comments
Thanks for letting us know! I think the link in Episode 9 (http://swcarpentry.github.io/python-novice-gapminder/09-plotting/index.html) used to go to a page like this: https://matplotlib.org/3.5.3/api/_as_gen/matplotlib.pyplot.html but I prefer your link as a more durable URL that provides a nicer overview of matplotlib as opposed to an API landing page 😅 . We'd welcome a PR to correct the link in episode 9, it'd be good to adjust all of those similar links if possible. I counted at least 4 of them there from a cursory grep. That matplotlib page is linked twice, once early on in the summary and again when introducing plotting via pandas dataframes, probably that second one could go away..? |
Hi @alee, I recently completed my Instructor Training and I am looking for issues that need to be fixed. Do you still need a hand with this? If I understand correctly, the PR would have updated links. Any other than the three discussed (matplotlib, tick_params, savefig)? |
Absolutely, I don't believe anyone else has put forward a PR or indicated they are up for working on this. Thanks @vhmcck ! |
Thanks, @alee! I will go through the links and update them as necessary. I will let you know when I have submitted the PR with these changes. |
The link for matplotlib.pyplot near the top of the page is broken. The link points to https://matplotlib.org/stable/api/_as_gen/matplotlib.pyplot.html#module-matplotlib.pyplot but this gives me a page not found error. I'm not certain where the link should point, perhaps https://matplotlib.org/stable/tutorials/introductory/pyplot.html ?
The text was updated successfully, but these errors were encountered: