Alexander Ivrii

Results 63 comments of Alexander Ivrii

@Cryoris: I am happy to change names of the synthesis classes. However, calling a synthesis algorithm `MCXVChain` is bad as this clashes with the already existing gate name. `VChain` is...

@Cryoris, many thanks for your feedback! Actually, I will ask: would you like to work on this together? In a sense, we are doing this already. :) One tricky complication...

@mtreinish, thanks for the review! I presume that > except for one potential bug in line refers to iterating over nodes without taking removals into account, but (as I commented...

Hmm, building docs started to fail after merging the main branch. Is this an instance of "things may go wrong on week-ends"?

Thanks @poig, the animation looks fantastic. But one clarification: is this PR about computing gate layers or about drawing them? Please take a look at the `QuantumCircuit.depth()` function which internally...

Oh, so the real question is how to draw things (no experience here on my part). There seems to be another new related issue: #8374.

I am wondering if the `CommutativeCancellation` pass also tries to do something similar?

@JMuff22, you are correct, CZ is symmetric. Could you please provide a code snippet that results in the above error?

@JMuff22, seems that no one picked up on this. Sure, modifying the code in `gate_direction.py` would work. Alternatively, there was some earlier discussion in https://github.com/Qiskit/qiskit-terra/pull/7875#pullrequestreview-936996092 to annotate a gate type...

@ShellyGarion , @mtreinish, how would [LinearFunctionsSynthesis](https://qiskit.org/documentation/stubs/qiskit.transpiler.passes.LinearFunctionsSynthesis.html#qiskit.transpiler.passes.LinearFunctionsSynthesis) know if it's supposed to improve gate-count or depth? I might be wrong, but I don't recall seeing the optimization criteria as part of...