-
Notifications
You must be signed in to change notification settings - Fork 59
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
Get random2 #171
Get random2 #171
Conversation
I'm done with this PR. It will be nice to review it. |
Can you please rebase to fix the conflicts? Thanks. |
Merge done. |
24f114e
to
814ef9e
Compare
Libjade fails:
cc @tfaoliveira |
I will try to understand why this happens |
The following program (similar to the usage in Kyber) fails:
with the error:
Is there something wrong with it? |
Looks like a bug in this PR. I’ll try to understand. |
Fixed! Thanks Tiago for your help. |
I'm compiling this branch to run some tests locally as well |
It seems fine on my side: the checksums of the implementations match; I think I didn't miss anything. |
Great! I’ve also changed the tests a bit to avoid link failures in some environments. |
Fills its argument with unspecified data Co-authored-by: Benjamin Grēgoire <[email protected]>
Are we ready ? |
The patch for the test works on my MAC |
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 a major, long-awaited, change!
Need to fix the ocaml part.