-
Notifications
You must be signed in to change notification settings - Fork 28
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
Frost and MuSig joint key Point type parameter #131
Conversation
* Change musig and frost keys to allow for any point type * keygen creates Normal points * signing operations are done with EvenY points
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merging the two states of the key into one looks good.
- The type parameter should only be set to
T
in one function:gen_nonce_rng
.FrostKey
needs to be passed in here. - Delete the nonce generation stuff from
binonce
as the rng sutff supersedes it. - Add a
gen_nonce_rng
to theMuSig
type which takes theAggKey
in the same way you did it infrost
.
2c6de42
to
501d8f1
Compare
I believe this PR is ready for merge. I think we will leave cloning |
secret: &Scalar, | ||
session_id: &[u8], | ||
public_key: Option<Point<impl Normalized>>, | ||
// public_key: Option<Point<impl Normalized>>, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess just remove this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Recreating #128 minus the cherry-picked commit from #129