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
Disentangle Pedersen generators from Bulletproofs generators #163
Conversation
It looks like a toolchain bump is in order! I updated other PRs to use |
6f6593e
to
53b5266
Compare
I also had to amend the commit to change the benchmarks to match the new API, so it was probably broken in at least two ways |
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.
LGTM
(I just have one q about the randomness)
|
||
#[test] | ||
fn test_delta() { | ||
let mut rng = OsRng::new().unwrap(); | ||
let mut rng = rand::thread_rng(); |
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.
what's the motivation for changing the source of randomness?
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.
There's no really good reason -- I removed the OsRng
that was used in other parts of the test code while changing that code, then removed the use
declaration that brought it into scope, then realized that the test_delta
function was using it, and didn't want to re-add it, so I just made everything use thread_rng
Closes #161.