-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Create a gauge for idle moments #7648
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
3e47db9
Create a gauge for idle moments
NoureldinYosri 69a6d5c
nit
NoureldinYosri 74e9998
sytle
NoureldinYosri 66e2b5c
nit
NoureldinYosri f45f1a0
use Zlatko's docstring
NoureldinYosri 408506d
add tests
NoureldinYosri e4a6010
Merge branch 'main' into gauge_empty
NoureldinYosri 2eede92
fix ci
NoureldinYosri 38dc9ca
lint
NoureldinYosri e8557b5
coverage
NoureldinYosri 0fa5726
address comments
NoureldinYosri 9c26f87
nit
NoureldinYosri f05489f
Merge branch 'main' into gauge_empty
NoureldinYosri 99bbacb
nit
NoureldinYosri 1285bb7
nit
NoureldinYosri 1863215
use MockRng
NoureldinYosri f3dbbbd
nit
NoureldinYosri File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
222 changes: 222 additions & 0 deletions
222
cirq-core/cirq/transformers/gauge_compiling/idle_moments_gauge.py
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,222 @@ | ||
# Copyright 2025 The Cirq Developers | ||
# | ||
# Licensed under the Apache License, Version 2.0 (the "License"); | ||
# you may not use this file except in compliance with the License. | ||
# You may obtain a copy of the License at | ||
# | ||
# https://www.apache.org/licenses/LICENSE-2.0 | ||
# | ||
# Unless required by applicable law or agreed to in writing, software | ||
# distributed under the License is distributed on an "AS IS" BASIS, | ||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
# See the License for the specific language governing permissions and | ||
# limitations under the License. | ||
from __future__ import annotations | ||
|
||
import functools | ||
from typing import Hashable, Iterator, Sequence, TYPE_CHECKING | ||
|
||
import attrs | ||
import numpy as np | ||
|
||
import cirq.circuits as circuits | ||
import cirq.ops as ops | ||
import cirq.protocols as protocols | ||
import cirq.transformers.transformer_api as transformer_api | ||
|
||
if TYPE_CHECKING: | ||
import cirq | ||
|
||
_PAULIS: tuple[cirq.Gate, ...] = (ops.I, ops.X, ops.Y, ops.Z) # type: ignore[has-type] | ||
_CLIFFORDS = tuple(ops.SingleQubitCliffordGate.all_single_qubit_cliffords) | ||
_INV_CLIFFORDS = tuple(c**-1 for c in ops.SingleQubitCliffordGate.all_single_qubit_cliffords) | ||
|
||
_NAME_TO_GATES = {'pauli': _PAULIS, 'clifford': _CLIFFORDS, 'inv_clifford': _INV_CLIFFORDS} | ||
|
||
|
||
def _gauges_arg_converter(gauges: str | Sequence[cirq.Gate] = 'clifford') -> tuple[cirq.Gate, ...]: | ||
if isinstance(gauges, str): | ||
if gauges not in _NAME_TO_GATES: | ||
valid_names = tuple(_NAME_TO_GATES.keys()) | ||
raise ValueError(f"{gauges} is not a valid gauge name, valid names are {valid_names}") | ||
return _NAME_TO_GATES[gauges] | ||
return tuple(gauges) | ||
|
||
|
||
def _repr_fn(gauges: tuple[cirq.Gate, ...]) -> str: | ||
if gauges is _PAULIS or gauges == _PAULIS: | ||
return '"pauli"' | ||
if gauges is _CLIFFORDS or gauges == _CLIFFORDS: | ||
return '"clifford"' | ||
if gauges is _INV_CLIFFORDS or gauges == _INV_CLIFFORDS: | ||
return '"inv_clifford"' | ||
return repr(gauges) | ||
|
||
|
||
def _get_structure( | ||
active: list[tuple[int, bool]], | ||
min_length: int, | ||
n: int, | ||
gauge_beginning: bool, | ||
gauge_ending: bool, | ||
) -> Iterator[tuple[int, int]]: | ||
if gauge_beginning: | ||
stop, is_mergable = active[0] | ||
if min_length <= stop: | ||
if is_mergable: | ||
yield (0, stop) | ||
else: | ||
yield (0, stop - 1) | ||
|
||
for i in range(len(active) - 1): | ||
left_pos, left_is_mergable = active[i] | ||
right_pos, right_is_mergable = active[i + 1] | ||
if right_pos - left_pos - 1 >= min_length: | ||
yield (left_pos + 1 - left_is_mergable, right_pos - 1 + right_is_mergable) | ||
|
||
if gauge_ending: | ||
stop, is_mergable = active[-1] | ||
if min_length <= n - stop - 1: | ||
if is_mergable: | ||
yield (stop, n - 1) | ||
else: | ||
yield (stop + 1, n - 1) | ||
|
||
|
||
def _merge(g1: cirq.Gate, g2: cirq.Gate, q: cirq.Qid, tags: Sequence[Hashable]) -> cirq.Operation: | ||
u1 = protocols.unitary(g1) | ||
u2 = protocols.unitary(g2) | ||
return ops.PhasedXZGate.from_matrix(u2 @ u1)(q).with_tags(*tags) | ||
|
||
|
||
@transformer_api.transformer | ||
@attrs.frozen | ||
class IdleMomentsGauge: | ||
r"""A transformer that inserts identity-preserving "gauge" gates around idle qubit moments. | ||
|
||
This transformer identifies sequences of consecutive idle moments on a single qubit | ||
that meet a `min_length` threshold. For each such sequence, it inserts a randomly | ||
selected gate `G` from `gauges` at the start of the idle period and its inverse `G^-1` | ||
at the end. This ensures the logical circuit behavior remains unchanged ($G \cdot G^{-1} = I$). | ||
|
||
The primary goal is to introduce specific structure into idle periods, which is | ||
useful for experiments. | ||
|
||
Attributes: | ||
min_length: Minimum number of consecutive idle moments for a gauge to be applied (>= 1). | ||
|
||
gauges: A sequence of `cirq.Gate` objects to randomly select from. | ||
Can be a custom tuple or a string alias: | ||
- `"pauli"`: Uses single-qubit Pauli gates (I, X, Y, Z). | ||
- `"clifford"`: Uses all 24 single-qubit Clifford gates. | ||
|
||
gauge_beginning: If `True`, applies a gauge to idle moments at the circuit's start, | ||
before any other qubit operation. Defaults to `False`. | ||
|
||
gauge_ending: If `True`, applies a gauge to idle moments at the circuit's end, | ||
after the last qubit operation. Defaults to `False`. | ||
""" | ||
|
||
min_length: int = attrs.field( | ||
validator=(attrs.validators.instance_of(int), attrs.validators.ge(1)) | ||
) | ||
gauges: tuple[cirq.Gate, ...] = attrs.field( | ||
default='clifford', converter=_gauges_arg_converter, repr=_repr_fn | ||
) | ||
gauge_beginning: bool = False | ||
gauge_ending: bool = False | ||
|
||
@functools.cached_property | ||
def gauges_inverse(self) -> tuple[cirq.Gate, ...]: | ||
if self.gauges is _PAULIS: | ||
return _PAULIS | ||
if self.gauges is _CLIFFORDS: | ||
return _INV_CLIFFORDS | ||
if self.gauges is _INV_CLIFFORDS: | ||
NoureldinYosri marked this conversation as resolved.
Show resolved
Hide resolved
|
||
return _CLIFFORDS | ||
return tuple(g**-1 for g in self.gauges) | ||
|
||
def __call__( | ||
self, | ||
circuit: cirq.AbstractCircuit, | ||
*, | ||
context: transformer_api.TransformerContext | None = None, | ||
rng_or_seed: np.random.Generator | int | None = None, | ||
): | ||
"""Apply the IdleMomentGauge transformer. | ||
|
||
Args: | ||
circuit: The circuit to process. | ||
context: The TransformerContext. | ||
rng_or_seed: The source of randomness. | ||
|
||
Returns: | ||
A transformed circuit. | ||
|
||
Raises: | ||
ValueError: if the TransformerContext has deep=True. | ||
""" | ||
rng = ( | ||
rng_or_seed | ||
if isinstance(rng_or_seed, np.random.Generator) | ||
else np.random.default_rng(rng_or_seed) | ||
) | ||
context = ( | ||
context | ||
if isinstance(context, transformer_api.TransformerContext) | ||
else transformer_api.TransformerContext(deep=False) | ||
) | ||
if context.deep: | ||
raise ValueError("IdleMomentsGauge doesn't support deep TransformerContext") | ||
|
||
tags_to_ignore = frozenset(context.tags_to_ignore) | ||
all_qubits = circuit.all_qubits() | ||
|
||
active_moments: dict[cirq.Qid, list[tuple[int, bool]]] = {q: [] for q in all_qubits} | ||
for m_id, moment in enumerate(circuit): | ||
if not tags_to_ignore.isdisjoint(moment.tags): | ||
for q in all_qubits: | ||
active_moments[q].append((m_id, False)) | ||
else: | ||
for op in moment: | ||
is_mergable = ( | ||
len(op.qubits) == 1 | ||
and tags_to_ignore.isdisjoint(op.tags) | ||
and op.gate is not None | ||
) | ||
for q in op.qubits: | ||
active_moments[q].append((m_id, is_mergable)) | ||
|
||
single_qubit_moments = [{q: op for op in m if len(op.qubits) == 1} for m in circuit] | ||
non_single_qubit_moments = [[op for op in m if len(op.qubits) != 1] for m in circuit] | ||
|
||
for q, active in active_moments.items(): | ||
for s, e in _get_structure( | ||
active, self.min_length, len(circuit), self.gauge_beginning, self.gauge_ending | ||
): | ||
gate_index = rng.choice(len(self.gauges)) | ||
gate = self.gauges[gate_index] | ||
gate_inv = self.gauges_inverse[gate_index] | ||
|
||
if existing_op := single_qubit_moments[s].get(q, None): | ||
existing_gate = existing_op.gate | ||
assert existing_gate is not None | ||
single_qubit_moments[s][q] = _merge(existing_gate, gate, q, existing_op.tags) | ||
else: | ||
single_qubit_moments[s][q] = gate(q) | ||
|
||
if existing_op := single_qubit_moments[e].get(q, None): | ||
existing_gate = existing_op.gate | ||
assert existing_gate is not None | ||
single_qubit_moments[e][q] = _merge( | ||
gate_inv, existing_gate, q, existing_op.tags | ||
) | ||
else: | ||
single_qubit_moments[e][q] = gate_inv(q) | ||
|
||
return circuits.Circuit.from_moments( | ||
*( | ||
[op for op in sq.values()] + nsq | ||
for sq, nsq in zip(single_qubit_moments, non_single_qubit_moments, strict=True) | ||
) | ||
) |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.