-
Notifications
You must be signed in to change notification settings - Fork 238
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
Expand the Solidity verifier (Barretenberg) compatibility section in docs to cover Top 10 EVMs and how to contribute compatibility tests #4964
Comments
I'd recommend just linking off to https://www.evmdiff.com/features?feature=precompiles and just telling people it needs to support ec_add, ec_mul, ec_pairing. This avoids us from having to explicitly enumerate every single EVM network which wants a shoutout in our docs or gatekeeping this along with the inevitable arguments about which networks are deserving (i.e. if it's notable enough for inclusion in noir docs then it should be notable enough for evm.diff) |
@Savio-Sou I added Tom's suggestion in the linked above PR. I see that evm diff doesnt cover the top 10 EVM chains by TVL, but do you think this is sufficient? Or do you still think we should cover the rest of the list in the description? |
Thanks for the suggestion @TomAFrench! @critesjosh that's a great starting step 👍 with 4 of the top 10 EVMs by TVL not covered by EVM Diff (as of this time), it is worthwhile still to test them imo for wider reach and better neutrality in the marketing to follow. |
you're right, it shouldn't be a big effort. it's just a lower priority so we haven't gotten around to it yet |
# Description Links to evmdiff.com to show which EVM chains have the required precompiles ## Problem\* related to #4964 ## Summary\* ## Additional Context ## Documentation\* Check one: - [ ] No documentation needed. - [x] Documentation included in this PR. - [ ] **[For Experimental Features]** Documentation to be submitted in a separate PR. # PR Checklist\* - [ ] I have tested the changes locally. - [ ] I have formatted the changes with [Prettier](https://prettier.io/) and/or `cargo fmt` on default settings.
Once we have the subset to test, these come in handy for chainids and rpc providers. Faucets still likely a manual process though :/ |
This just looks like a contract creation tx. Don't you need to send a tx that calls |
It's a view function so you can't see it on the block explorer. I'm working on making noir-starter easier to "attach" to a deployed contract though |
# Description Closes #4964 Adds some chains we empirically know can verify Noir proofs.
Problem
We know Barretenberg, Noir's default proving backend loaded, generates Solidity verifiers for programs that are deployable on Ethereum.
It is however not directly clear with the same applies to other EVMs which might miss certain precompiles or specific EVM features.
Happy Case
Expand the current list we have in the Noir docs to cover:
Project Impact
Nice-to-have
Impact Context
Great reference for:
Workaround
None
Workaround Description
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response
The text was updated successfully, but these errors were encountered: