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.

Sign up for GitHub

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

Jump to bottom

OpenCV3 Regression error #310

Closed
miquelmassot opened this issue Nov 30, 2017 · 2 comments
Closed

OpenCV3 Regression error #310

miquelmassot opened this issue Nov 30, 2017 · 2 comments

Comments

@miquelmassot
Copy link
Contributor

See opencv/opencv#7599

In

image_pipeline/stereo_image_proc/include/stereo_image_proc/processor.h

Line 180 in 2589ed2

mutable cv::Mat_<uint32_t> labels_;
and

image_pipeline/stereo_image_proc/include/stereo_image_proc/processor.h

Line 181 in 2589ed2

mutable cv::Mat_<uint32_t> wavefront_;
two cv::Mat_<uint32_t> matrices are declared but not used, and their types are no longer supported by OpenCV3.

Related from srv/srv_tools#17 and ros-infrastructure/rep#140
miquelmassot added a commit to miquelmassot/image_pipeline that referenced this issue Dec 14, 2017
@miquelmassot
Removed unused mutable scratch buffers
9b91b07 
The uint32_t buffers conflicted with newer release of OpenCV3, as explained here ros-perception#310
@miquelmassot miquelmassot mentioned this issue Dec 14, 2017
Open
@miquelmassot miquelmassot mentioned this issue Dec 27, 2017
Merged
@tspicer01
Copy link

Is there an ETA on when we could expect the fix ?
We would really like to move to the new version of opencv3.

vrabaud pushed a commit that referenced this issue May 10, 2018
@miquelmassot @vrabaud
Removed unused mutable scratch buffers (#315)
696518b 
The uint32_t buffers conflicted with newer release of OpenCV3, as explained here #310
@JWhitleyWork
Copy link
Collaborator

Closed by #315.

generush pushed a commit to generush/image_pipeline that referenced this issue May 9, 2019
@miquelmassot @vrabaud
Removed unused mutable scratch buffers (ros-perception#315)
673ef53 
The uint32_t buffers conflicted with newer release of OpenCV3, as explained here ros-perception#310
jonasgerne pushed a commit to jonasgerne/stereo_image_proc that referenced this issue Nov 5, 2019
@miquelmassot @vrabaud
Removed unused mutable scratch buffers (#315)
4b13246 
The uint32_t buffers conflicted with newer release of OpenCV3, as explained here ros-perception/image_pipeline#310
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@miquelmassot @JWhitleyWork @tspicer01