Remove deprecated class PauliTransform (quantumlib#5703)

rht · May 1, 2023 · 5b31132 · 5b31132
1 parent a7bb0e8
commit 5b31132
Show file tree

Hide file tree

Showing 5 changed files with 3 additions and 25 deletions.
diff --git a/cirq-core/cirq/__init__.py b/cirq-core/cirq/__init__.py
@@ -267,7 +267,6 @@
     PauliSum,
     PauliSumExponential,
     PauliSumLike,
-    PauliTransform,
     phase_damp,
     phase_flip,
     PhaseDampingChannel,

diff --git a/cirq-core/cirq/ops/__init__.py b/cirq-core/cirq/ops/__init__.py
@@ -16,7 +16,7 @@
 
 from cirq.ops.arithmetic_operation import ArithmeticGate
 
-from cirq.ops.clifford_gate import CliffordGate, PauliTransform, SingleQubitCliffordGate
+from cirq.ops.clifford_gate import CliffordGate, SingleQubitCliffordGate
 
 from cirq.ops.dense_pauli_string import (
     BaseDensePauliString,

diff --git a/cirq-core/cirq/ops/clifford_gate.py b/cirq-core/cirq/ops/clifford_gate.py
@@ -12,13 +12,12 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-import dataclasses
 from typing import Any, cast, Dict, List, Optional, Sequence, Tuple, TYPE_CHECKING, Union
 
 
 import numpy as np
 
-from cirq import _compat, protocols, value, linalg, qis
+from cirq import protocols, value, linalg, qis
 from cirq._import import LazyLoader
 from cirq.ops import common_gates, named_qubit, raw_types, pauli_gates, phased_x_z_gate
 from cirq.ops.pauli_gates import Pauli
@@ -33,15 +32,8 @@
 transformers = LazyLoader("transformers", globals(), "cirq.transformers")
 
 
-@_compat.deprecated_class(deadline='v0.16', fix='Use DensePauliString instead.')
-@dataclasses.dataclass
-class PauliTransform:
-    to: Pauli
-    flip: bool
-
-
 def _to_pauli_tuple(matrix: np.ndarray) -> Optional[Tuple[Pauli, bool]]:
-    """Converts matrix to PauliTransform.
+    """Converts matrix to Pauli gate.
 
     If matrix is not ±Pauli matrix, returns None.
     """
@@ -677,11 +669,6 @@ def dense_pauli_string(self, pauli: Pauli) -> 'cirq.DensePauliString':
         coefficient = -1 if pauli_tuple[1] else 1
         return dense_pauli_string.DensePauliString(str(pauli_tuple[0]), coefficient=coefficient)
 
-    @_compat.deprecated(deadline='v0.16', fix='Use pauli_tuple() or dense_pauli_string() instead')
-    def transform(self, pauli: Pauli) -> PauliTransform:
-        pauli_tuple = self.pauli_tuple(pauli)
-        return PauliTransform(to=pauli_tuple[0], flip=pauli_tuple[1])
-
     def to_phased_xz_gate(self) -> phased_x_z_gate.PhasedXZGate:
         """Convert this gate to a PhasedXZGate instance.
 

diff --git a/cirq-core/cirq/ops/clifford_gate_test.py b/cirq-core/cirq/ops/clifford_gate_test.py
@@ -81,13 +81,6 @@ def test_init_from_xz(trans_x, trans_z):
     _assert_no_collision(gate)
 
 
-def test_transform_deprecated():
-    gate = cirq.SingleQubitCliffordGate.from_xz_map((cirq.X, True), (cirq.Y, False))
-    with cirq.testing.assert_deprecated('pauli_tuple', deadline='v0.16', count=4):
-        assert gate.transform(cirq.X).to == cirq.X
-        assert gate.transform(cirq.Z).to == cirq.Y
-
-
 def test_dense_pauli_string():
     gate = cirq.SingleQubitCliffordGate.from_xz_map((cirq.X, True), (cirq.Y, False))
     assert gate.dense_pauli_string(cirq.X) == cirq.DensePauliString('X', coefficient=-1)

diff --git a/cirq-core/cirq/protocols/json_test_data/spec.py b/cirq-core/cirq/protocols/json_test_data/spec.py
@@ -46,7 +46,6 @@
         'NeutralAtomDevice',
         'PauliSumCollector',
         'PauliSumExponential',
-        'PauliTransform',
         'PeriodicValue',
         'PointOptimizationSummary',
         'Points',