-
Notifications
You must be signed in to change notification settings - Fork 51
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
Support for BunJS? #153
Comments
Nope, this OpenCV4nodeJS is built on top of the previous Node native API. I tried to build a new one from scratch using the modern N-API and mapping the same binary as the official Python OpenCV API, but it took me more time than I can allocate to it. See: https://github.com/UrielCh/opencv-sandbox I have converted some common Python binding calls to N-API, and I think the project needs four more months to complete. |
Ah I see, well that's a shame but completely understandable. Ill see if I can use the sandbox to try and use the functionality I need or even contribute to it. Thanks for the great work anyways. |
I finally change my mind and start to migrate the project to N-API so I will be able to use it with Deno and BunJS. |
@UrielCh how is it going? Is there something I can help with? |
I have updated the project node-addon-examples to improve the N-API samples (4 PR opened), but the maintainers don't seem very interested in my proposal. The active branch for this migration is: Currently, I need to focus on finding a full-time job, so I have little time to dedicate to this project. |
Hmm. How is this related to opencv4nodejs n-api migration? |
Hey,
I'm wondering if there is a potential way to support BunJS?
As for now I am running in to the following issue with regards to Node API:
The API
napi_register_module_v1
is supported in BunJS but it seems like it still does not work for me. Been trying to hack it too with no progress but any suggestions very much welcome.For reference I have also asked bun about this issue here.
As a work around I have been building .mjs scripts and piping back the result to Bun. Which as you can imagine is super cumbersome.
The text was updated successfully, but these errors were encountered: