-
-
Notifications
You must be signed in to change notification settings - Fork 18.1k
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
Push some of Series.ix into Series.__getitem__/__setitem__ #86
Comments
Why not push all of it into getitem/setitem? def getitem(self, key): def setitem(self, key, value): |
Oof. Please steer clear of this issue, it's full of performance landmines. I need to set up some benchmarks so I can evaluate all of the use cases |
In principle having all of it in getitem/setitem would be nice. but the use cases are complex |
sounds like a reasonable approach. i'll let you take this one then :) |
Alright, this is done. Need to document the related API changes and add lots of WARNING, WARNING |
Should be able to do stuff like:
to mirror ts.ix[date1:date2]. Should respect integer labels while Series.ix will not
The text was updated successfully, but these errors were encountered: