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

[WIP] towards OpenCV4 support #94

Closed
wants to merge 2 commits into from
Closed

Conversation

umlaeute
Copy link
Contributor

this is the starting point for OpenCV-4 support.

some problems are trivial to fix by including the relevant new header files).

however, afaict the high-level object detection code has seen a major overhaul, and needs a complete rewrite.

this PR at least fixes the simple problems (allowing to compile facedetect, but not yet facebl0r.

imgproc_c.h defines the constants CV_BGR2GRAY, CV_FILLED & CV_AA.

it also includes "core/core_c.h", which then provides
  `cvGetTickCount()` and `cvGetTickFrequency()`
@umlaeute
Copy link
Contributor Author

relates to #93

@StefanBruens
Copy link
Contributor

The deprecated constructors have also been removed from OpenCV 3.4.9.

@@ -21,6 +21,9 @@
#include <stdio.h>
#include <string.h>
#include <opencv2/opencv.hpp>
#if CV_MAJOR_VERSION >= 4
# include <opencv2/imgproc/imgproc_c.h>
#endif
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't like this very much. This should be fixed by switching to the newer C++ API, see #99.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

if #99 fixes compat withOpenCV4, by all means please go ahead and use that!

@kwizart
Copy link
Contributor

kwizart commented May 25, 2020

This PR can be dropped...

@umlaeute
Copy link
Contributor Author

is there already OpenCV4 support?
if so, close this PR. if not keep it open.

@kwizart
Copy link
Contributor

kwizart commented May 25, 2020

I would keep a "ticket" for opencv4 support open, but others PR are more relevant than this one.
(#97 and #99), I've tried to consolidate theses with #101 with an additional patch that compile with OpenCV4.2 using cmake.

So this PR can be closed.

@ddennedy ddennedy closed this Sep 15, 2020
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.

None yet

5 participants