Skip to content
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

Handle whole domain commitment by pointer #1984

Merged
merged 6 commits into from
Jan 27, 2025

Conversation

Trivo25
Copy link
Member

@Trivo25 Trivo25 commented Jan 27, 2025

previously, returning a UintXArray from wasm inside a worker caused the worked to idle indefinitely because it waits for a pointer to indicate when the worker finished processing a function. however, UintXArray return values were coerced into zero pointers - resulting in the worker idling forever. we now return a pointer instead of a UintXArray directly and later consuming that pointer in the main thread. see https://github.com/o1-labs/o1js-bindings/blob/46960832d283b6286c63b9784f1e34e80c28a2d1/js/web/worker-spec.js#L110

companion
o1-labs/o1js-bindings#327
MinaProtocol/mina#16529

This reverts commit 3a4f6cd.
@Trivo25 Trivo25 marked this pull request as ready for review January 27, 2025 10:21
@Trivo25 Trivo25 requested a review from a team as a code owner January 27, 2025 10:21
@Trivo25 Trivo25 merged commit 3029504 into fix/compatibility Jan 27, 2025
0 of 4 checks passed
@Trivo25 Trivo25 deleted the fix/compatibility-followup branch January 27, 2025 13:04
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please update the npm-deps-hash

@@ -1 +1 @@
Subproject commit 36df9f04e0fd37c2178f22972a94fffc106dd78d
Subproject commit 98d8ebb109e3e25e3cfeabec0f11bdf0a202d8e3
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Subproject commit 98d8ebb109e3e25e3cfeabec0f11bdf0a202d8e3
Subproject commit 98d8ebb109e3e25e3cfeabec0f11bdf0a202d8e3-dirty

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants