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.mnist: init at 0.2.2 #56518
Conversation
@GrahamcOfBorg build python2.pkgs.mnist python3.pkgs.mnist |
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.
Please add yourself to maintainer-list.nix
in a separate commit and change the commit message of the existing one to the PR's title.
Ran 0 tests in 0.000s
Please provide an appropriate checkPhase
or, if there are no tests, set doCheck = false
and add a comment with the reason.
Finally, please make the python2 build work.
2077059
to
2eb21c1
Compare
@dmrauh are you able to address the comments above and continue with this PR? |
Please excuse my long hiatus. I hope that i have addressed all comments. The only problem i had was that some tests using sockets during check phase were failing which i had to disable in the end. |
f8e1299
to
cdca919
Compare
@dotlambda @aanderse do you think my PR can be merged or is there anything else to do? |
@dmrauh I don't see any issues. Thanks for following up. @dotlambda good to go from your perspective? |
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.
nix-review
passes on NixOS
diff LGTM
commits LGTM
[3 built, 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/56518
3 package were built:
python27Packages.mnist python37Packages.mnist python38Packages.mnist
Thanks to all reviewers for your help! |
... it did take 8 months... that's quite a while. what's impressive to me, is that this means we've had 17k PRs in 8 months, which is averaging like 2.1k a month O.o |
Motivation for this change
Add python mnist utilities.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)