-
Notifications
You must be signed in to change notification settings - Fork 81
perf: new keccak run reth #2328
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
base: develop-new-keccak
Are you sure you want to change the base?
Conversation
CodSpeed Performance ReportMerging #2328 will not alter performanceComparing
|
…that runs with it
…dd ayush vector tests
…can only take multiple of 4 lengths
…ld be zero ed out in the len % 4 != 0 case
687a6ca to
ccb6238
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
2bd040d to
592b485
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
5d9cbdd to
36849f5
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Reth run panicked due to a error and it is unclear if it is due to the new keccak or not but @shuklaayush mentioned he will take a look. @shuklaayush can merge the PR if he likes to. https://github.com/axiom-crypto/openvm-reth-benchmark/actions/runs/20386012415/job/58586939468 |
Commit: 02c19f5 |
Closes INT-5779