Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
BUG/API: PeriodIndex.unique returns ndarray #7540
Comments
|
this is a historical / compat as well. In theory could fix all of them, but I think lots of things (tests) will break. I am not sure if anything real will actually break. Give a try and see (but I agree |
jreback
added API Design Period Compat
labels
Jun 21, 2014
sinhrks
referenced
this issue
Jun 30, 2014
Open
INT: mixin for PeriodIndex / Datetimeindex for common methods #6469
jreback
added this to the
0.15.0
milestone
Jul 7, 2014
sinhrks
referenced
this issue
Jul 25, 2014
Merged
BUG: PeriodIndex.unique results in Int64Index #7843
jreback
closed this
in #7843
Jul 26, 2014
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
sinhrks commentedJun 21, 2014
Currently,
Index.uniquereturnsndarraybutDatetimeIndex.uniquereturnsDatetimeIndexto preserve datetime information. I thinkPeriodIndex.uniqueshould also returnPeriodIndex(currently it returnsndarray).I'd like to ask which is better to fix all
Index.uniqueto returnIndexfor consistency, or only fixPeriodIndex.