-
Notifications
You must be signed in to change notification settings - Fork 25
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
Prf eval #795
Prf eval #795
Conversation
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 haven't gotten all the way yet. I'm sure that others will help further.
I recommend trying cargo fmt
at some point.
src/ff/curve_points.rs
Outdated
type Output = Self; | ||
|
||
fn add(self, rhs: Self) -> Self::Output { | ||
Self((self.0.decompress().unwrap()+rhs.0.decompress().unwrap()).compress()) |
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.
This is potentially problematic. Do we really want to do compression and decompression for every local operation? Could we instead store this object uncompressed and only compress when we sent it out?
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.
Probably need to look more detailed into the tradeoff between memory cost vs cpu. I think decompressed points take 3 times more memory (I haven't checked how costly the compression is). Currently we do not do much arithmetics on curve points, just one scalar multiplication and one reveal which reconstructs the shares using two additions per party.
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.
good start, I haven't looked at everything, can make another pass later
Co-authored-by: Alex Koshelev <[email protected]>
Co-authored-by: Alex Koshelev <[email protected]>
Co-authored-by: Alex Koshelev <[email protected]>
Co-authored-by: Alex Koshelev <[email protected]>
…F-Eval # Conflicts: # src/protocol/ipa_prf/prf_sharding/bucket.rs
closed, see #833 (same functionality, cleaned up commit history) |
implementation of DY evaluation protocol based on replicated secret sharing