Mirko Amico

> I'm glad to see we have other pulse shapes in Qiskit. If this is going to be backend-supported, you may also want to update > > qobj converter, >...

Picking this up again since [this PR](https://github.com/Qiskit/qiskit-terra/pull/9329) was merged recently. As it is now, `GaussianSquareEcho` only contains the Gaussian square pulses without the single qubit gates for the echo. There...

posting here some comments received through other channels: - before digging into the tutorial, I think it would help a ton to motivate the problem a bit more. You mention...

I would vote for having appendices as this seems to be a common occurence in the utility notebook I've been working on. Maybe a link to external reference would be...

I've added the estimate for the QPU time. In my experiments it took 6s per function call of COBYLA (and about 40 calls). I've also switched from `transpile` to the...

> Just noticed/fixed a bug in the small scale example where the call to instantiate the sampler wasn't using the session being generated. > > I'm also wondering about editing...

I'm working on a version of this notebook that will be using the hardware at utility scale. I'd hold off any further work on this version based on the simulator

I think the problem is not just with the fact that there can be redundant layouts if some qubits are not used (but should be included in the transpired circuit)....

I'm generally more inclined for the "all imports up top" option as that makes it easy for users to copy paste code from the notebook knowing they can find all...

This looks great! Just a couple comments - I think we should also refer to the [KQD paper ](https://arxiv.org/abs/2407.14431)(and [tutorial](https://quantum.cloud.ibm.com/docs/en/tutorials/krylov-quantum-diagonalization)) as the convergence guarantee of the unitary krylov subspace are...