Deprecate GlobalisePhasedX (#1683)

CQCL · Nov 19, 2024 · 6c23c79 · 6c23c79
1 parent afe3102
commit 6c23c79
Show file tree

Hide file tree

Showing 5 changed files with 29 additions and 2 deletions.
diff --git a/pytket/binders/passes.cpp b/pytket/binders/passes.cpp
@@ -489,7 +489,16 @@ PYBIND11_MODULE(passes, m) {
       "DecomposeMultiQubitsCX", &DecomposeMultiQubitsCX,
       "Converts all multi-qubit gates into CX and single-qubit gates.");
   m.def(
-      "GlobalisePhasedX", &GlobalisePhasedX,
+      "GlobalisePhasedX",
+      [](bool squash) {
+        PyErr_WarnEx(
+            PyExc_DeprecationWarning,
+            "The GlobalisePhasedX pass is unreliable and deprecated. It will "
+            "be removed no earlier that three months after the pytket 1.35 "
+            "release.",
+            1);
+        return GlobalisePhasedX(squash);
+      },
       "Turns all PhasedX and NPhasedX gates into global gates\n\n"
       "Replaces any PhasedX gates with global NPhasedX gates. "
       "By default, this transform will squash all single-qubit gates "
@@ -499,6 +508,8 @@ PYBIND11_MODULE(passes, m) {
       "performance. If squashing is disabled, each non-global PhasedX gate "
       "will be replaced with two global NPhasedX, but any other gates will "
       "be left untouched."
+      "\n\nDEPRECATED: This pass will be removed no earlier than three months "
+      "after the pytket 1.35 release."
       "\n\n:param squash: Whether to squash the circuit in pre-processing "
       "(default: true)."
       "\n\nIf squash=true (default), the `GlobalisePhasedX` transform's "

diff --git a/pytket/binders/transform.cpp b/pytket/binders/transform.cpp
@@ -386,7 +386,16 @@ PYBIND11_MODULE(transform, m) {
           "DecomposeNPhasedX", &Transforms::decompose_NPhasedX,
           "Decompose NPhasedX gates into single-qubit PhasedX gates.")
       .def_static(
-          "GlobalisePhasedX", &Transforms::globalise_PhasedX,
+          "GlobalisePhasedX",
+          [](bool squash) {
+            PyErr_WarnEx(
+                PyExc_DeprecationWarning,
+                "The GlobalisePhasedX transform is unreliable and deprecated. "
+                "It will be removed no earlier than three months after the "
+                "pytket 1.35 release.",
+                1);
+            return Transforms::globalise_PhasedX(squash);
+          },
           "Turns all PhasedX and NPhasedX gates into global gates\n\n"
           "Replaces any PhasedX gates with global NPhasedX gates. "
           "By default, this transform will squash all single-qubit gates "
@@ -396,6 +405,8 @@ PYBIND11_MODULE(transform, m) {
           "performance. If squashing is disabled, each non-global PhasedX gate "
           "will be replaced with two global NPhasedX, but any other gates will "
           "be left untouched."
+          "\n\nDEPRECATED: This transform will be removed no earlier than "
+          "three months after the pytket 1.35 release."
           "\n\n:param squash: Whether to squash the circuit in pre-processing "
           "(default: true)."
           "\n\nIf squash=true (default), the `GlobalisePhasedX` transform's "

diff --git a/pytket/docs/changelog.rst b/pytket/docs/changelog.rst
@@ -24,6 +24,7 @@ Features:
 Deprecations:
 
 * Deprecate `ClassicalExpBox` and related methods, in favour of `ClExprOp`.
+* Deprecate `GlobalisePhasedX` pass and transform.
 
 Fixes:
 

diff --git a/pytket/pytket/_tket/passes.pyi b/pytket/pytket/_tket/passes.pyi
@@ -439,6 +439,8 @@ def GlobalisePhasedX(squash: bool = True) -> BasePass:
     
     Replaces any PhasedX gates with global NPhasedX gates. By default, this transform will squash all single-qubit gates to PhasedX and Rz gates before proceeding further. Existing non-global NPhasedX will not be preserved. This is the recommended setting for best performance. If squashing is disabled, each non-global PhasedX gate will be replaced with two global NPhasedX, but any other gates will be left untouched.
     
+    DEPRECATED: This pass will be removed no earlier than three months after the pytket 1.35 release.
+    
     :param squash: Whether to squash the circuit in pre-processing (default: true).
     
     If squash=true (default), the `GlobalisePhasedX` transform's `apply` method will always return true. For squash=false, `apply()` will return true if the circuit was changed and false otherwise.

diff --git a/pytket/pytket/_tket/transform.pyi b/pytket/pytket/_tket/transform.pyi
@@ -158,6 +158,8 @@ class Transform:
         
         Replaces any PhasedX gates with global NPhasedX gates. By default, this transform will squash all single-qubit gates to PhasedX and Rz gates before proceeding further. Existing non-global NPhasedX will not be preserved. This is the recommended setting for best performance. If squashing is disabled, each non-global PhasedX gate will be replaced with two global NPhasedX, but any other gates will be left untouched.
         
+        DEPRECATED: This transform will be removed no earlier than three months after the pytket 1.35 release.
+        
         :param squash: Whether to squash the circuit in pre-processing (default: true).
         
         If squash=true (default), the `GlobalisePhasedX` transform's `apply` method will always return true. For squash=false, `apply()` will return true if the circuit was changed and false otherwise.