Weird behavior using nlargest/nsmallest when there are the n smallest/largest values are identical #15297

Closed
RogerThomas opened this Issue Feb 3, 2017 · 4 comments

Comments

Projects
None yet
2 participants

RogerThomas commented Feb 3, 2017 edited by jreback

Code Sample, a copy-pastable example if possible

python -c "import pandas as pd; df = pd.DataFrame(dict(a=[1, 1, 2, 3], b=[1, 2, 3, 4])); print(df.nsmallest(2, 'a'))"

Problem description

When using nlargest/nsmallest and the n largest / smallest values are identical, the method seems to return the dataframe concatenated with the filtered version of itself.
Furthermore if all values are identical, you get the full dataframe concatenated with itself, regardless of the choice of n

Expected Output

Not really sure, I guess in the example above you should simply get a dataframe that looks like this
pd.DataFrame(dict(a=[1, 1], b=[1, 2]))
however if you were to have
df = pd.DataFrame(dict(a=[1, 1, 1, 1], b=[1, 2, 3, 4]))
and asked for
df.nlargest(2, 'a') you should again get
pd.DataFrame(dict(a=[1, 1], b=[1, 2]))

Output of pd.show_versions()

INSTALLED VERSIONS ------------------ commit: None python: 2.7.12.final.0 python-bits: 64 OS: Linux OS-release: 4.8.0-34-generic machine: x86_64 processor: x86_64 byteorder: little LC_ALL: None LANG: en_IE.UTF-8 LOCALE: None.None

pandas: 0.19.2
nose: 1.3.7
pip: 9.0.1
setuptools: 28.3.0
Cython: 0.23.4
numpy: 1.12.0
scipy: 0.16.1
statsmodels: 0.6.1
xarray: None
IPython: None
sphinx: 1.3.1
patsy: 0.4.1
dateutil: 2.6.0
pytz: 2016.10
blosc: None
bottleneck: None
tables: 3.2.0
numexpr: 2.4.6
matplotlib: 1.5.0
openpyxl: None
xlrd: None
xlwt: None
xlsxwriter: None
lxml: None
bs4: None
html5lib: None
httplib2: 0.9.2
apiclient: None
sqlalchemy: 1.0.9
pymysql: None
psycopg2: None
jinja2: 2.8
boto: 2.38.0
pandas_datareader: None

Contributor

jreback commented Feb 3, 2017

so this was fixed for duplicates in the index 6e514da (for 0.19.2).

yeah this does look a bit odd.So looks the the 'dups' are getting duplicated. Want to have a look and see if you can find where?

In [5]: df = pd.DataFrame(dict(a=[1, 1, 2, 3], b=[1, 2, 3, 4]))

In [7]: df.nsmallest(3, 'a')
Out[7]: 
   a  b
0  1  1
1  1  2
0  1  1
1  1  2
2  2  3

jreback added this to the 0.20.0 milestone Feb 3, 2017

Sure, I'll take a look!

Contributor

jreback commented Feb 3, 2017

great!

Contributor

jreback commented Feb 21, 2017

xref this: #14846 (comment)

@jreback jreback modified the milestone: 0.20.0, Next Major Release Mar 23, 2017

@jreback jreback modified the milestone: 0.20.0, Next Major Release Mar 31, 2017

jreback closed this in c112252 Apr 6, 2017

@linebp linebp added a commit to linebp/pandas that referenced this issue Apr 17, 2017

@linebp Roger Thomas + linebp BUG: Fix nsmallest/nlargest With Identical Values
closes #15297

Author: Roger Thomas <roger.thomas@cremeglobal.com>

Closes #15299 from RogerThomas/fix_nsmallest_nlargest_with_n_identical_values and squashes the following commits:

d3964f8 [Roger Thomas] Fix nsmallest/nlargest With Identical Values
7e30d89
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment