Skip to content

Conversation

GYHHAHA
Copy link
Contributor

@GYHHAHA GYHHAHA commented Apr 6, 2022

@GYHHAHA GYHHAHA changed the title Patch 2 BUG: df.nsmallest get wrong results when NaN in the sorting column Apr 6, 2022
@GYHHAHA
Copy link
Contributor Author

GYHHAHA commented Apr 8, 2022

Should this pr be recorded in 1.4.3 whatsnew? I see your 1.4.3 tag (instead of 1.5) on this issue. @simonjayhawkins

@phofl
Copy link
Member

phofl commented Apr 8, 2022

Yes, this should move to 1.4.3

@simonjayhawkins simonjayhawkins added this to the 1.4.3 milestone Apr 8, 2022
@simonjayhawkins simonjayhawkins added the Numeric Operations Arithmetic, Comparison, and Logical operations label Apr 8, 2022
@jreback jreback merged commit eab2cfa into pandas-dev:main Apr 10, 2022
@jreback
Copy link
Contributor

jreback commented Apr 10, 2022

thanks @GYHHAHA

@jreback
Copy link
Contributor

jreback commented Apr 10, 2022

@meeseeksdev backport 1.4.x

@lumberbot-app
Copy link

lumberbot-app bot commented Apr 10, 2022

Could not push to auto-backport-of-pr-46656-on-1.4.x due to error, aborting.

meeseeksmachine pushed a commit to meeseeksmachine/pandas that referenced this pull request Apr 10, 2022
@lumberbot-app
Copy link

lumberbot-app bot commented Apr 10, 2022

Could not push to auto-backport-of-pr-46656-on-1.4.x due to error, aborting.

@simonjayhawkins
Copy link
Member

@meeseeksdev backport 1.4.x

@lumberbot-app
Copy link

lumberbot-app bot commented Apr 12, 2022

Could not push to auto-backport-of-pr-46656-on-1.4.x due to error, aborting.

simonjayhawkins added a commit that referenced this pull request Apr 12, 2022
…he sorting column (#46748)

Co-authored-by: Yuanhao Geng <41546976+GYHHAHA@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Numeric Operations Arithmetic, Comparison, and Logical operations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: df.nsmallest get wrong results when row contains NaN
4 participants