Skip to content

Commit f421fe1

Browse files
committed
Add a note about moving the merge tests to NetQASM API
1 parent 3cad396 commit f421fe1

File tree

1 file changed

+16
-0
lines changed

1 file changed

+16
-0
lines changed

tests/slow/merges/test_merges.py

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,22 @@
2727

2828
_logger = logging.getLogger("test_merges")
2929

30+
# Diego Rivera: 2026-01-16:
31+
# Note about moving these tests to NetQASM API.
32+
# NetQASM is a library that needs to be implemented by the simulators. It also
33+
# aims to be a higher-level abstraction layer, so it does not expose implementation
34+
# details from the underlying quantum simulator.
35+
# On the other hand, these tests aim to check the functionality of merging two
36+
# qubits into a single register. This functionality *is specific to the SimulaQron
37+
# simulator*, since it is required to allow quantum simulation distributed in
38+
# multiple hosts.
39+
# Being this said, I see two main difficulties to move these tests to NetQASM API:
40+
# * Since NetQASM aims to hide all implementation details, there is no NetQASM
41+
# primitive to express "moving a qubit from one host to another".
42+
# * The functionality tested here is SimulaQron-specific. It would be difficult
43+
# to rewrite these tests in NetQASM, which might hide APIs needed to fully test
44+
# the underlying functionality.
45+
3046

3147
class localNode(pb.Root):
3248
def __init__(self, node, classicalNet):

0 commit comments

Comments
 (0)