Fix and re-enable e2e uniswap test #10030
Comments
lucasxia01
pushed a commit
that referenced
this issue
Dec 11, 2024
…ow 20k. (#10214) Resolves #10030 Bump of AVM startup gas to 20k is mostly arbitrary, but considering some individual opcodes cost more than its previous value of 512, it certainly needed to be much higher. I thought 20k is at least _more_ reasonable to account for the constraint cost of verifying an AVM proof. The l2 gas maximum per-tx-public-portion ensures that there is some hard limit on execution per AVM proof. For now, we use that limit to ensure that you cannot overflow the AVM trace. --------- Co-authored-by: IlyasRidhuan <[email protected]>
AztecBot
pushed a commit
to AztecProtocol/barretenberg
that referenced
this issue
Dec 12, 2024
…ow 20k. (#10214) Resolves AztecProtocol/aztec-packages#10030 Bump of AVM startup gas to 20k is mostly arbitrary, but considering some individual opcodes cost more than its previous value of 512, it certainly needed to be much higher. I thought 20k is at least _more_ reasonable to account for the constraint cost of verifying an AVM proof. The l2 gas maximum per-tx-public-portion ensures that there is some hard limit on execution per AVM proof. For now, we use that limit to ensure that you cannot overflow the AVM trace. --------- Co-authored-by: IlyasRidhuan <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No description provided.
The text was updated successfully, but these errors were encountered: