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

fix: The DefGate.matrix property will no longer raise an exception when the matrix contains a mix of atomic and object types. #1685

Conversation

MarquessV
Copy link
Contributor

Description

Closes #1681. Thanks to @steve-jeffrey for the report and suggested fix!

Checklist

  • The PR targets the master branch
  • The above description motivates these changes.
  • The change is atomic and can be described by a single commit (your PR will be squashed on merge).
  • All changes to code are covered via unit tests.
  • (Bugfix) The associated issue is referenced above using auto-close keywords.

…when the matrix contains a mix of atomic and object types.
@MarquessV MarquessV requested a review from a team as a code owner October 27, 2023 18:04
@rigetti-githubbot
Copy link

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
6874 6018 88% 87% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
pyquil/quilbase.py 93% 🟢
TOTAL 93% 🟢

updated for commit: 8c5ab11 by action🐍

pyquil/quilbase.py Show resolved Hide resolved
@MarquessV MarquessV enabled auto-merge (squash) October 31, 2023 16:40
@MarquessV MarquessV disabled auto-merge October 31, 2023 16:41
@MarquessV MarquessV merged commit d57cce9 into master Oct 31, 2023
18 checks passed
@MarquessV MarquessV deleted the 1681-pyquil-v4-defgate-matrix-property-fails-if-unitary-has-mixed-types branch October 31, 2023 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

pyQuil v4: DefGate "matrix" property fails if unitary has mixed types
3 participants