-
-
Notifications
You must be signed in to change notification settings - Fork 213
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
Removed line 2108 in cvui.h #77
Conversation
This line called the function cv::rectangle from opencv with an empty Mat, which throws an exception
Hi and thank you for your contributions! Pull requests should be made against the |
I changed the base to dev. Sorry for the misunderstanding. |
Thank you! No need to be sorry, it's my own fault not to document those things ;) |
Thank you! 🎉 I'll merge your PR along with the other PRs as soon as I have some free time to work on cvui. |
Thanks! And thank you for the work you've done, cvui is extremely useful :) |
The pleasure is all mine to see cvui being used. And to receive time and energy from other developers to help improve it. Thank you! :) |
Hello, remainder, that this should be merged. Almost year after this report, I hit the assertion error of #81 in debug mode. So I fixed it locally, and then found this old report. Please merge... |
Ok, merged. I am truly sorry for the absence regarding the maintenance of this project in the last year. I am doing my best to allocate time for it. |
Thank you for the merge. I can understand the lack of time. Thanks for this useful tool. |
Hi. This PR solves the bug stated in #71.