Jake Lishman

It's not hard to do from the compiler side but Terra can't represent it yet, so I didn't implement it; we'd have to stray too far from the input source,...

At some point, the question is "what is the purpose of this package?". This is not meant to be an optimising compiler, it's just meant to get programs into the...

tbf, I think this is `tox` doing what it's intended to do. If you want to skip the reinstall step with tox 4, the command is ``` tox run --skip-pkg-install...

I'd suggest both against overloading `copy_operations` and against excessive deep-copying. From Chris' use-case, I don't think deep-copying is appropriate, and is likely just to introduce performance bottlenecks. If somebody puts...

@Andir16: sorry for the delay, December was a busy/holiday-ish month. Yes, I think changing to `dag.metadata = circuit.metadata.copy()` is an acceptable change here.

Thanks for the report - this looks like a clear mistake in `depth`. I can post a PR in a sec that does a little bit of tidying up of...

Thanks for the report. I was just looking into this and made #12128 to point out a few more things I'd spotted in `OptimizeSwapBeforeMeasure`, but as I was doing that,...

Thanks for this - looks like the immediate problem is in the OpenQASM 3 exporter not writing out a `gphase` statement, since the importer seems to be interpreting `gphase` correctly...

Aha - more local testing: seems like `gphase` in `qe-qasm` doesn't like having a sole numeric literal in its argument, which is unfortunate, since that's going to be 99.9999% of...

At the moment there's no real scope for this going in. It's correct from the Qiskit and OQ3 side, but in practice, if we merged it, we'd break any submission...