Skip to content

Import OptionError in pandas.errors #32142

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

Conversation

sumanau7
Copy link
Contributor

@sumanau7 sumanau7 commented Feb 21, 2020

@sumanau7 sumanau7 force-pushed the import_OptionError_in_pandas_errors branch from 8f2d630 to 2188d7b Compare February 21, 2020 07:17
@simonjayhawkins simonjayhawkins added API Design Error Reporting Incorrect or improved errors from pandas labels Feb 21, 2020
@sumanau7
Copy link
Contributor Author

@jorisvandenbossche I am experiencing a build failure on my forked master as well, is this some setup issue, would need your help to resolve this.

Also Is the pr good to go or more changes would be required ?

@jbrockmendel
Copy link
Member

Travis failure should be unrelated, just restarted that build.

@jbrockmendel
Copy link
Member

Can you add this to test_exception_importable in tests.test_errors

@sumanau7 sumanau7 force-pushed the import_OptionError_in_pandas_errors branch from 8f5ed3a to 30c293b Compare February 22, 2020 15:09
@sumanau7
Copy link
Contributor Author

@jbrockmendel Tests are changed, a whatsnew entry is created. Would you be able to review this or suggest someone who can help in reviewing this PR.

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.

ping on green.

@jreback jreback added this to the 1.1 milestone Feb 23, 2020
@sumanau7 sumanau7 force-pushed the import_OptionError_in_pandas_errors branch 2 times, most recently from c38da31 to f73b8eb Compare February 23, 2020 16:15
@sumanau7 sumanau7 force-pushed the import_OptionError_in_pandas_errors branch from f73b8eb to 3c6c813 Compare February 23, 2020 16:29
@sumanau7
Copy link
Contributor Author

@jreback Please review.

@jreback jreback merged commit 7d32184 into pandas-dev:master Feb 24, 2020
@jreback
Copy link
Contributor

jreback commented Feb 24, 2020

thanks @sumanau7

roberthdevries pushed a commit to roberthdevries/pandas that referenced this pull request Mar 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Design Error Reporting Incorrect or improved errors from pandas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OptionError raised by top-level function pandas.set_option is private
5 participants