-
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
WIP - DataFrames refactoring #612
base: main
Are you sure you want to change the base?
Conversation
Benchmark Results
Benchmark PlotsA plot of the benchmark results have been uploaded as an artifact to the workflow run for this PR. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #612 +/- ##
===========================================
- Coverage 100.00% 99.05% -0.95%
===========================================
Files 15 15
Lines 639 633 -6
===========================================
- Hits 639 627 -12
- Misses 0 6 +6 ☔ View full report in Codecov by Sentry. |
e43d70a
to
863f120
Compare
The function to create graph et al from a csv folder has been split into two functions. The first reads the csv folder into a new TableTree structure. The second processes the TableTree structure into graph et al.
Creates compute_constraints_partitions functions to use TableTree to build the partitions of the constraints. This is done in two parts. First, we create unrolled partitions, which will substitute the individual partitions vectors stores in the graphs. Second, we build constraints partitions still using compute_rp_partitions, but using only DataFrames. The result is also stored in a DataFrame, which should also serve as the final DataFrame to be used instead of energy_problem.dataframes.
863f120
to
1ac6ef4
Compare
Describe the changes made in this pull request
List of related issues or pull requests
Closes #ISSUE_NUMBER
Collaboration confirmation
As a contributor I confirm