Skip to content
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: add pyarrow list and struct to pandas engine #1699

Merged
merged 3 commits into from
Jun 27, 2024

Conversation

aaravind100
Copy link
Contributor

Adds pyarrow list and struct types to pandas engine. Per #1676

Copy link

codecov bot commented Jun 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.60%. Comparing base (812b2a8) to head (54aaa90).
Report is 150 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1699       +/-   ##
===========================================
- Coverage   94.28%   82.60%   -11.68%     
===========================================
  Files          91      117       +26     
  Lines        7013     8715     +1702     
===========================================
+ Hits         6612     7199      +587     
- Misses        401     1516     +1115     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cosmicBboy
Copy link
Collaborator

hey @aaravind100 mind rebasing this onto main? the failing docs build should go away after that

@aaravind100
Copy link
Contributor Author

hey @aaravind100 mind rebasing this onto main? the failing docs build should go away after that

Sure, let me push the update.

@aaravind100 aaravind100 force-pushed the ajith/pyarrow-list-struct branch from 0b0c6e5 to ea10f2b Compare June 23, 2024 14:43
@cosmicBboy
Copy link
Collaborator

can we add some unit tests for these two new types?

@aaravind100 aaravind100 force-pushed the ajith/pyarrow-list-struct branch from 0ecace4 to e03af72 Compare June 26, 2024 18:34
@aaravind100 aaravind100 force-pushed the ajith/pyarrow-list-struct branch from e03af72 to 23462f1 Compare June 26, 2024 18:43
Signed-off-by: Ajith Aravind <[email protected]>
@aaravind100
Copy link
Contributor Author

@cosmicBboy I've added two tests

Copy link
Collaborator

@cosmicBboy cosmicBboy left a comment

Choose a reason for hiding this comment

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

thanks @aaravind100 🚀

@cosmicBboy cosmicBboy merged commit 44a9763 into unionai-oss:main Jun 27, 2024
155 of 156 checks passed
@aaravind100 aaravind100 deleted the ajith/pyarrow-list-struct branch June 28, 2024 07:08
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.

2 participants