Skip to content

DOC: Data links in Pandas API Reference are broken 404 #33428

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

Closed
Hackathorn opened this issue Apr 9, 2020 · 7 comments
Closed

DOC: Data links in Pandas API Reference are broken 404 #33428

Hackathorn opened this issue Apr 9, 2020 · 7 comments

Comments

@Hackathorn
Copy link

Location of the documentation

https://pandas.pydata.org/docs/reference/api/pandas.plotting.parallel_coordinates.html
...probably many examples in other sections

Documentation problem

Results in 404 not found error
df = pd.read_csv('https://raw.github.com/pandas-dev/pandas/master'
'/pandas/tests/data/csv/iris.csv')

Suggested fix for documentation

The GitHub site should be "raw.githubusercontent.com"

@Hackathorn Hackathorn added Docs Needs Triage Issue that has not been reviewed by a pandas team member labels Apr 9, 2020
@TomAugspurger
Copy link
Contributor

Thanks for the report, a pull request to fix it would be welcome.

@TomAugspurger TomAugspurger added this to the Contributions Welcome milestone Apr 9, 2020
@TomAugspurger TomAugspurger added good first issue and removed Needs Triage Issue that has not been reviewed by a pandas team member labels Apr 9, 2020
@jackattackyang
Copy link

take

@jackattackyang jackattackyang removed their assignment Apr 10, 2020
@jackattackyang
Copy link

The correct link is
https://raw.github.com/pandas-dev/pandas/master/pandas/tests/data/iris.csv

However doc string for parallel_coordinates actually contains the correct link. The referencing might be the problem. First time contributing and unfortunately running into problems with locally building pandas on Windows. Looking into it but have removed my assignment in the meantime.

@konstell
Copy link

The path in the current doc is incorrect, but not because it has the incorrect domain name. The second line of the read_csv command should say '/pandas/tests/data/iris.csv' instead of '/pandas/tests/data/csv/iris.csv'

@konstell
Copy link

take

@konstell
Copy link

konstell commented Apr 14, 2020

Upon further investigation, this has already been fixed in another PR 15 days ago: #33099

I did a search in the codebase for the exact match of data/csv/iris.csv and didn't find any other case.

@datapythonista
Copy link
Member

Fixed in #33099

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants