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

blueprint/zkevm: fix wrong assert in opcode_tester #217

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

blueprint/zkevm: fix wrong assert in opcode_tester #217

wants to merge 1 commit into from

Conversation

ioxid
Copy link
Contributor

@ioxid ioxid commented Dec 20, 2024

Fixes #216

@ioxid ioxid requested a review from akokoshn December 20, 2024 12:04
@github-actions GitHub Actions
Copy link

Clang Test Results

  161 files  ±0    161 suites  ±0   16m 42s ⏱️ -1s
7 082 tests  - 4  7 076 ✅  - 4  6 💤 ±0  0 ❌ ±0 
7 168 runs   - 4  7 162 ✅  - 4  6 💤 ±0  0 ❌ ±0 

Results for commit 3e8241b. ± Comparison against base commit 9b3fb27.

This pull request removes 4 tests. 
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

@github-actions GitHub Actions
Copy link

Gcc Test Results

  161 files  ±0    161 suites  ±0   16m 57s ⏱️ -3s
7 086 tests ±0  7 080 ✅ ±0  6 💤 ±0  0 ❌ ±0 
7 172 runs  ±0  7 166 ✅ ±0  6 💤 ±0  0 ❌ ±0 

Results for commit 3e8241b. ± Comparison against base commit 9b3fb27.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akokoshn akokoshn Awaiting requested review from akokoshn

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

zkEVM: wrong assert in opcode_tester
1 participant
@ioxid