-
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 docs on building and using precompiled binaries for the R package #3285
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Very clear guide!
Just dots for the consistency and one comment about the name of Windows artifact.
During reading this comment #3187 (comment) I came to the thought that it might be useful to upload the source distribution to Releases page (automatically after resolving #3283 ) which will be then submitted to CRAN. WDYT @jameslamb ? |
Co-authored-by: Nikita Titov <[email protected]>
yes definitely! I think that is a great idea and something we should be doing. I'll add it and a note in these docs. And then we can add it automatically in future releases (#3283 ) |
Co-authored-by: Nikita Titov <[email protected]>
I just added a source distribution (built with I also added it to the list of artifacts that should be produced by #3283 It can be installed like this from R. remotes::install_url(
"https://github.com/microsoft/LightGBM/releases/download/v3.0.0rc1/lightgbm-3.0.0-1-cran.tar.gz"
) |
I just added notes on installing the CRAN package from the releases page, thanks for the nudge @StrikerRUS ! |
Thanks a lot! I think after we put LightGBM on CRAN, we will have to produce only that artifact with source files and upload it to CRAN. All other artifacts with binaries should be uploaded reversely: from CRAN to Releases page (if needed) to have identical versions in both places and avoid errors like one just reported about glibc (#3188 (comment)). Please correct me if I'm wrong in my thoughts about further release pipeline. |
Yes I agree with this. By the way, CRAN does not provide binaries for Linux (only Mac and Windows), so I think ours should be considered experimental. |
Hmm, then we should build them with our Docker we use for artifacts at Azure Pipelines to ensure maximum compatibility. Or even better with CentOS image. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now all artifact names look consistent here and at release page.
Co-authored-by: Nikita Titov <[email protected]>
thanks for keeping me consistent @StrikerRUS , sorry for all the errors. |
This pull request has been automatically locked since there has not been any recent activity since it was closed. To start a new related discussion, open a new issue at https://github.com/microsoft/LightGBM/issues including a reference to this. |
This PR comes out of this discussion: #3188 (comment).
We're really close to getting the R package to CRAN, but I'm not sure how long it will take to fix #3187 . To make life easier for users until the package gets to CRAN, starting with https://github.com/microsoft/LightGBM/releases/tag/v3.0.0rc1 I think we should start providing precompiled binaries for releases.
When #3283 is done, these binaries can be produced automatically and attached to the releases page. For now, I think it will be useful to document how to use them (for users) and how to build them (for maintainers and adventurous users).