Skip to content

Conversation

@IlyasRidhuan
Copy link
Contributor

@IlyasRidhuan IlyasRidhuan commented Feb 12, 2026

Pre-audit for the public logs opcode.

Mostly verbose documentation and general cleanup. Major changes were

  • Simplification of some constraints,
  • Reduction of some relation degrees (which enabled the removal of 1 column) and
  • Removing some implied boolean constraints
  • Some negative tests

Copy link
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@IlyasRidhuan IlyasRidhuan force-pushed the ir/02-11-fix_avm_emit_public_log_preaudit branch from 0741434 to e6c0291 Compare February 12, 2026 11:19
@IlyasRidhuan IlyasRidhuan force-pushed the ir/02-10-fix_avm_rename_unencrypted_log_to_public_log branch from 7c91f4b to 680421f Compare February 12, 2026 11:19
@IlyasRidhuan IlyasRidhuan changed the title first pass fix(avm)!: emit public log pre-audit Feb 12, 2026
@IlyasRidhuan IlyasRidhuan marked this pull request as ready for review February 12, 2026 11:22
@IlyasRidhuan IlyasRidhuan force-pushed the ir/02-11-fix_avm_emit_public_log_preaudit branch from e6c0291 to d19d59b Compare February 12, 2026 17:37
@IlyasRidhuan IlyasRidhuan force-pushed the ir/02-10-fix_avm_rename_unencrypted_log_to_public_log branch from 680421f to b2e3e0f Compare February 12, 2026 17:37
@AztecBot
Copy link
Collaborator

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/01543e760bf5897f�01543e760bf5897f8;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_epochs/epochs_proof_fails.parallel.test.ts "does not allow submitting proof after epoch end" (241s) (code: 0) group:e2e-p2p-epoch-flakes

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants