Skip to content
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.

Sign up for GitHub

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

Jump to bottom

BUG: IntervalIndex constructor doesn't infer closed #18334

Closed
jschendel opened this issue Nov 17, 2017 · 0 comments · Fixed by #18340
Closed

BUG: IntervalIndex constructor doesn't infer closed #18334

jschendel opened this issue Nov 17, 2017 · 0 comments · Fixed by #18340
Labels
Bug Interval Interval data type
Milestone
0.21.1

Comments

@jschendel
Copy link
Member

jschendel commented Nov 17, 2017 •
edited
Loading

Code Sample, a copy-pastable example if possible

From a list of intervals:

In [2]: ivs = [pd.Interval(3, 14, closed='both'), pd.Interval(15, 92, closed='both')]

In [3]: pd.IntervalIndex(ivs)
Out[3]:
IntervalIndex([(3, 14], (15, 92]]
              closed='right',
              dtype='interval[int64]')

From an existing IntervalIndex:

In [4]: idx = pd.IntervalIndex.from_tuples([(3, 14), (15, 92)], closed='both')
    ...: idx
    ...:
Out[4]:
IntervalIndex([[3, 14], [15, 92]]
              closed='both',
              dtype='interval[int64]')

In [5]: pd.IntervalIndex(idx)
Out[5]:
IntervalIndex([(3, 14], (15, 92]]
              closed='right',
              dtype='interval[int64]')

Problem description

The intervals start with closed='both' but the resulting IntervalIndex has closed='right'.

Expected Output

I'd expect the resulting IntervalIndex to have closed='both':

IntervalIndex([[3, 14], [15, 92]]
              closed='both',
              dtype='interval[int64]')

Output of pd.show_versions()

INSTALLED VERSIONS

commit: None
python: 3.6.3.final.0
python-bits: 64
OS: Windows
OS-release: 10
machine: AMD64
processor: Intel64 Family 6 Model 78 Stepping 3, GenuineIntel
byteorder: little
LC_ALL: None
LANG: None
LOCALE: None.None

pandas: 0.21.0
pytest: 3.1.2
pip: 9.0.1
setuptools: 27.2.0
Cython: 0.26
numpy: 1.13.3
scipy: 0.19.1
pyarrow: None
xarray: None
IPython: 6.1.0
sphinx: 1.5.6
patsy: 0.4.1
dateutil: 2.6.1
pytz: 2017.2
blosc: None
bottleneck: 1.2.1
tables: 3.2.2
numexpr: 2.6.4
feather: None
matplotlib: 2.0.2
openpyxl: 2.4.7
xlrd: 1.0.0
xlwt: 1.2.0
xlsxwriter: 0.9.6
lxml: 3.7.3
bs4: 4.6.0
html5lib: 0.999
sqlalchemy: 1.1.9
pymysql: None
psycopg2: None
jinja2: 2.9.6
s3fs: None
fastparquet: None
pandas_gbq: None
pandas_datareader: None
@jschendel jschendel changed the title BUG: IntervalIndex constructor doesn't infer closed with list of Intervals BUG: IntervalIndex constructor doesn't infer closed Nov 17, 2017
@jschendel jschendel mentioned this issue Nov 17, 2017
Merged
5 tasks
@jreback jreback added Bug Interval Interval data type labels Nov 17, 2017
@jreback jreback added this to the 0.21.1 milestone Nov 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Interval Interval data type
Projects
None yet
Milestone
0.21.1
Development

Successfully merging a pull request may close this issue.

BUG: Fix IntervalIndex constructor and copy with non-default closed jschendel/pandas
2 participants
@jreback @jschendel