Remove claimed_sum and LogupSums

[shaharsamocha7]

No description provided.

[reviewable-StarkWare]

This change is 

[shaharsamocha7]

• Rename total_sum to claimed_sum #982
• Remove claimed_sum and LogupSums #979 👈 (View in Graphite)
• Logup cumsum constraint with cumsum_shift #978
• Remove padding examples #974
• dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.27%. Comparing base (9316f98) to head (021ccf8).

Additional details and impacted files

@@            Coverage Diff             @@
##              dev     #979      +/-   ##
==========================================
+ Coverage   92.02%   92.27%   +0.25%     
==========================================
  Files         105      105              
  Lines       14274    14221      -53     
  Branches    14274    14221      -53     
==========================================
- Hits        13135    13123      -12     
+ Misses       1064     1024      -40     
+ Partials       75       74       -1     

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

[ohad-starkware]

Reviewed 21 of 21 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @shaharsamocha7)

[ohad-starkware]

Reviewed 1 of 1 files at r2, 11 of 11 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @shaharsamocha7)

---

crates/prover/src/constraint_framework/info.rs line 30 at r3 (raw file):

}
impl InfoEvaluator {
    pub fn new(log_size: u32, preprocessed_columns: Vec<String>, total_sum: SecureField) -> Self {

isnt this a Gali change?

[shaharsamocha7]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @shaharsamocha7)

---

crates/prover/src/constraint_framework/info.rs line 30 at r3 (raw file):

Previously, ohad-starkware (Ohad) wrote…

isnt this a Gali change?

I rebased over her change.
If you check r0->r3 the change is only on total_sum

[shaharsamocha7]

Merge activity

• Jan 15, 3:47 AM EST: A user started a stack merge that includes this pull request via Graphite.
• Jan 15, 3:55 AM EST: Graphite rebased this pull request as part of a merge.
• Jan 15, 3:58 AM EST: A user merged this pull request with Graphite.