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
pythonPackage.datatable: 0.8.0 -> 0.9.0 #68716
Conversation
018341f
to
beae056
Compare
I briefly started looking into this since it still shows up as one of the "top" failures for 19.09 and can confirm that the version upgade solves the cause of the failure. I also gave the build a run on nixos and it works for me. I can't say much for the removing of the compiler monkeypatching for darwin though. We should bump this PR. |
@GrahamcOfBorg build python37Packages.dask-xgboost python37Packages.datatable python37Packages.optuna python37Packages.xgboost i rebased off current master, and did nix-review again
|
@GrahamcOfBorg build python37Packages.dask-xgboost python37Packages.datatable python37Packages.optuna python37Packages.xgboost |
Backported as 3f25baa. |
Thanks for taking care of this one! |
Was this tested for darwin? I had to make some changes ( korayal@9580b8d ) to make this work on Mac OS X 10.14.6 |
Tested? no. I just run NixOS |
@korayal would you make a PR for this? I just managed to get this compiling on macOS Mojave
|
@the-alex glad that it worked for you. Unfortunately, because of the current situation I do not have the means to maintain MacOS support for this code. So I'm OK with you making a PR. |
Motivation for this change
#68361
bumped version while fixing build
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @
optuna is broken by lightgbm which is addressed here: #68719