Skip to content

testing case sort_index with pd.set_option('use_inf_as_na', True) #38542

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
Closed

testing case sort_index with pd.set_option('use_inf_as_na', True) #38542

wants to merge 1 commit into from

Conversation

weikhor
Copy link
Contributor

@weikhor weikhor commented Dec 17, 2020

Create test cases for sort_index works with 'pd.set_option('mode.use_inf_as_na',True)

@pep8speaks
Copy link

Hello @weikhor! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 860:1: W293 blank line contains whitespace
Line 866:1: W293 blank line contains whitespace
Line 879:1: W391 blank line at end of file

Copy link
Member

@arw2019 arw2019 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The snippet in the original ticket works on master. To close the ticket you'll need to figure out which commit fixed the issue (likely by running git bisect) and only add new tests if there aren't any already

now = pd.Timestamp.now()
df.index = [now, now - pd.Timedelta('1m'), now + pd.Timedelta('2m')]

try:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please use pytest.raises here

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(this specific bit doesn't raise so this is a general pointer)

@jreback jreback added the Testing pandas testing functions or related to the test suite label Jan 1, 2021
@jreback
Copy link
Contributor

jreback commented Jan 1, 2021

@weikhor can you merge master and update to comments

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2021

This pull request is stale because it has been open for thirty days with no activity. Please update or respond to this comment if you're still interested in working on this.

@github-actions github-actions bot added the Stale label Feb 1, 2021
@jreback
Copy link
Contributor

jreback commented Feb 11, 2021

closing as stale. if you want to continue, pls ping and can re-open.

@jreback jreback closed this Feb 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

'sort_index"does not work with 'pd.set_option('mode.use_inf_as_na',True)'
4 participants