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.

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

keccak dynamic and round #239

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

keccak dynamic and round #239

wants to merge 4 commits into from

Conversation

Valeh2012
Copy link
Contributor

Keccak component in bbf; closes #147

Copy link

github-actions bot commented Jan 3, 2025

Clang Test Results

  164 files  +1    164 suites  +1   14m 46s ⏱️ -36s
7 282 tests +5  7 276 ✅ +5  6 💤 ±0  0 ❌ ±0 
7 356 runs  +5  7 350 ✅ +5  6 💤 ±0  0 ❌ ±0 

Results for commit 274a8b0. ± Comparison against base commit e256ae8.

This pull request removes 4 and adds 9 tests. Note that renamed tests count towards both.
blueprint_plonk_test_suite ‑ blueprint_plonk_hashes_keccak_round_pallas
blueprint_plonk_test_suite ‑ blueprint_plonk_hashes_keccak_round_pallas_diff_perm_cols
zkevm_opcode_test_suite ‑ pushx
zkevm_opcode_test_suite ‑ pushx_strings
blueprint_plonk_test_suite ‑ blueprint_plonk_hashes_keccak_round_bbf_not_random_pallas
blueprint_plonk_test_suite ‑ blueprint_plonk_hashes_keccak_round_bbf_random_pallas
lpc_math_polynomial_suite ‑ lpc_basic_skipping_layers_test
lpc_math_polynomial_suite ‑ lpc_dfs_basic_test
lpc_params_test_suite ‑ lpc_batches_num_3_test
lpc_params_test_suite ‑ lpc_different_hash_types_test
zkevm_bbf_exp ‑ one_exponent
zkevm_opcode_bbf_test_suite ‑ pushx
zkevm_opcode_bbf_test_suite ‑ pushx_strings

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 3, 2025

Gcc Test Results

  164 files  +1    164 suites  +1   15m 49s ⏱️ -24s
7 282 tests +1  7 276 ✅ +1  6 💤 ±0  0 ❌ ±0 
7 356 runs  +1  7 350 ✅ +1  6 💤 ±0  0 ❌ ±0 

Results for commit 274a8b0. ± Comparison against base commit e256ae8.

This pull request removes 4 and adds 5 tests. Note that renamed tests count towards both.
blueprint_plonk_test_suite ‑ blueprint_plonk_hashes_keccak_round_pallas
blueprint_plonk_test_suite ‑ blueprint_plonk_hashes_keccak_round_pallas_diff_perm_cols
zkevm_opcode_test_suite ‑ pushx
zkevm_opcode_test_suite ‑ pushx_strings
blueprint_plonk_test_suite ‑ blueprint_plonk_hashes_keccak_round_bbf_not_random_pallas
blueprint_plonk_test_suite ‑ blueprint_plonk_hashes_keccak_round_bbf_random_pallas
zkevm_bbf_exp ‑ one_exponent
zkevm_opcode_bbf_test_suite ‑ pushx
zkevm_opcode_bbf_test_suite ‑ pushx_strings

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

keccak_round_bbf
3 participants