Skip to content
Merged
Show file tree
Hide file tree
Changes from 52 commits
Commits
Show all changes
72 commits
Select commit Hold shift + click to select a range
62100c1
added abstract initial mapper and identity initial mapper
ammareltigani Aug 15, 2022
9df051b
added __str__ and __repr__ for MappingManager
ammareltigani Aug 15, 2022
8610ae2
minor bug
ammareltigani Aug 15, 2022
d9a7a3c
made MappingManager not serializable
ammareltigani Aug 16, 2022
c774674
removed unused import
ammareltigani Aug 16, 2022
a06c240
merging with mapping-manager repr and str PR #5828
ammareltigani Aug 16, 2022
c12cc03
pushed AbstractInitialMapping and IdentityInitialMapping name to 'cir…
ammareltigani Aug 16, 2022
83e976f
minor lint fix
ammareltigani Aug 16, 2022
24acc3d
addressed comments
ammareltigani Aug 16, 2022
2e1cec7
Merge branch 'add-str-and-repr-to-mapping_manager' into routing-initi…
ammareltigani Aug 16, 2022
79061de
addressed comments
ammareltigani Aug 16, 2022
dfe80a9
fixed bug with edges not being sorted for graph equality testing
ammareltigani Aug 16, 2022
cb798dc
Merge branch 'add-str-and-repr-to-mapping_manager' into routing-initi…
ammareltigani Aug 16, 2022
6ee60a9
fixed bug with digraphs repr method in MappingManager and added test …
ammareltigani Aug 16, 2022
0c6de8b
Merge branch 'add-str-and-repr-to-mapping_manager' into routing-initi…
ammareltigani Aug 16, 2022
ecadfdb
addressed some comments
ammareltigani Aug 16, 2022
7819263
added grid testing device
ammareltigani Aug 17, 2022
5dd4577
added grid routing testing device
ammareltigani Aug 16, 2022
cc79064
formatting
ammareltigani Aug 16, 2022
fecc4b0
added line_initial_mapper and some tests; needs more testing
ammareltigani Aug 16, 2022
b3a1445
merged with device setup
ammareltigani Aug 17, 2022
9bcf54f
formatting
ammareltigani Aug 17, 2022
f4dae72
formatting
ammareltigani Aug 17, 2022
9ecb52a
changed interface for LineInitialMapper and added better tests; test …
ammareltigani Aug 17, 2022
719b284
addressed comments and added ring device
ammareltigani Aug 17, 2022
8cbbe8f
Merge branch 'routing-initial_mapping_device_setup' into routing-line…
ammareltigani Aug 17, 2022
735cba8
added test for supportin directed graphs
ammareltigani Aug 17, 2022
9ba9cef
changed interface for AbstractInitialMapper
ammareltigani Aug 17, 2022
211bb2d
Merge branch 'routing-initial_mapping_setup' into routing-line_initia…
ammareltigani Aug 17, 2022
82cdbc0
formatting
ammareltigani Aug 17, 2022
632dfe6
Merge branch 'master' into routing-initial_mapping_device_setup
ammareltigani Aug 19, 2022
fca9052
changed RoutingTestingDevice interface; need to change is_isomorphic …
ammareltigani Aug 19, 2022
34ef897
added hard-coded isomorphism tests
ammareltigani Aug 19, 2022
67546d8
fixed type issue
ammareltigani Aug 19, 2022
e120be2
Merge branch 'master' into routing-line_initial_mapper
ammareltigani Aug 19, 2022
0d2345c
removed redundant imports
ammareltigani Aug 19, 2022
209cc35
Merge branch 'routing-initial_mapping_device_setup' into routing-line…
ammareltigani Aug 19, 2022
2544069
merged with routing testing device PR #5830
ammareltigani Aug 19, 2022
4493826
simplified _value_equalit_values_
ammareltigani Aug 19, 2022
8827b91
addressed comments
ammareltigani Aug 19, 2022
70de81e
Merge branch 'routing-initial_mapping_device_setup' into routing-line…
ammareltigani Aug 19, 2022
60ed0ac
removed unused import
ammareltigani Aug 19, 2022
eefc089
Merge branch 'routing-initial_mapping_device_setup' into routing-line…
ammareltigani Aug 19, 2022
c7506bd
fixed nits
ammareltigani Aug 20, 2022
d92f515
Merge branch 'routing-initial_mapping_device_setup' into routing-line…
ammareltigani Aug 20, 2022
d41cedb
Merge branch 'master' into routing-line_initial_mapper
tanujkhattar Aug 20, 2022
7509aa9
addressed comments
ammareltigani Aug 23, 2022
c21eae2
formatting
ammareltigani Aug 23, 2022
854e867
Merge branch 'routing-line_initial_mapper' of https://github.com/amma…
ammareltigani Aug 23, 2022
bad20d3
small fixes
ammareltigani Aug 23, 2022
26f14bd
removed unused import
ammareltigani Aug 23, 2022
4170236
modified test file
ammareltigani Aug 23, 2022
6bed99a
debugging
ammareltigani Aug 23, 2022
5f4b848
removed print statements
ammareltigani Aug 23, 2022
e1de30a
debugging statement
ammareltigani Aug 23, 2022
3237bd7
debugging statement
ammareltigani Aug 24, 2022
64f6ba8
fix
ammareltigani Aug 24, 2022
a2d8d2d
fix
ammareltigani Aug 24, 2022
1b8c315
print statement
ammareltigani Aug 24, 2022
06ab64e
edges sorting
ammareltigani Aug 24, 2022
80b1f0e
addressed comments; ready for review
ammareltigani Aug 24, 2022
0f1454d
fixed type bug
ammareltigani Aug 24, 2022
7c4ebbd
cleanup
ammareltigani Aug 24, 2022
9835eb1
ready for review
ammareltigani Aug 25, 2022
c72de13
type and lint fixes
ammareltigani Aug 25, 2022
2ecb278
slightly modified _make_circuit_graph()
ammareltigani Aug 25, 2022
54e96ca
Merge branch 'master' into routing-line_initial_mapper
tanujkhattar Aug 25, 2022
6989acc
added test for testing valid circuits and fixed bug in _make_circuit_…
ammareltigani Aug 25, 2022
6c8a661
Merge branch 'routing-line_initial_mapper' of https://github.com/amma…
ammareltigani Aug 25, 2022
b6c4bb3
Merge branch 'master' into routing-line_initial_mapper
ammareltigani Aug 26, 2022
cfe8704
fixed nits
ammareltigani Aug 27, 2022
c69a0db
Merge branch 'routing-line_initial_mapper' of https://github.com/amma…
ammareltigani Aug 27, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions cirq-core/cirq/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -348,6 +348,7 @@
expand_composite,
HardCodedInitialMapper,
is_negligible_turn,
LineInitialMapper,
MappingManager,
map_moments,
map_operations,
Expand Down
1 change: 1 addition & 0 deletions cirq-core/cirq/protocols/json_test_data/spec.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,7 @@
# Routing utilities
'HardCodedInitialMapper',
'MappingManager',
'LineInitialMapper',
# global objects
'CONTROL_TAG',
'PAULI_BASIS',
Expand Down
8 changes: 7 additions & 1 deletion cirq-core/cirq/transformers/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,13 @@
two_qubit_gate_product_tabulation,
)

from cirq.transformers.routing import AbstractInitialMapper, HardCodedInitialMapper, MappingManager

from cirq.transformers.routing import (
AbstractInitialMapper,
HardCodedInitialMapper,
LineInitialMapper,
MappingManager,
)

from cirq.transformers.target_gatesets import (
create_transformer_with_kwargs,
Expand Down
1 change: 1 addition & 0 deletions cirq-core/cirq/transformers/routing/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,3 +16,4 @@

from cirq.transformers.routing.initial_mapper import AbstractInitialMapper, HardCodedInitialMapper
from cirq.transformers.routing.mapping_manager import MappingManager
from cirq.transformers.routing.line_initial_mapper import LineInitialMapper
191 changes: 191 additions & 0 deletions cirq-core/cirq/transformers/routing/line_initial_mapper.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,191 @@
# Copyright 2022 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.

"""Concrete implementation of AbstractInitialMapper that places lines of qubits onto the device."""
Comment thread
ammareltigani marked this conversation as resolved.
Outdated

from typing import Dict, List, Set, TYPE_CHECKING
import networkx as nx

from cirq.transformers.routing import AbstractInitialMapper
Comment thread
ammareltigani marked this conversation as resolved.
Outdated
from cirq import protocols

if TYPE_CHECKING:
import cirq


class LineInitialMapper(AbstractInitialMapper):
"""Places logical qubits in the circuit onto physical qubits on the device."""
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update docstring to reflect that specific strategy used by the LineInitialMapper.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I updated the docstring in initial_mapping(). Should I move it to the docstring for the class or keep it there?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a brief description to class docstring as well, highlighting high level details like an overview of strategy, expected complexity etc.


def __init__(self, device_graph: nx.Graph) -> None:
"""Initializes a LineInitialMapper.

Args:
device_graph: device graph
"""
self.device_graph = device_graph
self.mapped_physicals: Set['cirq.Qid'] = set()
self.partners: Dict['cirq.Qid', 'cirq.Qid'] = {}
Comment thread
ammareltigani marked this conversation as resolved.
Outdated

def _make_circuit_graph(self, circuit: 'cirq.AbstractCircuit') -> List[List['cirq.Qid']]:
"""Creates a (potentially incomplete) qubit connectivity graph of the circuit.

Iterates over moments in the circuit from left to right and adds edges between logical
qubits if the logical qubit pair l1 and l2
(1) have degree < 2,
(2) are involved in a 2-qubit operation in the current moment, and
(3) adding such an edge will not produce a cycle in the graph.

Args:
circuit: the input circuit with logical qubits

Returns:
The (potentially incomplete) qubit connectivity graph of the circuit, which is
guaranteed to be a forest of line graphs.
"""
circuit_graph: List[List['cirq.Qid']] = [[q] for q in sorted(circuit.all_qubits())]
Comment thread
ammareltigani marked this conversation as resolved.
Outdated
component_id: Dict['cirq.Qid', int] = {q[0]: i for i, q in enumerate(circuit_graph)}

def degree_lt_two(q: 'cirq.Qid'):
return any(circuit_graph[component_id[q]][i] == q for i in [-1, 0])

for op in circuit.all_operations():
if protocols.num_qubits(op) != 2:
continue

q0, q1 = op.qubits
c0, c1 = component_id[q0], component_id[q1]

# Keep track of partners for mapping isolated qubits later.
if q0 not in self.partners:
self.partners[q0] = q1
if q1 not in self.partners:
self.partners[q1] = q0

if not (degree_lt_two(q0) and degree_lt_two(q1) and c0 != c1):
continue

# Make sure c0/q0 are for the largest component.
if len(circuit_graph[c0]) < len(circuit_graph[c1]):
c0, c1, q0, q1 = c1, c0, q1, q0

# copy smaller component into larger one.
if circuit_graph[c0][0] == q0:
if circuit_graph[c1][0] == q1:
for q in circuit_graph[c1]:
circuit_graph[c0].insert(0, q)
Comment thread
ammareltigani marked this conversation as resolved.
Outdated
component_id[q] = c0
else:
for q in reversed(circuit_graph[c1]):
circuit_graph[c0].insert(0, q)
component_id[q] = c0
else:
if circuit_graph[c1][0] == q1:
for q in circuit_graph[c1]:
circuit_graph[c0].append(q)
component_id[q] = c0
else:
for q in reversed(circuit_graph[c1]):
circuit_graph[c0].append(q)
component_id[q] = c0

return sorted([circuit_graph[c] for c in set(component_id.values())], key=len, reverse=True)

def initial_mapping(self, circuit: 'cirq.AbstractCircuit') -> Dict['cirq.Qid', 'cirq.Qid']:
"""Maps disjoint lines of logical qubits onto lines of physical qubits.

Starting from the center physical qubit on the device, attempts to map disjoint lines of
logical qubits given by the circuit graph onto one long line of physical qubits on the
device, greedily maximizing each physical qubit's degree.
If this mapping cannot be completed as one long line of qubits in the circuit graph mapped
to qubits in the device graph, the line can be split as several line segments and then we:
(i) Map first line segment.
(ii) Find another high degree vertex in G near the center.
(iii) Map the second line segment
(iv) etc.
A line is split by mapping the next logical qubit to the nearest available physical qubit
to the center of the device graph.

Args:
circuit: the input circuit with logical qubits

Returns:
a dictionary that maps logical qubits in the circuit (keys) to physical qubits on the
device (values).
"""
qubit_map: Dict['cirq.Qid', 'cirq.Qid'] = {}
circuit_graph = self._make_circuit_graph(circuit)
physical_center = nx.center(self.device_graph)[0]
Comment thread
ammareltigani marked this conversation as resolved.
Outdated

def next_physical(current_physical: 'cirq.Qid') -> 'cirq.Qid':
# Greedily map to highest degree neighbor that that is available
Comment thread
ammareltigani marked this conversation as resolved.
Outdated
sorted_neighbors = sorted(
self.device_graph.neighbors(current_physical),
key=lambda x: self.device_graph.degree(x),
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we be sorting based on "unused degree", i.e. x > y if x has more unmapped neighbors than y?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll keep note of this change and try it during benchmarking!

reverse=True,
)
for neighbor in sorted_neighbors:
if neighbor not in self.mapped_physicals:
return neighbor
# If cannot map onto one long line of physical qubits, then break down into multiple
# small lines by finding nearest available qubit to the physical center
return self._closest_unmapped_qubit(physical_center)

pq = physical_center
first_isolated_idx = len(circuit_graph)
for idx, logical_line in enumerate(circuit_graph):
if len(logical_line) == 1:
first_isolated_idx = idx
break

for lq in logical_line:
self.mapped_physicals.add(pq)
qubit_map[lq] = pq
# Edge case: if mapping n qubits on an n-qubit device should not call next_physical
# when finished mapping the last logical qubit else will raise an error
if len(circuit.all_qubits()) != len(self.mapped_physicals):
Comment thread
ammareltigani marked this conversation as resolved.
Outdated
pq = next_physical(pq)

for i in range(first_isolated_idx, len(circuit_graph)):
lq = circuit_graph[i][0]
partner = qubit_map[self.partners[lq]] if lq in self.partners else physical_center
pq = self._closest_unmapped_qubit(partner)
self.mapped_physicals.add(pq)
qubit_map[lq] = pq

return qubit_map
Comment thread
tanujkhattar marked this conversation as resolved.

def _closest_unmapped_qubit(self, source: 'cirq.Qid') -> 'cirq.Qid':
"""Finds the closest available neighbor to a physical qubit 'source' on the device.

Args:
source: a physical qubit on the device.

Returns:
the closest available physical qubit to 'source'.

Raises:
ValueError: if there are no available qubits left on the device.
"""
for _, successors in nx.bfs_successors(self.device_graph, source):
for successor in successors:
if successor not in self.mapped_physicals:
return successor
raise ValueError("No available physical qubits left on the device.")

def __eq__(self, other) -> bool:
return nx.utils.graphs_equal(self.device_graph, other.device_graph)

def __repr__(self):
graph_type = type(self.device_graph).__name__
return f'cirq.LineInitialMapper(nx.{graph_type}({dict(self.device_graph.adjacency())}))'
117 changes: 117 additions & 0 deletions cirq-core/cirq/transformers/routing/line_initial_mapper_test.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,117 @@
# Copyright 2022 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.

import networkx as nx
import pytest

import cirq


def construct_small_circuit():
return cirq.Circuit(
[
cirq.Moment(cirq.CNOT(cirq.NamedQubit('1'), cirq.NamedQubit('3'))),
cirq.Moment(cirq.CNOT(cirq.NamedQubit('2'), cirq.NamedQubit('3'))),
cirq.Moment(
cirq.CNOT(cirq.NamedQubit('4'), cirq.NamedQubit('3')), cirq.X(cirq.NamedQubit('5'))
),
]
)


def construct_step_circuit(k: int):
q = cirq.LineQubit.range(k)
return cirq.Circuit([cirq.CNOT(q[i], q[i + 1]) for i in range(k - 1)])


def test_line_breaking_on_grid_device():
# tests
# -if strategy is able to map into several small lines if fails to map onto one long line
# -if # of physical qubits <= # of logical qubits then strategy should succeed

step_circuit = construct_step_circuit(49)
device = cirq.testing.construct_grid_device(7, 7)
device_graph = device.metadata.nx_graph
mapper = cirq.LineInitialMapper(device_graph)
mapping = mapper.initial_mapping(step_circuit)
mapped_circuit = step_circuit.transform_qubits(mapping)

# all qubits in the input circuit are placed on the device
assert set(mapping.keys()) == set(step_circuit.all_qubits())

# the first two moments are executable
print(mapped_circuit)
device.validate_circuit(mapped_circuit[:2])

# the induced graph of the device on the physical qubits in the map is connected
assert nx.is_connected(nx.induced_subgraph(device_graph, mapping.values()))

step_circuit = construct_step_circuit(50)
with pytest.raises(ValueError, match="No available physical qubits left on the device"):
mapper.initial_mapping(step_circuit)


def test_small_circuit_on_grid_device():
circuit = construct_small_circuit()

device_graph = cirq.testing.construct_grid_device(7, 7).metadata.nx_graph
mapper = cirq.LineInitialMapper(device_graph)
mapping = mapper.initial_mapping(circuit)

assert nx.center(device_graph)[0] == cirq.GridQubit(3, 3)
mapped_circuit = circuit.transform_qubits(mapping)
diagram = """(2, 2): ───@───────────
(2, 3): ───┼───────X───
(3, 2): ───X───X───X───
│ │
(3, 3): ───────@───┼───
(4, 2): ───────────@───"""
cirq.testing.assert_has_diagram(mapped_circuit, diagram)


@pytest.mark.parametrize(
"qubits, n_moments, op_density, random_state",
[
(5 * size, 20 * size, density, seed)
for size in range(1, 3)
for seed in range(10)
Comment thread
ammareltigani marked this conversation as resolved.
Outdated
for density in [0.4, 0.5, 0.6]
],
)
def test_random_circuits_grid_device(
qubits: int, n_moments: int, op_density: float, random_state: int
):
c_orig = cirq.testing.random_circuit(
qubits=qubits, n_moments=n_moments, op_density=op_density, random_state=random_state
)
device = cirq.testing.construct_grid_device(7, 7)
device_graph = device.metadata.nx_graph
mapper = cirq.LineInitialMapper(device_graph)
mapping = mapper.initial_mapping(c_orig)
c_mapped = c_orig.transform_qubits(mapping)

assert set(mapping.keys()) == set(c_orig.all_qubits())

device.validate_circuit(c_mapped[:2])

assert nx.is_connected(nx.induced_subgraph(device_graph, mapping.values()))


def test_repr():
device_graph = cirq.testing.construct_grid_device(7, 7).metadata.nx_graph
mapper = cirq.LineInitialMapper(device_graph)
cirq.testing.assert_equivalent_repr(mapper, setup_code='import cirq\nimport networkx as nx')