-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
ENH: allow start=range_object in RangeIndex constructor #40241
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
Conversation
hah I originally implemented this but then we settled on |
I'm +1 on this change. If we call something RangeIndex, it's reasonable that it takes a range in its constructor IMO. |
(no strong opinion here) |
lgtm. I think ok to also deprecated |
In the RangeIndex class doc stringm in the |
good idea. will add this to my upcoming core.indexes TYP branch if thats OK |
thanks @jbrockmendel followups when you can |
simplifies
Index.__new__
a bit