Skip to content

Pedersen hash on stark-curve#327

Merged
yelhousni merged 4 commits intodevelopfrom
pedersen-hash
Feb 3, 2023
Merged

Pedersen hash on stark-curve#327
yelhousni merged 4 commits intodevelopfrom
pedersen-hash

Conversation

@yelhousni
Copy link
Copy Markdown
Collaborator

@yelhousni yelhousni commented Feb 3, 2023

@yelhousni yelhousni mentioned this pull request Feb 3, 2023
Closed
yelhousni referenced this pull request Feb 3, 2023
@yelhousni
refactor(ecdsa): remove Params struct
eb8bce9
@yelhousni
Copy link
Copy Markdown
Collaborator Author

yelhousni commented Feb 3, 2023

Some optimisations in mind:

  • the scalar multiplication by highPart can be fully or partially hardcoded, because the points (p1 and p3) are constants and the scalar is always 4 bits.
  • instead of recovering the x coordinate in Pedersen() you can recover it in PedersenArray() and mutualise all the inverses 1/zi

Will merge this now.

@yelhousni yelhousni merged commit e77b091 into develop Feb 3, 2023
@yelhousni yelhousni deleted the pedersen-hash branch February 3, 2023 17:20
@yelhousni yelhousni mentioned this pull request Apr 13, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gbotrel gbotrel gbotrel approved these changes

@ThomasPiellard ThomasPiellard Awaiting requested review from ThomasPiellard

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yelhousni @gbotrel @anshulforyou
⚡