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

[QECGatesCost] Adjustments to match t_complexity rotations, part 2 #1323

Merged
merged 9 commits into from
Aug 21, 2024

Conversation

mpharrigan
Copy link
Collaborator

Following #1313 -- some more changes to how we count.

  • there's a QVR stub. The permanent fix would be to use qvr, but removing the erroneous t complexity is a start
  • LPRSInterimPrep has a bad t_complexity annotation, based on the current convention for counting CRz as one rotation
  • states and effects can be counted (these would throw an error in t_complexity)
  • drive by fixes to some str representations so it's clearer what the bloq is

@mpharrigan mpharrigan merged commit bd4d01e into quantumlib:main Aug 21, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants