pd.RangeIndex should accept range as the first argument #686
Conversation
let me push the code with the required test. |
|
@kshitiz305 checking on status of this PR |
|
@Dr-Irv Kindly have a look test for the range have been pushed |
|
So there is a failure because pyright 1.1.309 started finding a whole bunch of issues. Trying to figure out what version to pin pyright to so things will work |
|
Once #709 is merged, you will need to merge that in and revert changes to the 2 files I indicated. |
|
@kshitiz305 #709 has been merged, so can you merge with |
pyproject.toml
Outdated
| @@ -1,6 +1,6 @@ | |||
| [tool.poetry] | |||
| name = "pandas-stubs" | |||
| version = "2.0.1.230501" | |||
| version = "2.0.1.230507" | |||
There was a problem hiding this comment.
Please revert this change.
There was a problem hiding this comment.
I think you can simply run git checkout <commit hash from before you made any changes> pandas-stubs/_version.pyi pyproject.toml to revert these two files to what they were before.
pandas-stubs/core/indexes/range.pyi
Outdated
| def __init__( | ||
| self, | ||
| start: int | RangeIndex = ..., | ||
| cls, |
There was a problem hiding this comment.
This should be self, because it is in __init__()
I think this is the cause of the pyright error you are getting.
|
@kshitiz305 can you merge with main and get this finished up? |
pd.RangeIndexshould acceptrangeas the first argument #677 (Replace xxxx with the Github issue number)assert_type()to assert the type of any return value