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.nimfa: init at 1.3.1 #33918
Conversation
}: | ||
|
||
buildPythonPackage rec { | ||
name = "${pname}-${version}"; |
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.
no name
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.
Python packages shouldn't have a name
?
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.
buildPythonPackage
will automatically set the name according to pname
and version
.
}; | ||
|
||
propagatedBuildInputs = [ numpy scipy ]; | ||
buildInputs = [ matplotlib pytest ]; |
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.
checkInputs
|
||
propagatedBuildInputs = [ numpy scipy ]; | ||
buildInputs = [ matplotlib pytest ]; | ||
doCheck = false; # errors |
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.
What kind of errors? Are these upstream issues? If so, did you report them?
I believe all changes have been made |
Motivation for this change
New Python package for nonnegative matrix factorization (maths and science).
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)