-
-
Notifications
You must be signed in to change notification settings - Fork 156
GH1541 Revert Series[Any].__add__(str) #1542
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
GH1541 Revert Series[Any].__add__(str) #1542
Conversation
Dr-Irv
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.
I'm fine with this. Would like @cmp0xff to take a look as well.
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.
I think we still miss a few tests.
I also plan to simplify the tests structure for arithmetic operations, removing the arithmetic subfolder and move everything one level above. Hopefully this will confuse people less in the future.
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.
Just two more comments!
|
|
||
| # left operands | ||
| left_i = pd.DataFrame({"a": [1, 2, 3]})["a"] | ||
| left_str = pd.DataFrame({"a": ["1", "2", "3_"]})["a"] |
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.
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 @loicdiridollou !
Series[Any]should accept__add__with strings #1541assert_type()to assert the type of any return value)AGENTS.md.