better-rtplot 0.4.6__tar.gz → 0.4.7__tar.gz

{better_rtplot-0.4.6 → better_rtplot-0.4.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: better-rtplot
-Version: 0.4.6
+Version: 0.4.7
 Summary: 
 License: GPL V3.0
 Author: jmontp

{better_rtplot-0.4.6 → better_rtplot-0.4.7}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "better-rtplot"
-version = "0.4.6"
+version = "0.4.7"
 description = ""
 authors = ["jmontp <jmontp@umich.edu>"]
 license = "GPL V3.0"

{better_rtplot-0.4.6 → better_rtplot-0.4.7}/rtplot/server_browser.py

@@ -306,6 +306,12 @@ class Tab:
     data_rate_hz: float = 0.0
     _last_rx_ts: float = 0.0
 
+    # When the most recent client-supplied config failed to parse we
+    # stash the reason + a unix timestamp so the browser can surface
+    # "your last config was rejected at HH:MM:SS" instead of just
+    # silently rendering nothing. Cleared on the next successful parse.
+    last_config_error: Optional[dict] = None
+
     def ensure_buffer(self):
         if self.buffer is None:
             self.buffer = np.zeros((INITIAL_NUM_TRACES, MAX_LOCAL_STORAGE))
@@ -344,6 +350,7 @@ def tab_public(t: Tab) -> dict:
         "endpoint": t.endpoint,
         "status": t.status,
         "error": t.error,
+        "last_config_error": t.last_config_error,
     }
 
 
@@ -711,8 +718,30 @@ async def zmq_receiver(tab: Tab):
         tab._last_rx_ts = now
 
         if category == RECEIVED_PLOT_UPDATE:
-            cfg = await sock.recv_json(object_pairs_hook=OrderedDict)
-            parse_config(tab, cfg)
+            # Decode + parse are wrapped so a malformed config doesn't kill
+            # the receiver task. We stash the failure on the tab so the
+            # browser can show "your last config was rejected at HH:MM:SS"
+            # instead of looking like nothing happened. Existing valid
+            # state (if any) is preserved on parse failure.
+            try:
+                cfg = await sock.recv_json(object_pairs_hook=OrderedDict)
+            except Exception as exc:  # noqa: BLE001
+                msg = f"Could not decode config JSON: {type(exc).__name__}: {exc}"
+                print(f"[{tab.id}] {msg}")
+                tab.last_config_error = {"message": msg, "timestamp": time.time()}
+                await broadcast_tab(tab.id)
+                continue
+
+            try:
+                parse_config(tab, cfg)
+            except Exception as exc:  # noqa: BLE001
+                msg = f"Configuration rejected: {type(exc).__name__}: {exc}"
+                print(f"[{tab.id}] {msg}")
+                tab.last_config_error = {"message": msg, "timestamp": time.time()}
+                await broadcast_tab(tab.id)
+                continue
+
+            tab.last_config_error = None  # clear: this one was good
             tab.config_dict = cfg
             tab.config_message = build_config_message(tab, cfg)
             tab.initialized = True

{better_rtplot-0.4.6 → better_rtplot-0.4.7}/rtplot/static/index.html

@@ -67,6 +67,13 @@
   .plot-wrap { background: #fff; border: 1px solid #ddd; border-radius: 4px; padding: 8px; flex: 1 1 auto; min-width: 320px; }
   .plot-title { font-size: 13px; font-weight: 600; margin: 0 0 4px 4px; color: #333; }
   #empty-state { padding: 28px 32px; color: #888; font-size: calc(14px * var(--ui-scale)); font-style: italic; }
+  /* Persistent banner the server raises when a client-supplied plot
+     config fails to parse — the user used to be left guessing why
+     nothing rendered; now they get the reason + a local timestamp. */
+  .config-error-banner { margin: 12px 12px 0 12px; padding: 10px 14px; background: #fce8e8; color: #8a1a1a; border: 1px solid #f0c2c2; border-left: 4px solid #d24a4a; border-radius: 4px; font-size: calc(13px * var(--ui-scale)); }
+  .config-error-banner .ce-title { font-weight: 600; margin-right: 8px; }
+  .config-error-banner .ce-when { color: #6b1818; opacity: 0.8; font-family: monospace; font-size: calc(12px * var(--ui-scale)); }
+  .config-error-banner .ce-msg { display: block; margin-top: 4px; font-family: monospace; font-size: calc(12px * var(--ui-scale)); white-space: pre-wrap; word-break: break-word; }
   :root { --ctrl-unit-h: 38px; --ui-scale: 1; }
   .ctrl-row { display: flex; gap: 12px; align-items: center; padding: 10px 14px; background: #fff; border: 1px solid #ddd; border-radius: 4px; flex-wrap: wrap; }
   .ctrl-item { display: flex; align-items: center; gap: 6px; }
@@ -187,6 +194,7 @@
       <div id="res-unavail" class="res-unavail" style="display:none;">psutil not installed &mdash; pip install psutil for live stats</div>
     </div>
   </div>
+  <div id="config-error-banner" class="config-error-banner" style="display:none"></div>
   <div id="plots" class="row"></div>
   <script src="/static/uPlot.iife.min.js"></script>
   <script src="/static/katex/katex.min.js"></script>
@@ -210,6 +218,43 @@
         return 'rgb(0,0,0)';
       }
 
+      const configErrorBanner = document.getElementById('config-error-banner');
+
+      function fmtLocalTimestamp(unixSeconds) {
+        const d = new Date(unixSeconds * 1000);
+        const pad = (n) => String(n).padStart(2, '0');
+        return `${d.getFullYear()}-${pad(d.getMonth() + 1)}-${pad(d.getDate())} ` +
+               `${pad(d.getHours())}:${pad(d.getMinutes())}:${pad(d.getSeconds())}`;
+      }
+
+      function applyConfigErrorBanner() {
+        const t = activeTab && knownTabs[activeTab];
+        const err = t && t.last_config_error;
+        if (!err) {
+          configErrorBanner.style.display = 'none';
+          configErrorBanner.innerHTML = '';
+          return;
+        }
+        // Build with textContent on the leaf nodes so user-controlled
+        // strings (the error message) can't inject HTML.
+        configErrorBanner.innerHTML = '';
+        const head = document.createElement('div');
+        const title = document.createElement('span');
+        title.className = 'ce-title';
+        title.textContent = 'Last configuration was rejected';
+        head.appendChild(title);
+        const when = document.createElement('span');
+        when.className = 'ce-when';
+        when.textContent = `at ${fmtLocalTimestamp(err.timestamp)}`;
+        head.appendChild(when);
+        configErrorBanner.appendChild(head);
+        const body = document.createElement('span');
+        body.className = 'ce-msg';
+        body.textContent = err.message || '';
+        configErrorBanner.appendChild(body);
+        configErrorBanner.style.display = '';
+      }
+
       // ---- LaTeX rendering ($...$ inline, $$...$$ display) ----
       // KaTeX's auto-render walks an element's text nodes looking for the
       // configured delimiters, so plain labels without `$` are completely
@@ -1111,6 +1156,7 @@
         showEmptyState('Loading tab \u2026');
         const t = knownTabs[id];
         setZmqMode(t.mode, t.endpoint);
+        applyConfigErrorBanner();
         sendCtrl({ type: 'tab_subscribe', id: id });
       }
 
@@ -1183,6 +1229,7 @@
               if (activeTab) {
                 const t = knownTabs[activeTab];
                 setZmqMode(t.mode, t.endpoint);
+                applyConfigErrorBanner();
                 sendCtrl({ type: 'tab_subscribe', id: activeTab });
                 showEmptyState('Loading tab \u2026');
               }
@@ -1193,6 +1240,7 @@
                 renderTabs();
                 if (msg.tab.id === activeTab) {
                   setZmqMode(msg.tab.mode, msg.tab.endpoint);
+                  applyConfigErrorBanner();
                 }
               }
             } else if (msg.type === 'tab_removed') {