Skip to content

Conversation

@znah
Copy link

@znah znah commented Aug 29, 2014

No description provided.

@StevenPuttemans
Copy link

Thanks for the update, it was indeed necessary. However on windows and linux builds this change breaks java wrapper wich seems weird to me. Any clue on how this can happen?

@StevenPuttemans
Copy link

This is the original commit that made this change:

ernest-galbrun@b757d6c

@StevenPuttemans
Copy link

Maybe suggest to disable java bindings for xfeatures 2d module until solutions drop out of the box? Ths can be done here:https://github.com/Itseez/opencv/blob/master/modules/java/CMakeLists.txt#L11

@vpisarev what do you think?

@apavlenko
Copy link
Contributor

Let me re-run the builds, it may be an external problem

@StevenPuttemans
Copy link

👍

@apavlenko apavlenko self-assigned this Sep 4, 2014
@apavlenko
Copy link
Contributor

I suggest we disable java bindings for xfeatures2d module
(opencv/opencv#3191)

@vpisarev
Copy link
Contributor

@mshabunin, reassigning this to you. Hopefully, you could help to resolve this confusing dependency problems

@mshabunin
Copy link
Contributor

All builds are passing now.

@apavlenko
Copy link
Contributor

@mshabunin , then you may 👍 it!

@mshabunin
Copy link
Contributor

👍

@znah
Copy link
Author

znah commented Sep 24, 2014

Come on, let's merge it already :)

@StevenPuttemans
Copy link

It can take some time :) Just be patient :P

mshabunin added a commit that referenced this pull request Sep 24, 2014
make opencv_cudaarithm dependency for xfeatures2d optional
@mshabunin mshabunin merged commit e640d6a into opencv:master Sep 24, 2014
@znah znah deleted the xfeatures2d_fix branch September 24, 2014 12:44
bmegli pushed a commit to Extend-Robotics/opencv_contrib that referenced this pull request Jul 11, 2022
* Add body index map image publishing

* Change body index map image to mono8 format

* Fixed small syntax error

* Added topic description in usage.md

* Fixed typos in usage.md
allnes pushed a commit to allnes/opencv_contrib that referenced this pull request Jun 9, 2024
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.

5 participants