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

epic: point evaluation precompile sound #1003

Open
Tracked by #174
obatirou opened this issue Mar 13, 2025 · 1 comment
Open
Tracked by #174

epic: point evaluation precompile sound #1003

obatirou opened this issue Mar 13, 2025 · 1 comment
Assignees
Labels
epic Label an issue with sub issues
Milestone

Comments

@obatirou
Copy link
Collaborator

obatirou commented Mar 13, 2025

What

Implement the Point Evaluation in Cairo to make it sound

EELS uses the optimized version of elliptic curve ops from the py_ecc implementation.
It uses the projective space to define points. Our circuits (and those of Garaga) are defined on the affine space.
The switch "affine <=> projective space" is quite easy so, to re-use our current circuit we will only use the affine space in Cairo, and make the conversion within the serde/args_gen for testing purposes (so only in python).

Context

EIP-4844 Point Evaluation Precompile Specs

The calldata decoding as already been done as part of #180 (merged by #862 )

@Eikix
Copy link
Member

Eikix commented Mar 31, 2025

TO DO:
1/ create an epic for this feature
2/ create sub-issue + plan
3/ estimate + put in TODO the sub issues

@Eikix Eikix moved this from Backlog to Todo in Keth Mar 31, 2025
@zmalatrax zmalatrax changed the title feat: point evaluation sound epic: point evaluation precompile sound Mar 31, 2025
@ClementWalter ClementWalter added this to the Precompiles milestone Apr 3, 2025
@ClementWalter ClementWalter added the epic Label an issue with sub issues label Apr 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic Label an issue with sub issues
Projects
Status: Todo
Development

No branches or pull requests

4 participants