-
-
Notifications
You must be signed in to change notification settings - Fork 19.4k
ENH: Add index param to df.to_dict #48429
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
datapythonista
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.
Thanks @yuanx749, nice PR. General idea looks good, I just added couple of comments.
datapythonista
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.
Thanks for the updates @yuanx749, really nice, lgtm
mroeschke
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.
Looks pretty good. Could you resolve the merge conflict?
Resolved. |
|
Thanks @yuanx749 |
* EHN: Add index param to df.to_dict * Add whatsnew * Move whatsnew * Refactor duplicate code
indexparam toto_dictlike into_json. #46398doc/source/whatsnew/vX.X.X.rstfile if fixing a bug or adding a new feature.