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

[NOMERGE] [c++/python] Move DataFrame write path to use C++ bindings #2367

Closed
wants to merge 70 commits into from

Conversation

nguyenv
Copy link
Member

@nguyenv nguyenv commented Apr 3, 2024

This will be broken up into multiple smaller PRs.

Copy link

codecov bot commented Apr 3, 2024

Codecov Report

Merging #2367 (031f9a9) into main (05e9592) will increase coverage by 11.28%.
Report is 77 commits behind head on main.
The diff coverage is 92.41%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #2367       +/-   ##
===========================================
+ Coverage   78.63%   89.91%   +11.28%     
===========================================
  Files         136       34      -102     
  Lines       10653     3788     -6865     
  Branches      215        0      -215     
===========================================
- Hits         8377     3406     -4971     
+ Misses       2177      382     -1795     
+ Partials       99        0       -99     
Flag Coverage Δ
libtiledbsoma ?
python 89.91% <92.41%> (-0.66%) ⬇️
r ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
python_api 89.91% <92.41%> (-0.66%) ⬇️
libtiledbsoma ∅ <ø> (∅)

@johnkerl johnkerl changed the title [NOMERGE] Move DataFrame write path to use C++ bindings [NOMERGE] [c++/python] Move DataFrame write path to use C++ bindings Apr 3, 2024
@nguyenv
Copy link
Member Author

nguyenv commented Apr 23, 2024

This PR has now been fullly merged in as #2418, #2425, and #2431 and can be closed.

@nguyenv nguyenv closed this Apr 23, 2024
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.

1 participant