Fionn Malone

Ok, I guess walk_operator isn't serializable right now, I updated the set of bloqs to skip. Just the control error now.

It's odd because Prepare (which it's complaining about) does correctly serialize without issue when it's not apart of the walk operator, I noticed that the example walk_op was being skipped...


@tanujkhattar PTAL, I fixed a few bugs a long the way: 1. There was an extra comparator which shouldn't have been there. Prepare^ will uncompute this. 2. There was a...

That was weird, them mypy check failed but it automerged? @mpharrigan

bloq_counts mypy errors too: See for example and_bloq.py: Return type "Set[Tuple[int, Bloq]]" of "bloq_counts" incompatible with return type "Set[Tuple[Union[int, Expr], Bloq]]" in supertype "Bloq" [override]

I want an off-control.

Whoops, should have been: ``` python ctrl_spec = CtrlSpec(cvs=[0]) (ctrl,), q = bb.add(ZGate().controlled(ctrl_spec), ctrl=(ctrl,), q=q) ``` and ``` python ctrl_spec = CtrlSpec(QBit(), cvs=0b0) ctrl, q = bb.add(ZGate().controlled(ctrl_spec), ctrl=ctrl, q=q) ```

Right the second way seems a bit verbose. Very low priority though, just ran into it in #882

@tanujkhattar I hit this recently and remembered I was supposed to look into it. Any idea what's going on before I dig in?