Skip to content

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