Skip to content

Conversation

AztecBot
Copy link
Collaborator

@AztecBot AztecBot commented Sep 3, 2025

BEGIN_COMMIT_OVERRIDE
chore: audit ECCVM msm relation (#16532)
feat: add storage budget control for FileBackedMemory (#16690)
fix(bb): fix honk_solidity_key_gen debug build (#16744)
chore: wasm fix (#16751)
END_COMMIT_OVERRIDE

Audit of the MSM relation.

There were a couple of columns that I think were underconstrained. I have made the MSM relations more
robust/clear as to why they are correct.

Aside from a few changes to the relations, this PR mostly involves a lot of documentation. The most important part of this is re: the multiset equality check. The description of _why_ this suffices to correctly constrain the q_add columns is more transparent now.

---------

Co-authored-by: notnotraju <[email protected]>
@ludamad ludamad requested a review from charlielye as a code owner September 3, 2025 14:30
Copy link
Collaborator

@ludamad ludamad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Auto-approved

@AztecBot AztecBot added this pull request to the merge queue Sep 3, 2025
@AztecBot
Copy link
Collaborator Author

AztecBot commented Sep 3, 2025

🤖 Auto-merge enabled after 4 hours of inactivity. This PR will be merged automatically once all checks pass.

github-merge-queue bot pushed a commit that referenced this pull request Sep 3, 2025
BEGIN_COMMIT_OVERRIDE
chore: audit ECCVM msm relation (#16532)
feat: add storage budget control for FileBackedMemory (#16690)
fix(bb): fix honk_solidity_key_gen debug build (#16744)
END_COMMIT_OVERRIDE
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 3, 2025
@ludamad ludamad added the ci-full Run all master checks. label Sep 4, 2025
@AztecBot AztecBot added this pull request to the merge queue Sep 4, 2025
@AztecBot
Copy link
Collaborator Author

AztecBot commented Sep 4, 2025

🤖 Auto-merge enabled after 4 hours of inactivity. This PR will be merged automatically once all checks pass.

Merged via the queue into next with commit c29739c Sep 4, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-full Run all master checks. ci-no-squash
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants