-
Notifications
You must be signed in to change notification settings - Fork 88
Closed
Labels
Description
- change the
wire_symbolmethod to explicitly take aRegisterandidx: Tuple[int, ...]rather than oneSoquetsince the symbol shouldn't depend on thebinst. - Eliminate or rename or merge
short_nameto explicitly compliment the wire symbols. This string should serve (solely) as a "title" to be written above the wire symbols and should be included if and only if the wire symbols don't already imply the gate. E.g. it's not necessary to write "CNOT" over something that's clearly a CNOT. Practically: we can rename the existing method and let it returnNoneor we can use a sentinel value to thewire_symbolmethod (e.g.reg=None) to imply that we're asking for the (optional) title. - Single source of truth for drawing: modify the
GraphvizDrawerfor drawing composite bloq graphs to use thewire_symbolmethod and values to label the soquet rows in the table for each bloq. Use the title method for the title, and again omit if not required. Practically: we'll need a new method onWireSymbolto return a textual equivalent of the symbol
fdmalone