Uriel Chemouni
Uriel Chemouni
hi, I'm currently working on a new opencv binding, so I'm less available for now.
Look's correct; one more deep breath, and you can open your first PR. Spooler alert: To be merged, a PR must contain a test function covering the new code (should...
Extra note, small PR, are much more appreciated than bigger ones, so put each fix in a different PR. You can create multiple branches on your fork to keep it...
Try a fork; with the new version, you will lose less time in re-building. And do not forget to define a Global variable named `OPENCV_BUILD_ROOT` pointing to a build directory.
Thank you for pointing out the file that needs correction. I'm currently facing homelessness, so I'll address that once I've resolved this pressing issue.
I'm still not using electron, for now, can not help.
1st: do not use 4.7.0 use 4.6.0 2nd: define an env variable `OPENCV_BUILD_ROOT=c:/opencv` 3th: never use "opencvBinDir" to point a lib built by @u4/opencv-build 4th: I'm still not using electron....
Look for another DNN Implementation, and write a new binding from it. That kind of change is not difficult. and put your binding inside a `#if CV_VERSION_GREATER_EQUAL(4, 4, 0)` block...
Try to provide me with more context, check the dockerfiles and the GitHub actions files, they contain lots of build samples.
can you provide a sample project? I have some mac M1 to test, but I do not use electron. is [opencv-electron](https://github.com/UrielCh/opencv-electron) working ?