Skip to content

Commit 41dc418

Browse files
mtreinishraynelfss
andauthored
Fully port CheckMap to Rust (#13030)
* Fully port CheckMap to Rust This commit migrates the entirety of the CheckMap analysis pass to Rust. The pass operates solely in the rust domain and returns an `Option<(String, [u32; 2])>` to Python which is used to set the two property set fields appropriately. All the analysis of the dag is done in Rust. There is still Python interaction required though because control flow operations are only defined in Python. However the interaction is minimal and only to get the circuits for control flow blocks and converting them into DAGs (at least until #13001 is complete). This commit is based on top of #12959 and will need to be rebased after that merges. Closes #12251 Part of #12208 * Use a Vec<Qubit> for wire_map instead of a HashMap This commit switches to using a Vec<Qubit> for the internal wire_map used to map control flow qubits. A HashMap was originally used because in Python a dictionary is used. However, in the rust domain the inner qubits are contiguous integers starting from 0 so a Vec can be used for better performance in the case we have control flow. * Update crates/accelerate/src/check_map.rs Co-authored-by: Raynel Sanchez <87539502+raynelfss@users.noreply.github.com> --------- Co-authored-by: Raynel Sanchez <87539502+raynelfss@users.noreply.github.com>
1 parent 3d4bab2 commit 41dc418

File tree

5 files changed

+117
-29
lines changed

5 files changed

+117
-29
lines changed

crates/accelerate/src/check_map.rs

+98
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,98 @@
1+
// This code is part of Qiskit.
2+
//
3+
// (C) Copyright IBM 2024
4+
//
5+
// This code is licensed under the Apache License, Version 2.0. You may
6+
// obtain a copy of this license in the LICENSE.txt file in the root directory
7+
// of this source tree or at http://www.apache.org/licenses/LICENSE-2.0.
8+
//
9+
// Any modifications or derivative works of this code must retain this
10+
// copyright notice, and modified files need to carry a notice indicating
11+
// that they have been altered from the originals.
12+
13+
use hashbrown::HashSet;
14+
use pyo3::intern;
15+
use pyo3::prelude::*;
16+
use pyo3::wrap_pyfunction;
17+
18+
use qiskit_circuit::circuit_data::CircuitData;
19+
use qiskit_circuit::dag_circuit::{DAGCircuit, NodeType};
20+
use qiskit_circuit::imports::CIRCUIT_TO_DAG;
21+
use qiskit_circuit::operations::{Operation, OperationRef};
22+
use qiskit_circuit::Qubit;
23+
24+
fn recurse<'py>(
25+
py: Python<'py>,
26+
dag: &'py DAGCircuit,
27+
edge_set: &'py HashSet<[u32; 2]>,
28+
wire_map: Option<&'py [Qubit]>,
29+
) -> PyResult<Option<(String, [u32; 2])>> {
30+
let check_qubits = |qubits: &[Qubit]| -> bool {
31+
match wire_map {
32+
Some(wire_map) => {
33+
let mapped_bits = [
34+
wire_map[qubits[0].0 as usize],
35+
wire_map[qubits[1].0 as usize],
36+
];
37+
edge_set.contains(&[mapped_bits[0].into(), mapped_bits[1].into()])
38+
}
39+
None => edge_set.contains(&[qubits[0].into(), qubits[1].into()]),
40+
}
41+
};
42+
for node in dag.op_nodes(false) {
43+
if let NodeType::Operation(inst) = &dag.dag[node] {
44+
let qubits = dag.get_qargs(inst.qubits);
45+
if inst.op.control_flow() {
46+
if let OperationRef::Instruction(py_inst) = inst.op.view() {
47+
let raw_blocks = py_inst.instruction.getattr(py, "blocks")?;
48+
let circuit_to_dag = CIRCUIT_TO_DAG.get_bound(py);
49+
for raw_block in raw_blocks.bind(py).iter().unwrap() {
50+
let block_obj = raw_block?;
51+
let block = block_obj
52+
.getattr(intern!(py, "_data"))?
53+
.downcast::<CircuitData>()?
54+
.borrow();
55+
let new_dag: DAGCircuit =
56+
circuit_to_dag.call1((block_obj.clone(),))?.extract()?;
57+
let wire_map = (0..block.num_qubits())
58+
.map(|inner| {
59+
let outer = qubits[inner];
60+
match wire_map {
61+
Some(wire_map) => wire_map[outer.0 as usize],
62+
None => outer,
63+
}
64+
})
65+
.collect::<Vec<_>>();
66+
let res = recurse(py, &new_dag, edge_set, Some(&wire_map))?;
67+
if res.is_some() {
68+
return Ok(res);
69+
}
70+
}
71+
}
72+
} else if qubits.len() == 2
73+
&& (dag.calibrations_empty() || !dag.has_calibration_for_index(py, node)?)
74+
&& !check_qubits(qubits)
75+
{
76+
return Ok(Some((
77+
inst.op.name().to_string(),
78+
[qubits[0].0, qubits[1].0],
79+
)));
80+
}
81+
}
82+
}
83+
Ok(None)
84+
}
85+
86+
#[pyfunction]
87+
pub fn check_map(
88+
py: Python,
89+
dag: &DAGCircuit,
90+
edge_set: HashSet<[u32; 2]>,
91+
) -> PyResult<Option<(String, [u32; 2])>> {
92+
recurse(py, dag, &edge_set, None)
93+
}
94+
95+
pub fn check_map_mod(m: &Bound<PyModule>) -> PyResult<()> {
96+
m.add_wrapped(wrap_pyfunction!(check_map))?;
97+
Ok(())
98+
}

crates/accelerate/src/lib.rs

+1
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ use std::env;
1414

1515
use pyo3::import_exception;
1616

17+
pub mod check_map;
1718
pub mod circuit_library;
1819
pub mod commutation_analysis;
1920
pub mod commutation_checker;

crates/pyext/src/lib.rs

+7-6
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,12 @@
1313
use pyo3::prelude::*;
1414

1515
use qiskit_accelerate::{
16-
circuit_library::circuit_library, commutation_analysis::commutation_analysis,
17-
commutation_checker::commutation_checker, convert_2q_block_matrix::convert_2q_block_matrix,
18-
dense_layout::dense_layout, error_map::error_map,
19-
euler_one_qubit_decomposer::euler_one_qubit_decomposer, filter_op_nodes::filter_op_nodes_mod,
20-
isometry::isometry, nlayout::nlayout, optimize_1q_gates::optimize_1q_gates,
21-
pauli_exp_val::pauli_expval,
16+
check_map::check_map_mod, circuit_library::circuit_library,
17+
commutation_analysis::commutation_analysis, commutation_checker::commutation_checker,
18+
convert_2q_block_matrix::convert_2q_block_matrix, dense_layout::dense_layout,
19+
error_map::error_map, euler_one_qubit_decomposer::euler_one_qubit_decomposer,
20+
filter_op_nodes::filter_op_nodes_mod, isometry::isometry, nlayout::nlayout,
21+
optimize_1q_gates::optimize_1q_gates, pauli_exp_val::pauli_expval,
2222
remove_diagonal_gates_before_measure::remove_diagonal_gates_before_measure, results::results,
2323
sabre::sabre, sampled_exp_val::sampled_exp_val, sparse_pauli_op::sparse_pauli_op,
2424
star_prerouting::star_prerouting, stochastic_swap::stochastic_swap, synthesis::synthesis,
@@ -43,6 +43,7 @@ fn _accelerate(m: &Bound<PyModule>) -> PyResult<()> {
4343
add_submodule(m, qiskit_qasm2::qasm2, "qasm2")?;
4444
add_submodule(m, qiskit_qasm3::qasm3, "qasm3")?;
4545
add_submodule(m, circuit_library, "circuit_library")?;
46+
add_submodule(m, check_map_mod, "check_map")?;
4647
add_submodule(m, convert_2q_block_matrix, "convert_2q_block_matrix")?;
4748
add_submodule(m, dense_layout, "dense_layout")?;
4849
add_submodule(m, error_map, "error_map")?;

qiskit/__init__.py

+1
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,7 @@
9292
sys.modules["qiskit._accelerate.commutation_checker"] = _accelerate.commutation_checker
9393
sys.modules["qiskit._accelerate.commutation_analysis"] = _accelerate.commutation_analysis
9494
sys.modules["qiskit._accelerate.synthesis.linear_phase"] = _accelerate.synthesis.linear_phase
95+
sys.modules["qiskit._accelerate.check_map"] = _accelerate.check_map
9596
sys.modules["qiskit._accelerate.filter_op_nodes"] = _accelerate.filter_op_nodes
9697

9798
from qiskit.exceptions import QiskitError, MissingOptionalLibraryError

qiskit/transpiler/passes/utils/check_map.py

+10-23
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,8 @@
1414

1515
from qiskit.transpiler.basepasses import AnalysisPass
1616
from qiskit.transpiler.target import Target
17-
from qiskit.converters import circuit_to_dag
17+
18+
from qiskit._accelerate import check_map
1819

1920

2021
class CheckMap(AnalysisPass):
@@ -67,25 +68,11 @@ def run(self, dag):
6768
if not self.qargs:
6869
self.property_set[self.property_set_field] = True
6970
return
70-
wire_map = {bit: index for index, bit in enumerate(dag.qubits)}
71-
self.property_set[self.property_set_field] = self._recurse(dag, wire_map)
72-
73-
def _recurse(self, dag, wire_map) -> bool:
74-
for node in dag.op_nodes(include_directives=False):
75-
if node.is_control_flow():
76-
for block in node.op.blocks:
77-
inner_wire_map = {
78-
inner: wire_map[outer] for inner, outer in zip(block.qubits, node.qargs)
79-
}
80-
if not self._recurse(circuit_to_dag(block), inner_wire_map):
81-
return False
82-
elif (
83-
len(node.qargs) == 2
84-
and not dag.has_calibration_for(node)
85-
and (wire_map[node.qargs[0]], wire_map[node.qargs[1]]) not in self.qargs
86-
):
87-
self.property_set["check_map_msg"] = (
88-
f"{node.name}({wire_map[node.qargs[0]]}, {wire_map[node.qargs[1]]}) failed"
89-
)
90-
return False
91-
return True
71+
res = check_map.check_map(dag, self.qargs)
72+
if res is None:
73+
self.property_set[self.property_set_field] = True
74+
return
75+
self.property_set[self.property_set_field] = False
76+
self.property_set["check_map_msg"] = (
77+
f"{res[0]}({dag.qubits[res[1][0]]}, {dag.qubits[res[1][1]]}) failed"
78+
)

0 commit comments

Comments
 (0)