-
Notifications
You must be signed in to change notification settings - Fork 105
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
Pybind11 #264
Pybind11 #264
Conversation
May need to revert this, did you verify it builds with MSVC and Clang? |
Well I be damned, missed the branch switcharoo, will check today, but please try Clang. |
How am I supposed to do that?
I'll try it |
Could run a virtual machine I suppose. I thought it was merged since you closed, but it was the other one that was closed, got me confused. |
I didn't actively close anything, maybe something like that happened when I deleted the other branch? Sorry about the confusion. I don't have a win installation lying around. We should activate our the Jenkins on the windows machine again... Now installing CUDA 7.5, seems to have hit the repos recently. My old 7.0 doesn't support Clang. |
Works with clang. A build job for odlpp with clang wouldn't hurt I guess. |
This will have to wait, it seems pybind11 has introduced heaps of bugs for VS2013, I tried fixing some but more popped up, and apparently the maintainer has no plans to support "only parts of" C++11 (which I can understand). This would have been fine if Nvidia had released VS2015 support for CUDA, but it is currently only part of a "release candidate" that you can access as a "registered developer" (which I am, but I doubt most of our users are). Status: I'll have a look and see if it builds under VS2015 anyway. |
Well okay. Not great but understandable. When will this piece of crap On 02/09/2016 09:07 AM, Jonas Adler wrote:
Dr Holger Kohr |
Closing this since #336 will override most of this anyway. |
Pushed to a new branch by mistake - never mind. This seems to work according to tests on my machine and on our Jenkins. Can you test it on Windows @adler-j? If that goes through, I suggest we merge in odlpp to master, update the submodule and merge here.