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

adding updating native transcript with poseidon2, modify transcript to be vector of bn254 fr #777

Closed
lucasxia01 opened this issue Nov 17, 2023 · 0 comments · Fixed by AztecProtocol/aztec-packages#3718
Assignees

Comments

@lucasxia01
Copy link
Contributor

No description provided.

@lucasxia01 lucasxia01 self-assigned this Nov 17, 2023
@lucasxia01 lucasxia01 changed the title adding hashing to recursive transcript, updating native transcript with poseidon2 adding updating native transcript with poseidon2, modify transcript to be vector of bn254 fr Jan 19, 2024
lucasxia01 added a commit to AztecProtocol/aztec-packages that referenced this issue Jan 30, 2024
Change the transcript proof to be a vector of field elements instead of
a vector of bytes. We want this more crucially for the stdlib
transcript, but the native transcript should be implemented similarly.

The key contribution is the conversion functions in
field_conversion.hpp/cpp, which allow for types, in particular
grumpkin::fr, to be converted to and from bb::fr elements.

Some notes about why this was chosen:
https://hackmd.io/2XU21v9WTx6XUSnMgZIUig. In short, it is the most
efficient in terms of the number of gates.

Updates the transcript deserialization/serialization and corresponding
tests and makes other style updates.

Resolves AztecProtocol/barretenberg#777

Also some notes from trying to resolve bb.js ACIR tests:
https://hackmd.io/OrQUD_nTQLeIZVvy4zpzHA
AztecBot pushed a commit that referenced this issue Jan 31, 2024
Change the transcript proof to be a vector of field elements instead of
a vector of bytes. We want this more crucially for the stdlib
transcript, but the native transcript should be implemented similarly.

The key contribution is the conversion functions in
field_conversion.hpp/cpp, which allow for types, in particular
grumpkin::fr, to be converted to and from bb::fr elements.

Some notes about why this was chosen:
https://hackmd.io/2XU21v9WTx6XUSnMgZIUig. In short, it is the most
efficient in terms of the number of gates.

Updates the transcript deserialization/serialization and corresponding
tests and makes other style updates.

Resolves #777

Also some notes from trying to resolve bb.js ACIR tests:
https://hackmd.io/OrQUD_nTQLeIZVvy4zpzHA
michaelelliot pushed a commit to Swoir/noir_rs that referenced this issue Feb 28, 2024
Change the transcript proof to be a vector of field elements instead of
a vector of bytes. We want this more crucially for the stdlib
transcript, but the native transcript should be implemented similarly.

The key contribution is the conversion functions in
field_conversion.hpp/cpp, which allow for types, in particular
grumpkin::fr, to be converted to and from bb::fr elements.

Some notes about why this was chosen:
https://hackmd.io/2XU21v9WTx6XUSnMgZIUig. In short, it is the most
efficient in terms of the number of gates.

Updates the transcript deserialization/serialization and corresponding
tests and makes other style updates.

Resolves AztecProtocol/barretenberg#777

Also some notes from trying to resolve bb.js ACIR tests:
https://hackmd.io/OrQUD_nTQLeIZVvy4zpzHA
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant