Naoki Kanazawa

Results 47 comments of Naoki Kanazawa

Thanks @isuruf I confirmed `sym.And` is sympy compatible. Seems like this is correct syntax. ```python import sympy as sym a = sym.Symbol("a") expr = sym.And(a < 1, 0 < a)...

I assumed float or integer for `t` and `duration`, but it would be great complex is also acceptable. `abs(t - duration) > 0` makes sense to me.

We should extend tomography module for `QuantumCircuit` already existing in `qiskit-ignis/qiskit/ignis/verification/tomography`. When `QuantumCircuit` is given, the tomography module adds _state preparation circuits_ and _measurement circuits_ to the given circuit, then...

seems to be better to adopt 2. Do we have a plan to provide a pass to revert schedule to circuit? It sounds like the best option to use here....

I noticed an example in which 2 doesn’t work. In the case of QPT in Pauli basis, scheduler will generate following schedule. ``` ——[prep A]——[circ]——(barrier)——[proj B]——[acquire]—— t0 t1 t2 t3...

Yes, I did. If I understand correctly, scheduler works for list of circuits, but they are independently processed. https://github.com/Qiskit/qiskit-terra/blob/a0290821af49bd5ee8269ae07831ecad77d4cfe8/qiskit/compiler/schedule.py#L64 Maybe this can be modified so that _alap_ pass functions for...

Yes. It is also problem in Qasm QPT. Replacing projection circuit by U3 is good direction. But if this problem doesn't degrade QPT result, based on your experience, I'll just...

Thanks @mtreinish @wshanks I updated the release note and fixed failing tests.

Implementation of waveform counterpart, i.e. `continuous` and `discrete`, is not mandatory, because we can get the same array with `SymbolicPulse.get_waveform` method (but nothing stops you to implement).

Similar issue and https://github.com/Qiskit/qiskit-terra/pull/8232 is also waiting for symengine 0.10.