Results 300 comments of Matthew Treinish

I would say I don't like this undocumented behavior because it makes https://github.com/Qiskit/qiskit-terra/pull/7789 and similar refactors much harder to work with. https://github.com/Qiskit/qiskit-terra/pull/7789 still supports the list input (except on optimization...

ok, I've updated the tests to no longer rely on these backends having pulse defaults present. Hopefully things will pass now (this also caught a bug in the assembler which...

The tutorials failure should be fixed once: https://github.com/Qiskit/qiskit-tutorials/pull/1356 merges

> Looks good to me - the diff looks cleaner without the +1,800 file as well! I'm never very sure if we _should_ be making assertions about the exact forms...

FWIW, my normal suggestion in this case is to do something like `pip install --prefer-binary tweedledum qiskit-terra` which should install tweedledum from a precompiled binary along with terra. @jakelishman environment...

In order to speed up the review cycle on this a bit, instead of inline suggestions I pushed this PR to your branch: https://github.com/1ucian0/qiskit-terra/pull/30 with some changes I think we...

Should we add `SabreLayout` and `SabreSwap` to this list because of: #8552 ?

I'm changing the target here to 0.20.0, because while the first piece of adding the target to the `PassManagerConfig` is done in #7227 and the basis translator will have initial...

So for this issue I think we should concentrate just on passes that are either basis aware or noise aware. These are places where the target provides a more rich...

I've crossed `CommutativeCancellation` off the list because I'm not sure there is anything extra a target can add to the pass. It is used solely to know which z rotation...