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

OpenCV4 Support. Take 57 #321

Closed
wants to merge 4 commits into from
Closed

Conversation

OAkyildiz
Copy link

@OAkyildiz OAkyildiz commented Feb 28, 2020

As requested in #308 and #288. The original commit belongs to @solosito, I cherry-picked it into the melodic branch. CI test is with Melodic on Bionic Beaver (as opposed to trusty).

! Original PR's were a) tested on kinetic w/ Thar, b) pulled from ros2

Copy link

@BarathwajAnandan BarathwajAnandan left a comment

Choose a reason for hiding this comment

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

.

@OAkyildiz
Copy link
Author

Should I close this?

@mjcarroll mjcarroll mentioned this pull request Mar 28, 2020
@ghost
Copy link

ghost commented Jun 22, 2021

Sorry, any update on this?

@OAkyildiz
Copy link
Author

@GuglielmoDev I don't believe this is not an issue anymore

@mjcarroll
Copy link
Contributor

OpenCV 4 support was added in the noetic release #323

@mjcarroll mjcarroll closed this Nov 23, 2021
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.

4 participants