Remove InteractionElements LookupValues and trace_generation

[shaharsamocha7]

This change is 

[shaharsamocha7]

• Remove InteractionElements LookupValues and trace_generation #812 👈
• Refactor wide fibonacci Remove lookup - each row contain independent instance #811
• SIMD backend for poseidon252 merkle ops - CPU IMPL #809
• dev

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

Join @shaharsamocha7 and the rest of your teammates on Graphite

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.65%. Comparing base (f14134e) to head (51446b2).

Additional details and impacted files

@@            Coverage Diff             @@
##              dev     #812      +/-   ##
==========================================
+ Coverage   91.48%   92.65%   +1.16%     
==========================================
  Files          92       89       -3     
  Lines       12489    12048     -441     
  Branches    12489    12048     -441     
==========================================
- Hits        11426    11163     -263     
+ Misses        952      778     -174     
+ Partials      111      107       -4     

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

[spapinistarkware]

Reviewed 12 of 13 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @alonh5)

[spapinistarkware]

Reviewed 3 of 3 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @alonh5)

[spapinistarkware]

Reviewed 2 of 2 files at r4, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @alonh5)

[alonh5]

Reviewed 10 of 13 files at r1, 1 of 3 files at r3, 2 of 2 files at r4, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @shaharsamocha7)

[alonh5]

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

[shaharsamocha7]

Merge activity

• Aug 29, 6:21 AM EDT: @shaharsamocha7 started a stack merge that includes this pull request via Graphite.
• Aug 29, 6:23 AM EDT: Graphite couldn't merge this pull request because a downstack PR Refactor wide fibonacci Remove lookup - each row contain independent instance #811 failed to merge.

[alonh5]

Reviewed 10 of 10 files at r6, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @shaharsamocha7)

---

crates/prover/src/trace_generation/prove.rs line 1 at r6 (raw file):

use itertools::Itertools;

This file should be deleted no?

[shaharsamocha7]

Reviewable status: 11 of 13 files reviewed, 1 unresolved discussion (waiting on @alonh5)

---

crates/prover/src/trace_generation/prove.rs line 1 at r6 (raw file):

Previously, alonh5 (Alon Haramati) wrote…

This file should be deleted no?

Yeah, merge conflict :(

[shaharsamocha7]

Reviewed 1 of 2 files at r7.
Reviewable status: 12 of 13 files reviewed, 1 unresolved discussion (waiting on @alonh5)

[shaharsamocha7]

Reviewed 1 of 2 files at r7, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @alonh5)

[alonh5]

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

[andrewmilson]

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