Remove _t_complexity_ overrides where build_call_graph exists.
#905
Assignees
Labels
Comments
dstrain115
added a commit
to dstrain115/Qualtran
that referenced
this issue
May 24, 2024
- Part of quantumlib#905. - Removed _t_complexity_ calculation and moved to assert since this has a call_graph, the T count can be inferred.
dstrain115
added a commit
that referenced
this issue
May 28, 2024
- Part of #905. - Removed _t_complexity_ calculation and moved to assert since this has a call_graph, the T count can be inferred.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The t_complexity protocol can quickly use the call graph protocol to derive accurate t counts from subbloqs. This handles meta-bloqs like
Adjointcorrectly.For bloqs where
build_call_graphis implemented and correct -- the_t_complexity_override should be removed. If "leaf values" are provided by the_t_complexity_override, they should be moved to a unit test.This is of extreme importance for the places where the
_t_complexity_(self, adjoint: bool)method has been hacked to accept anadjointparameter.The text was updated successfully, but these errors were encountered: