-
-
Notifications
You must be signed in to change notification settings - Fork 154
Type DataFrame
#1481
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
Type DataFrame
#1481
Conversation
2330345 to
40fe0fb
Compare
cmp0xff
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Please fix the default values of
axis - Maybe add
# TODO: resolve with pandas-dev/pandas#63111tofrom_dataframe
cmp0xff
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there.
pandas-stubs/core/frame.pyi
Outdated
| def truediv( | ||
| self, | ||
| other: complex | ListLike | DataFrame, | ||
| axis: Axis | None = ..., |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| axis: Axis | None = ..., | |
| axis: Axis | None = "columns", |
|
Please set the PR to ready (and request a review) when it is ready. Thanks. |
|
sure, thanks, done sorry for the all the commits and good call on #1481 (comment), it revealed a mismatch between |
cmp0xff
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One comment left 🙂
cmp0xff
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @MarcoGorelli , we now have most DataFrame-related functionalities typed!
|
thanks for bearing with me on this one 😅 ! taking a look at index next |
assert_type()to assert the type of any return valuecloses #1446