Skip to content

Conversation

@querolita
Copy link
Member

@querolita querolita commented Oct 16, 2025

Creates JSOO bindings for the gate-vector.

o1js side: o1-labs/o1js#2564
mina side: #17951 (this)
proof-systems side: o1-labs/proof-systems#3341

(parent PR: #17888)

Closes o1-labs/o1js#2512

@querolita querolita changed the title [WIP] (mina) Native Prover with Napi - GateVector (mina) Native Prover with Napi - GateVector Oct 16, 2025
@querolita querolita force-pushed the native/napi-polycomm branch from 003af6b to fa80794 Compare October 16, 2025 17:45
@querolita querolita force-pushed the native/napi-gatevector branch from a2359e4 to d4ff2e9 Compare October 16, 2025 17:50
@querolita querolita marked this pull request as ready for review October 20, 2025 18:12
@querolita querolita requested a review from a team as a code owner October 20, 2025 18:12
@bleepbloopsify
Copy link
Member

bleepbloopsify commented Nov 5, 2025

Is there a reason this branch is on napi-polycomm rather than native/napi? (your description says that the parent is native/napi)

@querolita querolita changed the base branch from native/napi-polycomm to native/napi November 5, 2025 23:13
@querolita querolita changed the base branch from native/napi to native/napi-polycomm November 5, 2025 23:14
@querolita
Copy link
Member Author

Is there a reason this branch is on napi-polycomm rather than native/napi? (your description says that the parent is native/napi)

nice catch! this must have been an error on my side, it's been a long time that napi/native-polycomm does not exist on the mina side (#17895)

Rebasing now, makes everything look better

@querolita querolita changed the base branch from native/napi-polycomm to native/napi November 5, 2025 23:15
@querolita querolita merged commit 6ea5599 into native/napi Nov 6, 2025
1 check passed
@querolita querolita deleted the native/napi-gatevector branch November 6, 2025 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants