Skip to content
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

DOC: redirect from_csv search #28075

Merged
merged 2 commits into from
Sep 28, 2019

Conversation

TomAugspurger
Copy link
Contributor

A web search for "DataFrame from CSV" leads to
https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.from_csv.html

which is currently a 404, since DataFrame.from_csv was removed. Redirect
it to read_csv.

Normally we shouldn't worry about this, but I suspect this is a common search term.

A web search for "DataFrame from CSV" leads to
https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.from_csv.html

which is currently a 404, since DataFrame.from_csv was removed. Redirect
it to read_csv.
@WillAyd
Copy link
Member

WillAyd commented Aug 23, 2019

Not super familiar with this file but aren't there already redirects for from_csv therein? Maybe just need to be updated?

@TomAugspurger
Copy link
Contributor Author

TomAugspurger commented Aug 23, 2019 via email

@WillAyd
Copy link
Member

WillAyd commented Aug 25, 2019

Cool thanks for confirming

@WillAyd
Copy link
Member

WillAyd commented Aug 25, 2019

I assume this is for 1.0.0 but merge for whichever milestone you feel best

@WillAyd WillAyd added this to the 1.0 milestone Aug 25, 2019
Copy link
Member

@gfyoung gfyoung left a comment

Choose a reason for hiding this comment

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

I think 0.25.2 makes sense. No need to wait for this.

@WillAyd
Copy link
Member

WillAyd commented Sep 5, 2019

@TomAugspurger did you want to merge this?

@TomAugspurger
Copy link
Contributor Author

TomAugspurger commented Sep 5, 2019 via email

@@ -1579,3 +1579,6 @@ generated/pandas.unique,../reference/api/pandas.unique
generated/pandas.util.hash_array,../reference/api/pandas.util.hash_array
generated/pandas.util.hash_pandas_object,../reference/api/pandas.util.hash_pandas_object
generated/pandas.wide_to_long,../reference/api/pandas.wide_to_long

# Cached searches
reference/api/pandas.DataFrame.from_csv.html,../reference/api/pandas.read_csv
Copy link
Member

Choose a reason for hiding this comment

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

I think the .html is not needed here (at leas the ones above don't have it)

@jorisvandenbossche
Copy link
Member

I fixed the redirect, and tested locally that this works.

@jorisvandenbossche jorisvandenbossche merged commit c4489cb into pandas-dev:master Sep 28, 2019
josibake pushed a commit to josibake/pandas that referenced this pull request Sep 29, 2019
A web search for "DataFrame from CSV" leads to
https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.from_csv.html
which is currently a 404, since DataFrame.from_csv was removed. Redirect
it to read_csv.

Co-authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
proost pushed a commit to proost/pandas that referenced this pull request Dec 19, 2019
A web search for "DataFrame from CSV" leads to
https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.from_csv.html
which is currently a 404, since DataFrame.from_csv was removed. Redirect
it to read_csv.

Co-authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
proost pushed a commit to proost/pandas that referenced this pull request Dec 19, 2019
A web search for "DataFrame from CSV" leads to
https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.from_csv.html
which is currently a 404, since DataFrame.from_csv was removed. Redirect
it to read_csv.

Co-authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
bongolegend pushed a commit to bongolegend/pandas that referenced this pull request Jan 1, 2020
A web search for "DataFrame from CSV" leads to
https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.from_csv.html
which is currently a 404, since DataFrame.from_csv was removed. Redirect
it to read_csv.

Co-authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants