Skip to content

Conversation

FilipLaurentiu
Copy link
Contributor

Cairo represent points on the curve just with the x coordinate (compressed). This is enough for signature verification but not enough for other operations on EC points.
This update provide a way to create a new AffinePoint from the x cordinate and the parity bit.

Pull Request type

  • Feature

What is the current behavior?

  • No option to create a new AffinePoint instance from the compressed representation.

What is the new behavior?

  • Create AffinePoints just from the x compressed representation

Does this introduce a breaking change?

No

Other information

Copy link
Collaborator

@gabrielbosio gabrielbosio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, I have left a small comment on the function documentation

@tdelabro tdelabro merged commit f984fb0 into starknet-io:main Jul 15, 2025
3 checks passed
@tdelabro
Copy link
Collaborator

Thanks @FilipLaurentiu, sorry for the 4 month it took us to merge this.
We will be more active on this repo from now on

@FilipLaurentiu
Copy link
Contributor Author

Thanks @FilipLaurentiu, sorry for the 4 month it took us to merge this. We will be more active on this repo from now on

Hey ! Thanks for the review and for the approval 👍

About the delay, the changes were not so impactful, but good to have, avoid the confusion about the point compression, but I notice delays on other, more important repos.
Starknet will benefit from giving more attention to these repos 🙏

@tdelabro
Copy link
Collaborator

@FilipLaurentiu, which other repos are you using where you faced this type of struggle? Maybe we can help improve things

@FilipLaurentiu
Copy link
Contributor Author

@FilipLaurentiu, which other repos are you using where you faced this type of struggle? Maybe we can help improve things

I am looking for some feedback on this one. It should be more involving and probably introduce some breaking changes, but it will be good to know if it's of interest or not.
And it is somehow related to this repo, here is my comment

@tdelabro
Copy link
Collaborator

@FilipLaurentiu do you have a telegram? If so, can you ping me at @tdelabro please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants