Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

Fixed base scalar mul needs to use two limbs to encode the scalar field of grumpkin #531

Closed
kevaundray opened this issue Sep 7, 2023 · 0 comments · Fixed by #532
Closed
Labels
bug Something isn't working

Comments

@kevaundray
Copy link
Contributor

Aim

The fixed base scalar multiplication algorithm needs more than a single field element to encode the scalar field of the embedded curve.

Expected Behavior

We should change it to have two limbs which works for the curves we currently care about and in the future, we should generalize it with perhaps a byte array abstraction, or if we find out that two limbs is always enough we can stick with two limbs

Bug

To Reproduce

Environment

No response

Additional Context

No response

Would you like to submit a PR for this Issue?

No

Support Needs

No response

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant