-
Notifications
You must be signed in to change notification settings - Fork 110
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
Package now requires a Julia built with GPL libraries enabled #157
Comments
i also opened gridap/SparseMatricesCSR.jl#21, and with that SparseMatricesCSR branch the XGBoost.jl tests pass locally on a Julia version built with |
I'm not in favor of re-implementing |
...due to the dependency on SparseMatricesCSR.jl (added in #136)
We're running XGBoost.jl in production but have had to fix the XGBoost.jl version to v2.0.2, as the newer versions depend on SparseMatricesCSR.jl, which requires a Julia built with GPL libraries enabled (at Julia build time), causing the following error on our Julia build:
I wonder if the SparseMatricesCSR.jl dependency is necessary and e.g. if there might be an alternative way to achieve the functionality added in #136 without taking on this dependency requiring GPL libraries?
The text was updated successfully, but these errors were encountered: