From 7376769c41ef505f2844c70887d8d7c17dae83ff Mon Sep 17 00:00:00 2001 From: jupfi Date: Wed, 16 Aug 2023 13:10:18 +0200 Subject: [PATCH] Fixed bug with displaying of phase. --- src/nqrduck_autotm/view.py | 20 ++++++++++++++++++-- 1 file changed, 18 insertions(+), 2 deletions(-) diff --git a/src/nqrduck_autotm/view.py b/src/nqrduck_autotm/view.py index 280cb69..7f41428 100644 --- a/src/nqrduck_autotm/view.py +++ b/src/nqrduck_autotm/view.py @@ -154,6 +154,10 @@ class AutoTMView(ModuleView): magnitude_ax = self._ui_form.S11Plot.canvas.ax magnitude_ax.clear() + + phase_ax = self._ui_form.S11Plot.canvas.ax.twinx() + phase_ax.clear() + # @ TODO: implement proper calibration if self.module.model.calibration is not None: # Calibration test: @@ -176,8 +180,20 @@ class AutoTMView(ModuleView): return_loss_db_corr = [-20 * cmath.log10(abs(g + 1e-12)) for g in gamma_corr] magnitude_ax.plot(frequency, return_loss_db_corr, color="red") - phase_ax = self._ui_form.S11Plot.canvas.ax.twinx() - phase_ax.clear() + """ open_calibration = self.module.model.open_calibration + short_calibration = self.module.model.short_calibration + load_calibration = self.module.model.load_calibration + + phase_difference = short_calibration.phase_deg - open_calibration.phase_deg + phase = (phase - open_calibration.phase_deg) / phase_difference + + amplitude_difference = open_calibration.return_loss_db - load_calibration.return_loss_db + amplitude = (return_loss_db - open_calibration.return_loss_db) / amplitude_difference + + magnitude_ax.plot(frequency, amplitude, color="green") + phase_ax.plot(frequency, phase, color="orange", linestyle="--") """ + + phase_ax.set_ylabel("|Phase (deg)|") phase_ax.plot(frequency, phase, color="orange", linestyle="--") phase_ax.set_ylim(-180, 180)