Skip to content

REG: Restore read_csv function for some file-likes #32577

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

Merged
merged 1 commit into from
Mar 11, 2020

Conversation

gfyoung
Copy link
Member

@gfyoung gfyoung commented Mar 10, 2020

Restore read_csv function for some file-likes

Restores behavior down to the fact that the Python engine cannot handle NamedTemporaryFile.

Closes #31819

Credit to @sasanquaneuf for originating idea.

@gfyoung gfyoung added Regression Functionality that used to work in a prior pandas version IO CSV read_csv, to_csv labels Mar 10, 2020
@gfyoung gfyoung added this to the 1.0.2 milestone Mar 10, 2020
Copy link
Contributor

@TomAugspurger TomAugspurger left a comment

Choose a reason for hiding this comment

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

Thanks @gfyoung.

cc @paihu. Anyone else that should take a look at this?

@gfyoung gfyoung force-pushed the file-like-obj-restore branch from cd1c3a4 to df83963 Compare March 10, 2020 20:30
Restores behavior down to the fact that the
Python engine cannot handle NamedTemporaryFile.

Closes pandas-dev#31819

Credit to @sasanquaneuf for originating idea.
@gfyoung gfyoung force-pushed the file-like-obj-restore branch from df83963 to 1bcaaba Compare March 10, 2020 21:04
@gfyoung
Copy link
Member Author

gfyoung commented Mar 10, 2020

Alrighty, everything is finally green! If people can take a look at and give feedback on this, that would be great so we can get this merged before tomorrow.

Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

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

lgtm.

@gfyoung gfyoung merged commit b9762eb into pandas-dev:master Mar 11, 2020
@gfyoung gfyoung deleted the file-like-obj-restore branch March 11, 2020 02:22
meeseeksmachine pushed a commit to meeseeksmachine/pandas that referenced this pull request Mar 11, 2020
jreback pushed a commit that referenced this pull request Mar 11, 2020
SeeminSyed pushed a commit to CSCD01-team01/pandas that referenced this pull request Mar 22, 2020
Restores behavior down to the fact that the
Python engine cannot handle NamedTemporaryFile.

Closes pandas-dev#31819

Credit to @sasanquaneuf for originating idea.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IO CSV read_csv, to_csv Regression Functionality that used to work in a prior pandas version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

pandas 1.0.1 read_csv() is broken for some file-like objects
3 participants