-
Notifications
You must be signed in to change notification settings - Fork 71
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
Logup generator returns multiple prefix sums #829
base: 09-08-Logup_-_rename_claimed_sum_to_total_sum_Take_total_sum_from_cum_sum_column
Are you sure you want to change the base?
Conversation
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @shaharsamocha7 and the rest of your teammates on Graphite |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 09-08-Logup_-_rename_claimed_sum_to_total_sum_Take_total_sum_from_cum_sum_column #829 +/- ##
=================================================================================================================
Coverage 91.81% 91.81%
=================================================================================================================
Files 89 89
Lines 12082 12088 +6
Branches 12082 12088 +6
=================================================================================================================
+ Hits 11093 11099 +6
Misses 882 882
Partials 107 107 ☔ View full report in Codecov by Sentry. |
d50d61f
to
b3f754b
Compare
483abed
to
4095bd6
Compare
b3f754b
to
4c3990c
Compare
4095bd6
to
f465a76
Compare
4c3990c
to
45206e8
Compare
Logup generator finalize get a list of indices and returns the corresponding prefix sum
This change is