-
Notifications
You must be signed in to change notification settings - Fork 18
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
Read CSV using TulipaIO and DuckDB and create table_tree from DB #634
Conversation
Benchmark Results
Benchmark PlotsA plot of the benchmark results have been uploaded as an artifact to the workflow run for this PR. |
This depends on the TulipaIO release. Check back after 11/May in the afternoon. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #634 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 15 15
Lines 656 691 +35
=========================================
+ Hits 656 691 +35 ☔ View full report in Codecov by Sentry. |
b095058
to
da167e4
Compare
da167e4
to
7397603
Compare
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.
LGTM
Co-authored-by: Suvayu Ali <suvayu@users.noreply.github.com>
Co-authored-by: Suvayu Ali <suvayu@users.noreply.github.com>
Pull request details
Describe the changes made in this pull request
table_tree
.table_tree
to use the functions above.List of related issues or pull requests
Closes #629
Collaboration confirmation
As a contributor I confirm