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

blst lagrange interpolation #95

Draft
wants to merge 10 commits into
base: unstable
Choose a base branch
from

Conversation

dknopik
Copy link
Member

@dknopik dknopik commented Jan 13, 2025

Proof of concept of using blst for signature recovery.

We should discuss whether we want to use this or blsful.

Arguments for this impl:

  • We avoid a bunch of new dependencies (which is nice, as we are working with keys here)
  • We do not have to add too much ugly switching of types this way (as blsful uses a whole stack of own types. Lighthouse instead uses their own bls crate that builds upon blst)
  • I daresay it's faster, but I have not tested that 😬

Arguments for blsful:

  • No unsafe needed
  • This impl needs some (hopefully harmless) changes upstreamed to blst
  • DIY cryptography is scary

@Zacholme7
Copy link
Member

Zacholme7 commented Jan 14, 2025

do you have a poc using blsful? Id almost say not having to use unsafe and roll some diy cryptography is worth more messy types but there are valid arguments for both libs.

@dknopik
Copy link
Member Author

dknopik commented Jan 14, 2025

@Zacholme7 not yet, will do

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.

2 participants