Skip to content

Replace _t_complexity_ with build_call_graph in comparison gates #1012

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

Merged

Conversation

tanujkhattar
Copy link
Collaborator

@tanujkhattar tanujkhattar commented May 29, 2024

Fixes #1010

Helps support symbolics and better resource counting for comparison gates.

Edit: Also updated LessThanEqual gate.

@tanujkhattar tanujkhattar changed the title Replace _t_complexity_ with build_call_graph in comparision gates Replace _t_complexity_ with build_call_graph in comparison gates May 29, 2024
@NoureldinYosri
Copy link
Contributor

@tanujkhattar this PR becomes obsolete once I merge my upcoming PR that reimplements all comparison gates + adds missing comparison gates. I will send the PR today.

@tanujkhattar
Copy link
Collaborator Author

I think this is a fairly straightforward change, so lets merge this and then I will lookout for your PR with the revamp? In case we are not able to merge your PR in time for the release; this would be a good temporary solution to unblock other upgrades.

@tanujkhattar tanujkhattar enabled auto-merge (squash) May 29, 2024 18:08
@tanujkhattar tanujkhattar merged commit e092d87 into quantumlib:main May 29, 2024
7 checks passed
@mpharrigan
Copy link
Collaborator

this broke the leq_symb bloq example, which tragically isn't included in a unit test

@tanujkhattar
Copy link
Collaborator Author

Yeah, I'll send a fix soon. Realized it when implementing symbolic state prep

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.

SingleQubitCompare has wrong T-complexity annotation?
4 participants