From 8fed60e73b985a12fac5ddb8a50652014ae1b386 Mon Sep 17 00:00:00 2001 From: Dawn Date: Mon, 10 Aug 2026 09:45:07 +0200 Subject: [PATCH] docs: note the spreadsheet log-spam TODO in daq_worker load_spreadsheet/load_reference_tools log a GET they never send every ~12.5s poll while base_url is None; fix later by demoting to debug or logging once on the None->set edge. Co-Authored-By: Claude Fable 5 --- src/aare/gui/threads/daq_worker.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/aare/gui/threads/daq_worker.py b/src/aare/gui/threads/daq_worker.py index ca0e7765..e3dcb076 100644 --- a/src/aare/gui/threads/daq_worker.py +++ b/src/aare/gui/threads/daq_worker.py @@ -1166,6 +1166,11 @@ class DAQWorker(QObject): @Slot() def load_spreadsheet(self): if self._base_url is None: + # TODO: log spam. This (and load_reference_tools) fires every + # SPREADHSEET_FREQUENCY cycle (~12.5s) while base_url is None, + # logging a GET it never actually sends -> two INFO lines every + # poll. Fix by demoting to logger.debug, or log once on the + # None->set edge rather than on every poll. logger.info("GET /sample/spreadsheet") return