-
-
Notifications
You must be signed in to change notification settings - Fork 18.7k
BUG: encoding error in to_csv compression #21300
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
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
fdd3ce9
handle encoding type
minggli 826aa2c
enrich comment
minggli fac53c0
add encoding fixture
minggli a4de620
remove redundant import
minggli b833abd
add encoding fixture
minggli b625f08
handle PY2 differently
minggli 9d5c25b
restore original test case
minggli 8ed6fa2
update whatsnew
minggli 2d48d10
default bytes strings unless requires unicode string
minggli 4a6f5ff
assert filehandle open
minggli bf4225c
use buffer and avoid roundtrip and encoding error.
minggli 486f3ff
update whatsnew
minggli d8435ef
minor refactor of tests
minggli 16cc951
clearer categorisation of input type
minggli 6714e68
more compact if statements
minggli f73f9ff
documentation
minggli f891533
add comments on path_or_buf types
minggli f3c3ea7
remove close
minggli 4e914ff
test file-handle to_csv with compression and encoding
minggli File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
update whatsnew
- Loading branch information
commit 8ed6fa2e3c86dc58023db6f0de72ae9a03437329
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could also mention Series.to_csv here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added.