-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
[R-package] Add GPU support for CRAN package #3206
Comments
This has been added to #2302 . Leave a comment here or on that issue if you'd like to contribute this feature! |
I wonder, how will it be useful for users given that they have to install Boost and OpenCL by their own anyway? The only difference will be "CMake vs configure", if I'm not mistaken. Maybe it's better to provide a link for the guide how build from the most recent sources? |
I don't think it is something we have to do before going to CRAN, but it would be useful some day in the same way that Python users can do this pip install lightgbm --install-option=--gpu
I agree that it's ok to just have it documented in the README how to build for source, maybe for a long time, but I do think this could be useful one day. |
@jameslamb Would it make sense to add the same feature request for CUDA build? If you are OK, I can create an issue. |
yes definitely! I think @Laurae2 already did a proof of concept of "R package with CUDA", but dont remember where it was posted. Either way yes, definitely |
Created #3465.
Here it is https://gist.github.com/Laurae2/7195cebe65887907a06e9118a3ec7f96 |
oh perfect, thank you! |
Summary
#3188 added an installation strategy for the R package that is compliant with CRAN's restrictions. I originally added support for using a flag to build a GPU-enabled version from that CRAN package, but decided to pull it out of that pull request. I think it deserves more careful thought and testing.
Description
The goal of this issue is to be able to build the CRAN package for the GPU, like this:
sh build-cran-package.sh R CMD INSTALL \ --configure-args='--enable-gpu' \ lightgbm_2.3.2.tar.gz
The following code was removed from that PR, and adding it back would be a good start:
R-package/README.md
R-package/configure.ac
configure.win
The text was updated successfully, but these errors were encountered: