-
-
Notifications
You must be signed in to change notification settings - Fork 18k
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
DEPR: make arguments keyword only in to_dict #54630
Conversation
pandas/core/frame.py
Outdated
@@ -2005,25 +2009,25 @@ def to_dict( | |||
|
|||
You can specify the return orientation. | |||
|
|||
>>> df.to_dict('series') | |||
>>> df.to_dict(orient='series') |
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.
We could also keep allowing just orient
as positional only keyword, given how we have been using that ourselves in the docs?
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.
Sure, should I make that change then?
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.
@jorisvandenbossche should I remove the keyword orient
from the examples and tests as well?
@mroeschke should I remove the |
Yes |
@jorisvandenbossche @mroeschke I am done with the changes. |
Thanks @rsm-23 |
doc/source/whatsnew/v2.2.0.rst
file if fixing a bug or adding a new feature.