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

[noetic] Adapted to OpenCV4 #1068

Merged

Conversation

ahcorde
Copy link
Contributor

@ahcorde ahcorde commented Mar 26, 2020

Modified headers for OpenCV4

Signed-off-by: ahcorde ahcorde@gmail.com

Signed-off-by: ahcorde <ahcorde@gmail.com>
@chapulina chapulina mentioned this pull request Mar 30, 2020
Copy link
Contributor

@chapulina chapulina left a comment

Choose a reason for hiding this comment

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

I haven't tried it, but we have positive feedback from the community, so merging.

@chapulina chapulina merged commit 162adea into ros-simulation:noetic-devel Mar 31, 2020
@timonegk
Copy link

timonegk commented May 20, 2020

Could this be backported to melodic? The new header style has been introduced in OpenCV2.4.11, so that would introduce no problems for melodic (target cv version is 3.2+, see REP 3) but makes it possible to use melodic with cv4.

@chapulina
Copy link
Contributor

@timonegk , sure, please feel free to open a pull request against melodic and we can merge it if CI passes. Thanks!

@timonegk
Copy link

Okay, I opened a PR, thank you!

seanyen pushed a commit to ms-iot/gazebo_ros_pkgs that referenced this pull request Aug 31, 2020
Signed-off-by: ahcorde <ahcorde@gmail.com>
nim65s added a commit to nim65s/robotpkg-wip that referenced this pull request Sep 27, 2021
cohen39 pushed a commit to cohen39/gazebo_ros_pkgs that referenced this pull request Nov 15, 2021
Signed-off-by: ahcorde <ahcorde@gmail.com>
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

3 participants