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.

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

Loosen cirq_rigetti pyquil dependency to pyquil-3.* #6226

Merged
merged 1 commit into from Aug 1, 2023

Conversation

ryanhill1
Copy link
Contributor

From: PR Pin cirq_rigetti pyquil dependency to pyquil-3.* #6129:

Problem: pip install cirq --pre may install pyquil-4.0.0rcN which is
incompatible with cirq_rigetti.

Solution: pin the pyquil dependency to "3.*".

but changed pyquil dependency from pyquil>=3.2.0 $\rightarrow$ pyquil~=3.2.0

24abfa1

As of Aug 1 pyquil is on version 3.5.4. I propose that same solution can be achieved with less restrictive pyquil dependency by adding upper bound for version 4.

Loosen pyquil dependency
@ryanhill1 ryanhill1 requested review from vtomole, cduck and a team as code owners August 1, 2023 14:07
@ryanhill1 ryanhill1 requested a review from dabacon August 1, 2023 14:07
@CirqBot CirqBot added the Size: XS <10 lines changed label Aug 1, 2023
Copy link
Collaborator

@vtomole vtomole left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense

@vtomole vtomole enabled auto-merge (squash) August 1, 2023 15:17
@vtomole vtomole merged commit c510fff into quantumlib:master Aug 1, 2023
32 of 33 checks passed
@ryanhill1 ryanhill1 deleted the ryanhill1-pyquil-dependency branch August 1, 2023 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Size: XS <10 lines changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants