-
-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
pythonPackages.imbalanced-learn: 0.3.2 -> 0.3.3, fix build #38245
Conversation
@GrahamcOfBorg build python27Packages.imbalanced-learn python36Packages.imbalanced-learn |
Success on x86_64-linux (full log) Attempted: python27Packages.imbalanced-learn, python36Packages.imbalanced-learn Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: python27Packages.imbalanced-learn, python36Packages.imbalanced-learn Partial log (click to expand)
|
{ stdenv, buildPythonPackage, fetchPypi, scikitlearn, pandas, nose, pytest }: | ||
|
||
buildPythonPackage rec { | ||
|
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.
unnecessary whitespace
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.
Removed.
''; | ||
|
||
propagatedBuildInputs = [ scikitlearn ]; | ||
buildInputs = [ pandas ]; |
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.
Do you know if this is an optional dependency? Why is it in buildInputs
?
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.
Looking at the source code, it is optional and only used in a test utility. Without it, a number of tests are skipped. Moved to checkInputs
.
}; | ||
|
||
preConfigure = '' | ||
export HOME=$PWD |
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.
Is this needed for anything other than the tests? If not, it belongs into checkPhase
.
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.
Tests only. moved to checkPhase.
minor bump, fix tests, move to development/python-modules
f789a4d
to
6b2bc4b
Compare
@GrahamcOfBorg build python27Packages.imbalanced-learn python36Packages.imbalanced-learn |
Success on x86_64-linux (full log) Attempted: python27Packages.imbalanced-learn, python36Packages.imbalanced-learn Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: python27Packages.imbalanced-learn, python36Packages.imbalanced-learn Partial log (click to expand)
|
Backported in da36dff. |
Motivation for this change
ZHF #36453 . Build (tests) failed.
minor bump, fix tests, move to development/python-modules
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)