Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Sequencer] beta12 causes verifications to be off by one #850

Closed
mandrigin opened this issue Jul 26, 2024 · 0 comments · Fixed by #853
Closed

[Sequencer] beta12 causes verifications to be off by one #850

mandrigin opened this issue Jul 26, 2024 · 0 comments · Fixed by #853
Assignees
@V-Staykov
Milestone
S3. Sequencer on ...

Comments

@mandrigin
Copy link
Collaborator

In beta12 there is a regression that causes (with the mock prover) verifications to have off-by one error.

Basically, if a verification is for the batch 1, the state root of this verification will be of the batch 2, etc etc.

Reverting dc3cbcc59a95769626056c7bc70aade501e7741d and fc8972b017e43566adb588792df26bdccd02b4df solves the issue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@V-Staykov V-Staykov

Labels
None yet
Projects
None yet
Milestone
S3. Sequencer on Cardona Fork 11 (zkE...
Development

Successfully merging a pull request may close this issue.

fixed using correct blocks root for batch end 0xPolygonHermez/cdk-erigon
2 participants
@mandrigin @V-Staykov