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

make it visual studio compliant #2

Open
ahmidou opened this issue Apr 9, 2020 · 2 comments
Open

make it visual studio compliant #2

ahmidou opened this issue Apr 9, 2020 · 2 comments

Comments

@ahmidou
Copy link

ahmidou commented Apr 9, 2020

Could we have the option to build on windows visual studio?
I'm getting errors with Eigen related to the = operator:

D:/externalLibs/viper/Viper/CudaSolver.cu(137): error : function "Eigen::Quaternion<_Scalar, _Options>::operator=(const Eigen::Quaternion<float, 0> &) [with _Scalar=float, _Options=0]" (declared implicitly) cannot be referenced -- it is a deleted function

@drebain
Copy link
Collaborator

drebain commented Apr 30, 2020

There are also some additional things that would need to be changed in Subprocess.h that are currently Unix-specific to implement Windows support. We're not currently developing this, but if someone has modified it to be cross-platform, I'd be happy to accept a PR.

@besoft
Copy link

besoft commented Oct 8, 2020

Compilation error related to the Eigen = operator is caused by the fact that CudaSolver.cu is compiled by NVCC compiler instead of MSVC compiler. As _MSC_VER > 1900 is defined at the same time, the definition of operator = valid for Visual Studio 2019+ is used. Unfortunately, this definition is unsupported by NVCC.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants