Use rustworkx 0.15.0 features in DAGCircuit.remove_op_node (#12756)

This commit updates the minimum rustworkx version to 0.15.0 to pull in the new PyDiGraph.remove_node_retain_edges_by_id() method introduced in that release. This new function is used for the DAGCircuit.remove_op_node() method instead of the PyDiGraph.remove_node_retain_edges() function. This new method has much better scaling characteristics and should improve the performance characteristics of removing very wide operations from a DAGCircuit. Fixes #11677 Part of #12156
Qiskit · Jul 11, 2024 · 99ae318 · 99ae318
1 parent 4c9ca6e
commit 99ae318
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/qiskit/dagcircuit/dagcircuit.py b/qiskit/dagcircuit/dagcircuit.py
@@ -1986,9 +1986,7 @@ def remove_op_node(self, node):
                 "node type was wrongly provided."
             )
 
-        self._multi_graph.remove_node_retain_edges(
-            node._node_id, use_outgoing=False, condition=lambda edge1, edge2: edge1 == edge2
-        )
+        self._multi_graph.remove_node_retain_edges_by_id(node._node_id)
         self._decrement_op(node.name)
 
     def remove_ancestors_of(self, node):

diff --git a/releasenotes/notes/update-rustworkx-min-version-4f07aacfebccae80.yaml b/releasenotes/notes/update-rustworkx-min-version-4f07aacfebccae80.yaml
@@ -0,0 +1,7 @@
+---
+upgrade_misc:
+  - |
+    The minimum version of rustworkx required to run this release has been
+    increased from 0.14.0 to 0.15.0. This is required because Qiskit is now
+    using new functionality added in the rustworkx 0.15.0 release which
+    improves performance.
diff --git a/requirements.txt b/requirements.txt
@@ -1,4 +1,4 @@
-rustworkx>=0.14.0
+rustworkx>=0.15.0
 numpy>=1.17,<3
 scipy>=1.5
 sympy>=1.3