Skip to content

[opencv] enabling cublas#3635

Merged
vicroms merged 3 commits intomicrosoft:masterfrom
ahmadhasan2k8:master
Feb 27, 2019
Merged

[opencv] enabling cublas#3635
vicroms merged 3 commits intomicrosoft:masterfrom
ahmadhasan2k8:master

Conversation

@ahmadhasan2k8
Copy link
Copy Markdown
Contributor

If I don't enable cublas several cuda headers do not get installed such as, cudaarithm.hpp, cudabgseg.hpp, cudacodec.hpp, cudafeatures2d.hpp.

When trying to use cv::cuda::ORB I need the header cudafeatures2d.hpp but the header was not found. After enabling cublas the headers were installed.

I tested on Windows 10 with visual studio 2017 and it works.

Thanks,
Ahmad

@carmangary
Copy link
Copy Markdown

I don't think cublas is currently supported on x86. But, the x86 opencv[cuda] port doesn't work either, so maybe it doesn't matter.

@ahmadhasan2k8
Copy link
Copy Markdown
Contributor Author

I tested it on x64 and it works. If cuda is enabled why not cublas?

@carmangary
Copy link
Copy Markdown

Did you test on x86?

@ahmadhasan2k8
Copy link
Copy Markdown
Contributor Author

I did not.

@vicroms vicroms self-assigned this Feb 26, 2019
@msftclas
Copy link
Copy Markdown

msftclas commented Feb 26, 2019

CLA assistant check
All CLA requirements met.

@vicroms
Copy link
Copy Markdown
Member

vicroms commented Feb 27, 2019

Hi @ahmadhasan2k8,
Thanks for your contribution!
This PR passes the CI tests, so I will go ahead and merge it.

@vicroms vicroms merged commit a5e030f into microsoft:master Feb 27, 2019
@ahmadhasan2k8
Copy link
Copy Markdown
Contributor Author

Thanks

@cenit cenit mentioned this pull request Mar 7, 2019
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