Skip to content

Conversation

pavoljuhas
Copy link
Collaborator

Hopefully it is more informative as to why decomposition failed.
Also correct a typo in the if condition.

Resolves #6667

Hopefully it is more informative as to why decomposition failed.
Also correct a typo in the if condition.

Resolves quantumlib#6667
@pavoljuhas pavoljuhas requested review from a team and vtomole as code owners April 17, 2025 21:52
Copy link

codecov bot commented Apr 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.66%. Comparing base (c054b90) to head (f3b427e).
Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7281   +/-   ##
=======================================
  Coverage   98.66%   98.66%           
=======================================
  Files        1106     1106           
  Lines       96086    96086           
=======================================
  Hits        94808    94808           
  Misses       1278     1278           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@mhucka mhucka left a comment

Choose a reason for hiding this comment

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

LGTM

@pavoljuhas pavoljuhas added this pull request to the merge queue Apr 28, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 28, 2025
@pavoljuhas pavoljuhas added this pull request to the merge queue Apr 28, 2025
Merged via the queue into quantumlib:main with commit 5f7fc9a Apr 28, 2025
39 checks passed
@pavoljuhas pavoljuhas deleted the improve-decompose_once-error-message branch April 28, 2025 20:12
BichengYing pushed a commit to BichengYing/Cirq that referenced this pull request Jun 20, 2025
Hopefully it is more informative as to why decomposition failed.
Also correct a typo in the if condition.

Resolves quantumlib#6667
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.

cirq.decompose_once fails when called on cirq.CZ

2 participants