Skip to content

TST: A test to cover fault in issue #5291 #13833

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
wants to merge 4 commits into from

Conversation

ivannz
Copy link
Contributor

@ivannz ivannz commented Jul 29, 2016

The mentioned issue faulted under similar conditions as issue #13703, and was also fixed by PR #13788. This PR just adds a test to replicate the fault, as discussed in the issue's thread.

This PR also corrects accidental use of float("nan") instead of np.nan in test_parse_trim_buffers.

@ivannz ivannz changed the title A test to cover fault in issue #5291 TST: A test to cover fault in issue #5291 Jul 29, 2016
@ivannz
Copy link
Contributor Author

ivannz commented Jul 29, 2016

I think this PR and PR #13788 also close the issue #11793.

@codecov-io
Copy link

codecov-io commented Jul 29, 2016

Current coverage is 85.25% (diff: 100%)

Merging #13833 into master will increase coverage by 0.01%

@@             master     #13833   diff @@
==========================================
  Files           140        140          
  Lines         50451      50455     +4   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits          43003      43015    +12   
+ Misses         7448       7440     -8   
  Partials          0          0          

Powered by Codecov. Last update b2a1445...4a74d50

@sinhrks sinhrks added Testing pandas testing functions or related to the test suite IO CSV read_csv, to_csv labels Jul 30, 2016
@sinhrks sinhrks added this to the 0.19.0 milestone Jul 30, 2016
@ivannz ivannz closed this Aug 1, 2016
@ivannz ivannz reopened this Aug 2, 2016
@ivannz ivannz closed this Aug 2, 2016
gfyoung added a commit to forking-repos/pandas that referenced this pull request Nov 6, 2017
@gfyoung gfyoung modified the milestones: 0.19.0, No action Nov 6, 2017
gfyoung added a commit that referenced this pull request Nov 6, 2017
watercrossing pushed a commit to watercrossing/pandas that referenced this pull request Nov 10, 2017
No-Stream pushed a commit to No-Stream/pandas that referenced this pull request Nov 28, 2017
TomAugspurger pushed a commit to TomAugspurger/pandas that referenced this pull request Dec 8, 2017
TomAugspurger pushed a commit that referenced this pull request Dec 11, 2017
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 Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Segmentation fault or UnicodeDecodeError when reading csv-file depending on chunksize.
4 participants