Create a workflow to perform FSim based z-phase calibration #6568
Conversation
| ) -> Tuple[xeb_fitting.XEBCharacterizationResult]: | ||
| """Perform z-phase calibration for fermionic gates. | ||
|
|
||
| For a given fermionic two-qubit gate we assume an error model that can be described |
There was a problem hiding this comment.
Same comment as above.
| atol: float = 1e-3, | ||
| pool: Optional[Union[multiprocessing.Pool, concurrent.futures.ThreadPoolExecutor]] = None, | ||
| ) -> Tuple[xeb_fitting.XEBCharacterizationResult]: | ||
| """Perform z-phase calibration for fermionic gates. |
There was a problem hiding this comment.
Let's call it a "number-conserving two-qubit gate" instead of a "fermionic two-qubit gate"
| 1: ───Rz(b)───two_qubit_gate───Rz(d)─── | ||
| for some angles a, b, c, and d. | ||
|
|
||
| Since the two-qubit gate is a fermionic-gate, it can be represented by an FSimGate and the |
There was a problem hiding this comment.
Same comment as above.
| results = xeb_utils.execute_with_progress_par( | ||
| closure, | ||
| subselected_dfs, | ||
| pool=pool, | ||
| progress_bar=tqdm.tqdm, | ||
| desc='characterize fsim parameters', | ||
| ) |
There was a problem hiding this comment.
This step took 30 minutes to run when I tested it (colab). I know that the gate_cal_and_checks code is also slow like this, but the internal XEB calibration is fast, so hopefully there is a way to make this faster.
| options: The XEB-fitting options. If None, calibrate all 5 PhasedFSimGate parameters, | ||
| using the representation of a two-qubit gate as an FSimGate for the initial guess. |
There was a problem hiding this comment.
I think the default behavior should be to characterize the three
| options: The XEB-fitting options. If None, calibrate all 5 PhasedFSimGate parameters, | ||
| using the representation of a two-qubit gate as an FSimGate for the initial guess. |
There was a problem hiding this comment.
I tried this out, and it didn't seem to characterize any of the angles (colab).
There was a problem hiding this comment.
fixed, it should work now
There was a problem hiding this comment.
btw to speed things up, run the calibration method in a multiprocessing pool e.g.
with mulitprocessing.Pool(10) as pool:
result_zphase = cirq.experiments.calibrate_z_phases(sampler, qubits, pool=pool)There was a problem hiding this comment.
Could we make it create a pool by default?
|
This should probably go in a separate PR, but I think the next step after this is to write a transformer that replaces the two-qubit gates in a circuit with ones that correct for these z-phases. |
No description provided.