Skip to content

Commit 29b7dea

Browse files
author
Corey Ostrove
committed
Bugfix for #569 and generator infidelity bug.
Fixes a bug in the germs details tables reported in #569. The issue was that the wrong model was being pulled from the switchboard as the target model. Also includes and unrelated fix for the generator infidelity in reports which was accidentally pulling in the average gate infidelity.
1 parent a895c8a commit 29b7dea

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

pygsti/report/reportables.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1634,7 +1634,7 @@ def generator_infidelity(a, b, mx_basis):
16341634

16351635
return _tools.generator_infidelity(a, b, mx_basis)
16361636

1637-
Generator_infidelity = _modf.opsfn_factory(avg_gate_infidelity)
1637+
Generator_infidelity = _modf.opsfn_factory(generator_infidelity)
16381638
# init args == (model1, model2, op_label)
16391639

16401640
def model_model_angles_btwn_axes(a, b, mx_basis): # Note: default 'gm' basis

pygsti/report/section/gauge.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,15 +125,15 @@ class GaugeInvariantsGermsSection(_Section):
125125
def final_gates_vs_target_table_gauge_invgerms(workspace, switchboard=None, confidence_level=None,
126126
ci_brevity=1, **kwargs):
127127
return workspace.GatesVsTargetTable(
128-
switchboard.mdl_gaugeinv, switchboard.mdl_gaugeinv_ep,
128+
switchboard.mdl_gaugeinv, switchboard.mdl_target,
129129
_cri_gauge_inv(0, switchboard, confidence_level, ci_brevity),
130130
display=('evdiamond', 'evnudiamond'), virtual_ops=switchboard.germs
131131
)
132132

133133
@_Section.figure_factory(3)
134134
def germs_eigenvalue_table(workspace, switchboard=None, confidence_level=None, ci_brevity=1, **kwargs):
135135
return workspace.GateEigenvalueTable(
136-
switchboard.mdl_gaugeinv, switchboard.mdl_gaugeinv_ep,
136+
switchboard.mdl_gaugeinv, switchboard.mdl_target,
137137
_cri_gauge_inv(1, switchboard, confidence_level, ci_brevity),
138138
display=('evals', 'target', 'absdiff-evals', 'infdiff-evals', 'log-evals', 'absdiff-log-evals'),
139139
virtual_ops=switchboard.germs

0 commit comments

Comments
 (0)