From b4b37eae36cda5557857a6b00cb9ac1339cf9997 Mon Sep 17 00:00:00 2001 From: "Stefan J. Wernli" Date: Thu, 13 Aug 2020 22:11:55 -0700 Subject: [PATCH 1/2] Fixing circuit in test_gates to be deterministic `test_gates` in capi_test.cpp had a low but non-zero chance of measuring One instead of Zero, resulting in a test crash due to assertion failure. This rearranges the instructions to ensure the test correctly unprepares the state so that measurement is as deterministic as possible. Fixes #262 --- src/Simulation/Native/src/simulator/capi_test.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/Simulation/Native/src/simulator/capi_test.cpp b/src/Simulation/Native/src/simulator/capi_test.cpp index feb725db2fa..0330b18c4e9 100644 --- a/src/Simulation/Native/src/simulator/capi_test.cpp +++ b/src/Simulation/Native/src/simulator/capi_test.cpp @@ -112,9 +112,8 @@ void test_gates() assert(M(sim_id, 1)==false); X(sim_id, 0); - CRx(sim_id, 1.0, 0, 1); - H(sim_id, 1); + CRx(sim_id, 1.0, 0, 1); CRx(sim_id, -1.0, 0, 1); H(sim_id, 1); From e04f86f975f01be1075978f70c6a7c5bcfe2dff9 Mon Sep 17 00:00:00 2001 From: "Stefan J. Wernli" Date: Thu, 13 Aug 2020 22:24:24 -0700 Subject: [PATCH 2/2] use a more interesting circuit --- src/Simulation/Native/src/simulator/capi_test.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/Simulation/Native/src/simulator/capi_test.cpp b/src/Simulation/Native/src/simulator/capi_test.cpp index 0330b18c4e9..82a2ad1b47a 100644 --- a/src/Simulation/Native/src/simulator/capi_test.cpp +++ b/src/Simulation/Native/src/simulator/capi_test.cpp @@ -112,9 +112,10 @@ void test_gates() assert(M(sim_id, 1)==false); X(sim_id, 0); + CRx(sim_id, 1.0, 0, 1); + H(sim_id, 1); - CRx(sim_id, 1.0, 0, 1); - CRx(sim_id, -1.0, 0, 1); + CRz(sim_id, -1.0, 0, 1); H(sim_id, 1); assert(M(sim_id, 1)==false);