-
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
Upgrade XGBoost to use BB on all platforms #68
Conversation
I've switched this to WIP, because in my testing, Windows doesn't work yet, and |
So the problem with Windows is that the binaries immediately crash Julia upon
|
Alright, I'm not certain why I thought |
Thanks Elliot, this is exciting. Do you know if these binaries are built with openmp? |
e8b6f7f
to
ac236a7
Compare
All except for MacOS are. This is because on MacOS, we are using |
eminently. |
This upgrades the package (and a few stylistic points within the package itself) to use prebuilt binaries instead of requiring the user to compile on their own machine.