Skip to content
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.

Sign up for GitHub

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

Jump to bottom

installation fails on python 3.13 #6932

Open
richrines1 opened this issue Jan 9, 2025 · 4 comments
Open

installation fails on python 3.13 #6932

richrines1 opened this issue Jan 9, 2025 · 4 comments
Labels
kind/bug-report Something doesn't seem to work. triage/accepted A consensus emerged that this bug report, feature request, or other action should be worked on

Comments

@richrines1
Copy link
Contributor

richrines1 commented Jan 9, 2025 •
edited
Loading

Description of the issue

both cirq_google (dev version only) and cirq_rigetti (both dev and production versions) are currently failing to install on python 3.13

How to reproduce the issue

all of the following fail in a python 3.13 env:

pip install cirq_google==1.5.0.dev20250107193327  # ERROR: No matching distribution found for typedunits
pip install cirq_google==1.5.0.dev20250107193327  # ....metadata-generation-failed
pip install cirq_rigetti==1.4.1  # ....metadata-generation-failed

Cirq version

1.5.0.dev20250107193327 (both cirq_google and cirq_rigetti)
1.4.1 (cirq_rigetti only)
@richrines1 richrines1 added the kind/bug-report Something doesn't seem to work. label Jan 9, 2025
@maffoo maffoo mentioned this issue Jan 9, 2025
Closed
@maffoo
Copy link
Contributor

maffoo commented Jan 9, 2025

I filed an issue for typedunits: quantumlib/TypedUnits#26. Not sure what needs to be done for cirq_rigetti.

@NoureldinYosri NoureldinYosri added triage/discuss Needs decision / discussion, bring these up during Cirq Cynque triage/accepted A consensus emerged that this bug report, feature request, or other action should be worked on and removed triage/discuss Needs decision / discussion, bring these up during Cirq Cynque labels Jan 22, 2025
@pavoljuhas
Copy link
Collaborator

@pavoljuhas - check why cirq-rigetti fails and follow up with Rigetti contributors.

@richrines1
Copy link
Contributor Author

why cirq-rigetti fails

fwiw i think this is just because qcs-sdk-python doesn't have a python 3.13 build either (and possibly my local system not being configured correctly to compile the rust components from source)

This was referenced Jan 24, 2025
Open
Merged
pavoljuhas added a commit to pavoljuhas/Cirq that referenced this issue Jan 24, 2025
@pavoljuhas
CI - run the pytest-numpy-2 job also with Python 3.13
2374d41 
This CI job excludes cirq-rigetti which is not yet compatible with
Python 3.13.  This activates Python 3.13 testing for other Cirq packages.

Related to quantumlib#6932
github-merge-queue bot pushed a commit that referenced this issue Jan 28, 2025
@pavoljuhas
CI - run the pytest-numpy-2 job also with Python 3.13 (#6974)
ab62c25 
* CI - run the pytest-numpy-2 job also with Python 3.13

This CI job excludes cirq-rigetti which is not yet compatible with
Python 3.13.  This activates Python 3.13 testing for other Cirq packages.

Related to #6932

* Install only dependencies needed for running pytest

* Adjust mocking of EngineClient.get_processor for Python 3.13

In Python 3.13 `Mock(EngineClient)` creates an AsyncMock for
`EngineClient.get_processor()`.  Here we adjust the class
mocking so it works for both Python 3.12 and 3.13.
@pavoljuhas
Copy link
Collaborator

pavoljuhas commented Jan 29, 2025 •
edited
Loading

The ETA for cirq-rigetti dependencies is around the end of February - rigetti/pyquil#1816 (comment)

cross-ref: b/393145662

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug-report Something doesn't seem to work. triage/accepted A consensus emerged that this bug report, feature request, or other action should be worked on
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@maffoo @pavoljuhas @NoureldinYosri @richrines1