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
python: qiskit: 0.6.1 -> 0.7.3 #56078
Conversation
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 rebase on python-unstable
and target that branch.
cc @FRidh
pkgs/development/python-modules/marshmallow-polyfield/default.nix
Outdated
Show resolved
Hide resolved
}; | ||
|
||
patches = [ | ||
./marshmallow-polyfield.patch |
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 make an upstream PR with this patch and add a comment with the link.
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 adopted an appropriate version of marshmallow_polyfield
so this patch is not needed anymore.
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.
You should still open an upstream PR because our next automatic upgrade will break qiskit-terra otherwise.
It seems that |
Thank you for the review! I rebased on
I'm trying to figure out what's wrong. |
Now it has. However, we have too many build failures over there. Hence, let's not care about the qiskit bump which is already there and target master. I'm sorry :/
During the build of which package are you getting the error? You can use |
Only |
Rebased onto master. It works now. |
@GrahamcOfBorg build python3.pkgs.qiskit python2.pkgs.marshmallow-polyfield |
scikit-build seems to be failing again :( |
It looks like we've gotten up to qiskit 0.18.2, so this PR can probably be closed. |
Closed. Thanks! |
Motivation for this change
Upgrade Qiskit.
Things done
I chose
staging
as the base because the latest version of Qiskit needsscikit-build
, but I couldn't build one of the transitive dependency,libxml2Python
, with the following error:When I rebased onto master and cherry-picked
scikit-build
and sysconfig fix, it worked.sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)