Skip to content

CLN: Resubmit of GH14700. Fixes GH14554. Errors other than Indexing… #14912

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 1 commit into from

Conversation

clham
Copy link
Contributor

@clham clham commented Dec 18, 2016

…Error and KeyError now bubble up appropriately.

…ror and KeyError now bubble up appropriately.
@clham clham mentioned this pull request Dec 18, 2016
4 tasks
@codecov-io
Copy link

Current coverage is 84.57% (diff: 0.00%)

Merging #14912 into master will not change coverage

@@             master     #14912   diff @@
==========================================
  Files           144        144          
  Lines         51057      51057          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits          43180      43180          
  Misses         7877       7877          
  Partials          0          0          

Powered by Codecov. Last update f1cfe5b...458c0cc

@sinhrks sinhrks added Bug Error Reporting Incorrect or improved errors from pandas Indexing Related to indexing on series/frames, not to indexes themselves labels Dec 19, 2016
@jreback jreback added this to the 0.19.2 milestone Dec 19, 2016
@jreback
Copy link
Contributor

jreback commented Dec 19, 2016

so this is segfaulting (with the OP issue) on py3. any thoughts on this

@jreback jreback modified the milestones: 0.20.0, 0.19.2 Dec 19, 2016
@jreback jreback closed this in 3ccb501 Dec 19, 2016
@jreback
Copy link
Contributor

jreback commented Dec 19, 2016

thanks!

note as per my comment above, this (and master and 0.19.1) segfault in python 3. not sure what if anything can do about that (they segfault because of the recursion). any ideas?

jorisvandenbossche pushed a commit to jorisvandenbossche/pandas that referenced this pull request Dec 24, 2016
IdnexError and KeyError now bubble up appropriately.

closes pandas-dev#14554

Author: Chris Ham <[email protected]>

Closes pandas-dev#14912 from clham/gh14554-b and squashes the following commits:

458c0cc [Chris Ham] CLN: Resubmit of GH14700.  Fixes GH14554.  Errors other than IndexingError and KeyError now bubble up appropriately.

(cherry picked from commit 3ccb501)
ShaharBental pushed a commit to ShaharBental/pandas that referenced this pull request Dec 26, 2016
IdnexError and KeyError now bubble up appropriately.

closes pandas-dev#14554

Author: Chris Ham <[email protected]>

Closes pandas-dev#14912 from clham/gh14554-b and squashes the following commits:

458c0cc [Chris Ham] CLN: Resubmit of GH14700.  Fixes GH14554.  Errors other than IndexingError and KeyError now bubble up appropriately.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Error Reporting Incorrect or improved errors from pandas Indexing Related to indexing on series/frames, not to indexes themselves
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CLN/BUG: remove bare excepts
4 participants