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.qiskit-terra: init at 0.12.0 #80464
pythonPackages.qiskit-terra: init at 0.12.0 #80464
Conversation
0144593
to
29269ec
Compare
29269ec
to
07ee8d1
Compare
@GrahamcOfBorg build python37Packages.qiskit-terra python38Packages.qiskit-terra |
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.
otherwise LGTM
07ee8d1
to
d1b9cbf
Compare
@GrahamcOfBorg build python37Packages.qiskit-terra python38Packages.qiskit-terra |
d1b9cbf
to
ea81c30
Compare
@GrahamcOfBorg build python37Packages.qiskit-terra python38Packages.qiskit-terra |
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.
I guess the tests aren't too damaging to the huge size of the closure already:
[nix-shell:/home/jon/.cache/nixpkgs-review/pr-80464-1]$ nix path-info -Sh ./results/python37Packages.qiskit-terra
/nix/store/rrwx7l0gq49cc9zb3y49zkx8vy7j4y5m-python3.7-qiskit-terra-0.12.0 388.2M
[nix-shell:/home/jon/.cache/nixpkgs-review/pr-80464-1]$ du -hd0 ./results/python37Packages.qiskit-terra/lib/python3.7/site-packages/test/
4.5M ./results/python37Packages.qiskit-terra/lib/python3.7/site-packages/test/
[nix-shell:/home/jon/.cache/nixpkgs-review/pr-80464-1]$ du -hd0 ./results/python37Packages.qiskit-terra/lib/python3.7/site-packages/examples/
96K ./results/python37Packages.qiskit-terra/lib/python3.7/site-packages/examples/
@GrahamcOfBorg build python37Packages.qiskit-terra python38Packages.qiskit-terra |
Qiskit is a quantum information toolkit from IBM. This commit follows the new Qiskit scheme of breaking from one large package into smaller packages (terra, aer, etc), and then having a single meta-package "qiskit" that comprises them.
ea81c30
to
d3cdcb6
Compare
@GrahamcOfBorg build python37Packages.qiskit-terra python38Packages.qiskit-terra |
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.
diff LGTM
commit LGTM
[3 built, 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/80464
2 package built:
python37Packages.qiskit-terra python38Packages.qiskit-terra
@GrahamcOfBorg build python37Packages.qiskit-terra python38Packages.qiskit-terra |
Motivation for this change
Update qiskit according to new upstream package structure (collection of subpackages). This is broken out of #78772. Would like to get it into 20.03 if at all possible. Has one out-of-tree dependency: pythonPackages.pylatexenc (#78878). Will rebase & remove that commit when #78878 is approved.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)