-
Notifications
You must be signed in to change notification settings - Fork 0
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
GPU on Windows #2
Comments
Here you go. I'm doing this with CUDA 10.2, and the
I get a compilation error. Here is the output of
Here's my normal
I'm not sure what other debug info would be useful. Let me know. |
Thanks! It looks like it's not getting the correct C++14 flag. WRE says this
I would have assumed that Rtools would be the limiting factor though, not the R version. And you have the latest Rtools. Could you tell me the directory contents of |
It's potentially related to that, because this happens after I apply the fix listed here: https://community.rstudio.com/t/error-in-shlib-internal-args-c-14-standard-requested-but-cxx14-is-not-defined/16819 If not for that, I get the error:
Not much going on. |
Thanks. Linking with NVML on Windows may prove interesting. Hopefully it's in lib/${ARCH} somewhere. I've updated Makevars.win to something that's got to be closer to the right thing though. My guess is that your R version doesn't support C++14 but your Rtools does, so changing your |
Hah! I get a different error after upgrading to R 4.0.0 (shows how much I use Windows). Looks like it's some weird thing involving the fact that my user name has a space and a lack of string quoting? Sorry, this may be stuff related to general weird Windows Things that I just don't get -- I don't use it much.
|
Interesting! Looks like that's the issue. Let me add quotes to everything... |
I think that should do it... |
The quotes might have fixed it, but your prior commit seems to have broken something with the if statement on the architecture -- I get this error both on
|
Ok, I think I have the arch selector right now based on a re-reading of WRE... Thanks so much again for all the help so far! |
Well, it does seem to compile now, but it doesn't seem to compile with GPU support? Maybe the flag is ignored for some reason? When running your sample code, I get the error:
Attached is my
|
Weird. Would you mind giving me the compiler output from the Edit: Nevermind. I think I see the issue here. There's a subtlety with quotes that I don't think behaves this way in GNU make on linux. I'll push a fix this evening. |
I get this error now when doing
|
Did you clone with
The |
Ugh, I'm a doofus, I should have realized that. Sorry. I get a different error now:
|
No worries. I really appreciate your patience working with me so far. We are finally at the CUDA problems! Sorry this took so much of your time so far. I really thought the makefile was in good enough shape that we would start here... Well apparently (and I really should have seen this coming) Windows CUDA only supports Microsoft's VC++. I have no idea if installing that and setting the compiler to |
You're doing God's work by trying to grapple with any Windows-related compiler issues, so this is the least I can do. Let me know how I can help. |
I do not have the ability to test this, but I think the package can now be built with GPU backend support on Windows. I would appreciate any input/feedback on this.
If you are willing to test, you can download the development version of the package with git (there are submodules so
install_github()
-like things won't work):Modify the first 2 lines of src/Makevars.win. The first should be set to
USE_GPU = "TRUE"
. The second should be set to whatever your CUDA path is. The contents should beinclude
,lib
, etc. Actually, just giving me an ls of that path would be very helpful.Install the package dependencies as needed:
Next, try to install the package:
Here's a basic test you can run to see if it works:
Here's an example of the output I see on my Linux box:
The text was updated successfully, but these errors were encountered: