From e99047c29565c19c5c15e72c702d5c3aeaabba3d Mon Sep 17 00:00:00 2001
From: erikkaashoek <erik@kaashoek.com>
Date: Wed, 2 Jun 2021 08:17:20 +0200
Subject: [PATCH] New calibration method and TRACE AVERAGE markers

---
 nanovna.h |  6 +++---
 plot.c    |  9 ++++++++-
 sa_core.c | 24 +++++++++++++++++++-----
 ui_sa.c   |  1 +
 4 files changed, 31 insertions(+), 9 deletions(-)

diff --git a/nanovna.h b/nanovna.h
index 921981b..06e3915 100644
--- a/nanovna.h
+++ b/nanovna.h
@@ -18,7 +18,7 @@
  */
 #include "ch.h"
 
-#ifdef TINYSA_F303
+//#ifdef TINYSA_F303
 #include "adc_F303.h"
 #ifdef TINYSA_F072
 #error "Remove comment for #ifdef TINYSA_F303"
@@ -27,7 +27,7 @@
 #define TINYSA4
 #endif
 #define TINYSA4_PROTO
-#endif
+//#endif
 
 #ifdef TINYSA_F072
 #ifdef TINYSA_F303
@@ -685,7 +685,7 @@ int  shell_serial_printf(const char *fmt, ...);
 
 // marker
 enum {
-  M_NORMAL=0,M_REFERENCE=1, M_DELTA=2, M_NOISE=4, M_STORED=8, M_TRACKING=16, M_DELETE=32  // Tracking must be last.
+  M_NORMAL=0,M_REFERENCE=1, M_DELTA=2, M_NOISE=4, M_STORED=8, M_AVER=16, M_TRACKING=32, M_DELETE=64  // Tracking must be last.
 };
 
 enum {
diff --git a/plot.c b/plot.c
index 1eb69ce..7a15060 100644
--- a/plot.c
+++ b/plot.c
@@ -288,7 +288,14 @@ marker_to_value(const int i)
     ref_marker_levels = stored_t;
   else
     ref_marker_levels = actual_t;
-  return(value(ref_marker_levels[markers[i].index]));
+  float v = value(ref_marker_levels[markers[i].index]);
+  if (markers[i].mtype & M_AVER) {
+    v = 0;
+    for (int i=0; i<sweep_points; i++)
+      v += value(ref_marker_levels[i]);
+    v /= sweep_points;
+  }
+  return(v);
 }
 
 #ifdef TINYSA3
diff --git a/sa_core.c b/sa_core.c
index e358c9e..599a2a7 100644
--- a/sa_core.c
+++ b/sa_core.c
@@ -5866,7 +5866,6 @@ void calibrate(void)
 {
   int local_test_status;
   int old_sweep_points = setting._sweep_points;
-  in_selftest = true;
 #ifdef TINYSA4
   setting.test_argument = -7;
   self_test(0);
@@ -5877,13 +5876,25 @@ void calibrate(void)
   }
 #endif
   reset_calibration();
-  reset_settings(M_LOW);
 #ifdef TINYSA4
   bool calibrate_lna = false;
 again:
 #endif
   for (int k = 0; k<2; k++) {
     for (int j= 0; j < CALIBRATE_RBWS; j++ ) {
+#if 1
+      reset_settings(M_LOW);
+      set_refer_output(0);
+      set_sweep_frequency(ST_STOP, 60000000);
+      int test_case = TEST_POWER;
+      set_extra_lna(calibrate_lna);
+      set_average(AV_100);
+      for (int m=1; m<20; m++) {
+        test_acquire(test_case);                        // Acquire test
+        local_test_status = test_validate(test_case);
+      }
+      local_test_status = TS_PASS;
+#else
       //    set_RBW(power_rbw[j]);
       //    set_sweep_points(21);
 #if 0
@@ -5920,6 +5931,7 @@ again:
       test_acquire(test_case);                        // Acquire test
       test_acquire(test_case);                        // Acquire test
       local_test_status = test_validate(test_case);                       // Validate test also sets attenuation if zero span
+#endif
 #endif
       if (k ==0 || k == 1) {
         if (peakLevel < -50) {
@@ -5972,7 +5984,6 @@ again:
   }
 
 #endif
-
   config_save();
   ili9341_set_foreground(LCD_BRIGHT_COLOR_GREEN);
   ili9341_drawstring_7x13("Calibration complete", 40, 140);
@@ -5983,8 +5994,11 @@ quit:
   set_sweep_points(old_sweep_points);
   in_selftest = false;
   sweep_mode = SWEEP_ENABLE;
-  set_refer_output(-1);
-  reset_settings(M_LOW);
+//  set_refer_output(-1);
+//  reset_settings(M_LOW);
+  set_extra_lna(false);
+  set_average(AV_OFF);
+
 }
 
 
diff --git a/ui_sa.c b/ui_sa.c
index f9eb51d..ba7b1ec 100644
--- a/ui_sa.c
+++ b/ui_sa.c
@@ -2286,6 +2286,7 @@ const menuitem_t menu_marker_modify[] = {
   { MT_ADV_CALLBACK, M_NOISE,       "NOISE",    menu_marker_modify_acb},
   { MT_ADV_CALLBACK, M_TRACKING,    "TRACKING", menu_marker_modify_acb},
   { MT_ADV_CALLBACK, M_STORED,      "STORED",   menu_marker_modify_acb},
+  { MT_ADV_CALLBACK, M_AVER,      "TRACE\nAVERAGE",   menu_marker_modify_acb},
   { MT_SUBMENU,  0,                 "SEARCH",   menu_marker_search},
   { MT_CALLBACK, M_DELETE,          "DELETE",   menu_marker_delete_cb},
   { MT_CANCEL,   0,          S_LARROW" BACK", NULL },