Skip to content

Add debug-only asserts to check the 2*p relaxed modular form #1429

Closed
AztecProtocol/aztec-packages
#21178

Description

@ludamad

We allow fields to be over the modulus, up to 2p. Adding debug-only asserts would be good (partially) checked documentation

Activity

  1. ludamad commented on Oct 21, 2025

    @ludamad
    ContributorAuthor

    I think if we do this, it makes sense to do it before the external audit. I lean in favour as the code is a lot easier to read and can be useful in debug to catch a malformed field > 2p being used @ledwards2225

  2. notnotraju commented on Feb 26, 2026

    @notnotraju

    this is a great idea.

  3. notnotraju commented on Mar 5, 2026

    @notnotraju

    /claudebox please do this.

  4. AztecBot commented on Mar 5, 2026

    @AztecBot
    Collaborator

    ClaudeBox: please do this. ... workflow run

  5. added a commit that references this issue on Mar 13, 2026
    66422f3
  6. notnotraju commented on Mar 13, 2026

    @notnotraju

    done

  7. added a commit that references this issue on Mar 19, 2026
    03a3f34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

auditThings to do during the next auditnice-to-have

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions