-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add missing JS binding functions #14
Conversation
rbonichon
commented
May 8, 2023
- group example custom gates test functions together at the end
- provide simple int64_of_int32 missing conversion function
- add caml_fq_srs_add_lagrange_basis, whichs an updating function (no return value)
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 for adding these!
839ca2a
to
8c6be40
Compare
- group *_example_* custom gates test functions together at the end - provide simple int64_of_int32 missing conversion function - add caml_fq_srs_add_lagrange_basis, whichs an updating function (no return value)
8c6be40
to
5bb793e
Compare
This PR should specify what Mina / snarkyjs PR it is compatible with |
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 added a snarkyjs PR which includes these changes, to test them in CI: o1-labs/o1js#1001
|
||
// Provides: integers_uint64_of_uint32 | ||
// Requires: UInt64, caml_int64_of_int32 | ||
function integers_uint64_of_uint32(i) { |
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.
Makes sense to have this converter. However, I would rather forbid the use of uint64 in the code to avoid 64bits values as wasm supports only 32bits.
LGTM, waiting for the CI |