Julien Gacon

Outdated since the port of commutation checker to rust

I ran the utility scale benchmarks in `test/benchmarks` and this improves the performance for the QFT benchmarks by ~22% leaving the others unchanged. The depths also don't change so this...

You could comment out the test case saying that it requires #13828 to be fixed, but I wouldn't add a custom warning, since that's extra cleanup work once the bug...

Generally I agree with Jake that we've been trying to avoid adding methods that don't have an explicit use-case to not clutter the circuit (which already has loads of methods)...

To add some more details, this does indeed break because `Split2QUnitaries` splits the 2q `MyGate` into two 1q `U` instructions -- which are not part of the basis gate set....

So currently we run `Split2QUnitaries`, assuming that we have a complete 1-qubit basis set, which is not always given. For example, we also run into the same issue if the...

#13015 with #13095 should fix these issues so I'll close this -- feel free to re-open if we missed anything!

#14542 does step 1 here by removing the Python dependency of the `CommutationChecker`. To actually call the pass from C, we need to be able to convert `CircuitData` to a...

Thank you for your interest @DisantHoridnt. Due to the scope of this issue, we’re currently handling it in the maintainer team. However, once we've advanced on this, contributions on concrete,...

**Short answer:** Yeah we currently don't do value-based optimization for gates that are already in the target basis (note that the `AerSimulator` accepts any standard gate in its basis). To...