-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Unpin dependencies' versions #6464
Comments
I feel it depends case to case. Expressions like For the requirement specifications under |
I apologize if this is off the mark, but my $0.02 based just on general principles is that I agree with Pavol. For non-dev requirements, not using a version requirement or setting only a minimum version (using So, in the end, it seems like specifying the requirements as, e.g., |
- allow latest scipy because quimb>=1.18 works with scipy-1.13 (context in #6543) - allow latest rstcheck - fix dependency specification for tensorflow-docs so it works with `uv pip compile` - Enable the current latest versions of notebook, ipykernel, papermill which are only used for notebook tests - Add temporary specifications for notebook-related transitive dependencies to help pip dependency resolver (should be redundant after #6281) Partially fixes #6464
Partially implemented in #6616 - sufficient for the 1.4 release. Reopening to track the leftover dependencies. |
- Add support for Kraus operators, POVMs and parametric defgates - Update quil->cirq conversion Fixes quantumlib#6500 Partially implements quantumlib#6464
- Add support for Kraus operators, POVMs and parametric defgates - Update quil->cirq conversion Fixes quantumlib#6500 Partially implements quantumlib#6464
- Add support for Kraus operators, POVMs and parametric defgates - Update quil->cirq conversion Fixes quantumlib#6500 Partially implements quantumlib#6464
- Add support for Kraus operators, POVMs and parametric defgates - Update quil->cirq conversion Fixes quantumlib#6500 Partially implements quantumlib#6464
- Add support for Kraus operators, POVMs and parametric defgates - Update quil->cirq conversion Fixes quantumlib#6500 Partially implements quantumlib#6464
- allow latest scipy because quimb>=1.18 works with scipy-1.13 (context in quantumlib#6543) - allow latest rstcheck - fix dependency specification for tensorflow-docs so it works with `uv pip compile` - Enable the current latest versions of notebook, ipykernel, papermill which are only used for notebook tests - Add temporary specifications for notebook-related transitive dependencies to help pip dependency resolver (should be redundant after quantumlib#6281) Partially fixes quantumlib#6464
Description of the issue
We have several dependencies pinned at specific old version because newer version break the CI. Ideally we should either not have a version requirement of have version requirement beyond a specific release (e.g.
>=
or>
). specifying a version requirement with==
,~=
,<=
,<
shouldn't be used.The list of requirements that should be unpinned is
in
cirq-core/requirements.txt
:(pavoljuhas) leave as is, this pins the major version and includes the latest matplotlib-3.9.0
done in NEP-29 - enforce minimum Python version 3.10 #6591
(pavoljuhas) leave as is, pins the major version and includes the latest sortedcontainers-2.4.0
in
cirq-core/cirq/contrib/requirements.txt
:done in Support Python 3.12 #6516, includes the latest pylatex-1.4.2
done in Support Python 3.12 #6516, includes the latest quimb-1.8.1
deleted in Support Python 3.12 #6516, transitive dependency of quimb
in
dev_tools/pr_monitor/requirements.txt
:(pavoljuhas) - only used in containerized tool, defer after 1.4
in
dev_tools/requirements/deps/protos.txt
:done in Support Python 3.12 #6516, first version compatible with Python 3.12
(pavoljuhas) leave as is, works with the grpcio-tools
in
dev_tools/requirements/deps/format.txt
:done in Bump black from 23.3.0 to 24.3.0 in /dev_tools/requirements/deps #6512
in
dev_tools/requirements/deps/dev-tools.txt
:done in Support Python 3.12 #6516, 0.12.* is compatible with OpenQASM 2.0. 0.12.0 installs with Python 3.12
done in Freshen-up cirq dependencies #6616
in
dev_tools/requirements/deps/pylint.txt
:in
dev_tools/requirements/deps/mypy.txt
:(pavoljuhas) - defered to #6617
in
dev_tools/requirements/deps/notebook.txt
:done in Freshen-up cirq dependencies #6616, includes latest notebook-7.2.0
done in Freshen-up cirq dependencies #6616, includes latest ipykernel-6.29.4
done in Freshen-up cirq dependencies #6616, includes latest papermill-2.6.0
done in CI - deflake
Isolated pytest Ubuntu
#6603, includes latest seaborn-0.13.2in
dev_tools/requirements/deps/pytest.txt
:done in Release pins from all pytest-related packages #6602
done in Release pins from all pytest-related packages #6602, has restored
annotate
, includes the latest coverage-7.5.3done in Release pins from all pytest-related packages #6602
done in Support Python 3.12 #6516, includes the latest filelock-3.14.0
done in Release pins from all pytest-related packages #6602
Optionally these verndor related requirements should be unpinned
in
cirq-aqt/requirements.txt
:(pavoljuhas) leave as is, includes the latest requests-2.32.2
in
cirq-pasqal/requirements.txt
:(pavoljuhas) leave as is, includes the latest requests-2.32.2
in
cirq-ionq/requirements.txt
:(pavoljuhas) leave as is, includes the latest requests-2.32.2
in
cirq-rigetti/requirements.txt
:(pavoljuhas) - wip in update(cirq-rigetti): use pyquil v4 #6281
after which merge the minimum attrs version in Add
attrs
lower bound in requirements.txt #6633The text was updated successfully, but these errors were encountered: