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

[opencv] enabling cublas #3635

Merged
merged 3 commits into from
Feb 27, 2019
Merged

[opencv] enabling cublas #3635

merged 3 commits into from
Feb 27, 2019

Conversation

ahmadhasan2k8
Copy link
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

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
Contributor Author

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

@carmangary
Copy link

Did you test on x86?

@ahmadhasan2k8
Copy link
Contributor Author

I did not.

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

msftclas commented Feb 26, 2019

CLA assistant check
All CLA requirements met.

@vicroms
Copy link
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
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