Skip to content
This repository has been archived by the owner on Jul 5, 2024. It is now read-only.

Proof generation for EVM circuit fails with error: RootCircuit::new: Transcript(Other, "Identity point in preprocessed or proof is not yet supported") #1334

Closed
AronisAt79 opened this issue Mar 25, 2023 · 3 comments · Fixed by #1471
Assignees
Labels
T-bug Type: bug

Comments

@AronisAt79
Copy link
Contributor

What command(s) is the bug in?

No response

Describe the bug

Proof request for block consisting of single l2 transaction
Proof request options:

('{"jsonrpc":"2.0", "method":"proof", '
'"params":[{"block":33,"circuit":"evm","aggregate":true,"mock":false,"mock_feedback":true,"rpc":"http://leader-testnet-geth:8545/", '
'"retry":false}], "id":33}')

Error:
RootCircuit::new: Transcript(Other, "Identity point in preprocessed or proof is not yet supported")

Circuit Commit: https://github.com/privacy-scaling-explorations/zkevm-circuits.git@branch=main
Chain Commit: eaab99aeb31b92ed8870548414c37148e18ac78d

Concrete steps to reproduce the bug

No response

@AronisAt79 AronisAt79 added the T-bug Type: bug label Mar 25, 2023
@AronisAt79
Copy link
Contributor Author

@ed255
Copy link
Member

ed255 commented Jun 2, 2023

This error comes from the Root Circuit not supporting empty witness/fixed columns. Our halo2 fork always fills witness columns with blinding factors, so we rule out the possibility of having an empty witness column. This means we probably have a fixed column that is all 0s (which means that the column isn't used). Removing this column / assigning some values to it should solve this issue.

@ed255
Copy link
Member

ed255 commented Jun 2, 2023

I have identified a fixed column which had no assignment when taking the EVM Circuit as a standalone circuit. I've fixed it here 54f2643

@AronisAt79 whenever you have some time, could you try running the real prover test with EVM Circuit + Root circuit on that commit?

@ed255 ed255 closed this as completed in 0ecd781 Jun 20, 2023
cameron1024 pushed a commit to cameron1024/zkevm-circuits that referenced this issue Jun 30, 2024
* done

* fix

* block env from

* refactor trace struct
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-bug Type: bug
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants