Skip to content

Conversation

@ampx-mg
Copy link
Collaborator

@ampx-mg ampx-mg commented Feb 6, 2023

Similarly to #5 this PR adds a write_object support for format="json" from pd.DataFrame to json file. Adds tests as well.

@ampx-mg ampx-mg force-pushed the mg/add_json_write_support branch from d036b83 to a413a36 Compare February 6, 2023 13:58
@ampx-mg ampx-mg requested a review from vtuma February 6, 2023 15:31
@tmi tmi requested review from tmi and removed request for vtuma February 6, 2023 15:52
Copy link
Collaborator

@tmi tmi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm; the conflict with main is something trivial after the merge of that py.typed branch... you can just merge this one after fixing

@ampx-mg ampx-mg force-pushed the mg/add_json_write_support branch from ecbfd13 to bf3ff13 Compare February 7, 2023 07:20
@ampx-mg ampx-mg merged commit 14b9b94 into main Feb 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants