Closed victorkstarkware closed 4 months ago
Attention: Patch coverage is 91.42857%
with 3 lines
in your changes are missing coverage. Please review.
Files | Coverage Δ | |
---|---|---|
src/fibonacci/bitcoin_script.rs | 100.00% <100.00%> (ø) |
|
src/oods/bitcoin_script.rs | 100.00% <ø> (ø) |
|
src/fibonacci/mod.rs | 85.00% <85.00%> (ø) |
Implemented the (Fibonacci) composition polynomial inside the Fibonacci module. It doesn't have all the abstractions as in Stwo because we are not trying to build something general.
Note: Because in Stwo
step_constraint_eval_quotient_by_mask()
andboundary_constraint_eval_quotient_by_mask()
, are PRIVATE, their code had to be copied to fibonacci/mod.rs in 'FibonacciComposition'.How should we deal with this?
PS added "itertools" dependency.