-
Notifications
You must be signed in to change notification settings - Fork 88
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
Feat/pandas reader parquet #429
Feat/pandas reader parquet #429
Conversation
…ng/reading of a dataframe
…showing writing/reading of a dataframe
@skrawcz , last point in the checklist. Can you point me to the "Project documentation" mentioned? |
Yep so that should be automatically updated for you. It should show up here (based on the build of this branch) https://hamilton--429.org.readthedocs.build/en/429/reference/io/available-data-adapters/. |
…riter to handle kwargs not listed in Pandas' 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.
looks good, just the minor comment on the test!
You will need to gate this parameter for 3.7. So you'll see others have: if sys.version_info >= (3, 8) and self.dtype_backend is not None:
kwargs["dtype_backend"] = self.dtype_backend |
This pull request adds parquet read and write functionality for issue #406.
Changes
The changes to the files are as described below:
pandas_extensions.py
: Added the classes to read and write parquet filestest_pandas_extensions.py
: Added a single test case that exercises the writing and reading functionality respectivelynotebook.ipynb
: Added the example of parquet materializationmy_script.py
: Added the example of parquet materializationHow I tested this
By running successfully:
Notes
Checklist