@meri-imperiumi/signalk-dead-reckoning 0.12.1 → 0.12.3

package/CHANGELOG.md

@@ -7,6 +7,40 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.12.3] - 2026-10-03
+
+### Fixed
+- **Tier-3 weather current now parses the fields providers actually
+  serve.** `parseWeatherCurrent` read the REST docs' shape
+  (`current.set` rad / `current.drift` m/s), but the Weather API server
+  transforms nothing — both the JS API (`app.weatherApi.getForecasts`)
+  and REST return the registered provider's data as-is — and the
+  providers on this install never emit that shape: the default GRIB
+  provider carries no currents at all, and signalk-weather-router-plus
+  answers `water.surfaceCurrentDirection` (rad, the set, towards) /
+  `water.surfaceCurrentSpeed` (m/s), the shape of `@signalk/server-api`'s
+  TS `WeatherData` interface (which contradicts the docs' TypeBox
+  `WeatherDataModel`). Every poll therefore died with "forecast carries
+  no current data" and DR silently ran on derived/zero current. The
+  parser now accepts both shapes (water.* preferred), verified against
+  the live server; wind-only entries without current coverage are
+  skipped so interpolation brackets between the entries that carry
+  current.
+
+## [0.12.2] - 2026-09-28
+
+### Fixed
+- **The top-right plotter-extension area now behaves as a top-right
+  anchor.** Two defects made it read as "top left of the GPS status
+  bar": the area rode its column's left edge instead of the GPS
+  panel's right edge (it now shares the panel's right alignment), and
+  the widget grid rendered logical col 0 at the physical left for
+  every anchor — so the first placed widget sat farthest from the
+  viewport corner with the free slots between it and the edge.
+  Right-hand anchors now mirror the column axis: col 0 hugs the
+  right-hand corner and widgets pack corner-inward, symmetric with
+  the top-left area.
+
 ## [0.12.1] - 2026-09-28
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@meri-imperiumi/signalk-dead-reckoning",
-  "version": "0.12.1",
+  "version": "0.12.3",
   "description": "Offline-first dead reckoning and sensor fusion engine for Signal K",
   "main": "plugin/index.js",
   "scripts": {
@@ -27,7 +27,7 @@
   "signalk-plugin-enabled-by-default": false,
   "signalk": {
     "displayName": "Dead Reckoning",
-    "appIcon": "./icon.png",
+    "appIcon": "./icon-256.png",
     "requires": [
       "@meri-imperiumi/signalk-internet"
     ],

package/plugin/current.js

@@ -13,11 +13,17 @@
  *     observation outranks model products (sea trial 2026-08-30→09-05:
  *     ~11 nm DR error over 622 nm vs 47 nm for tier 3).
  *  3. **Signal K Weather API** (`/signalk/v2/api/weather/forecasts/point`)
- *     — a provider (e.g. a GRIB another process already downloaded)
- *     serves point forecasts carrying `current: {set (rad), drift (m/s)}`.
- *     The client polls it at the vessel position on a slow interval and
- *     caches the time-interpolated vector; the 1 Hz tick never touches
- *     the network.
+ *     — a weather provider carrying currents (e.g. signalk-weather-router-
+ *     plus: Copernicus SMOC) answers point forecasts; the client polls it
+ *     at the vessel position on a slow interval and caches the
+ *     time-interpolated vector; the 1 Hz tick never touches the network.
+ *     Current fields follow the provider: the REST docs' TypeBox model
+ *     defines top-level `current.set` (rad, towards) / `current.drift`
+ *     (m/s), while `@signalk/server-api`'s TS WeatherData and the real
+ *     providers use `water.surfaceCurrentDirection` /
+ *     `water.surfaceCurrentSpeed`. The server transforms nothing — the
+ *     JS API and REST both return provider data as-is — so the parser
+ *     accepts both shapes.
  *  4. Offline pilot charts (`offline_pilot_currents`) — table exists in
  *     the schema; the lookup hook is reserved.
  *  5. **Zero vector** — pure inertial water track.
@@ -44,8 +50,11 @@ function normalizeDeg360(deg) {
 
 /**
  * Parses a Weather API forecast series into the current vector valid at
- * `nowMs`. Entries carrying `current.set` (rad, direction the current
- * flows *toward*) and `current.drift` (m/s) are interpolated as u/v
+ * `nowMs`. Entries carrying current data — `water.surfaceCurrentDirection`
+ * (rad, the direction the current flows *toward*) and
+ * `water.surfaceCurrentSpeed` (m/s), or the REST docs' top-level
+ * `current.set` / `current.drift` (same units, same semantics) — are
+ * interpolated as u/v
  * components between the entries bracketing `nowMs`; outside the series
  * the nearest endpoint is used (the client's TTL bounds staleness).
  *
@@ -59,8 +68,13 @@ function parseWeatherCurrent(points, nowMs) {
   const entries = [];
   for (const p of points) {
     const t = Date.parse(p?.date ?? "");
-    const set = p?.current?.set;
-    const drift = p?.current?.drift;
+    // Two shapes exist in the wild: the REST docs' TypeBox model (top-
+    // level current.set/drift) and the server-api TS WeatherData interface
+    // plus real providers (water.surfaceCurrent*). The server serves
+    // provider data untransformed on both the JS API and REST, so accept
+    // either; prefer the water.* fields.
+    const set = p?.water?.surfaceCurrentDirection ?? p?.current?.set;
+    const drift = p?.water?.surfaceCurrentSpeed ?? p?.current?.drift;
     if (!Number.isFinite(t)) continue;
     if (!Number.isFinite(set) || !Number.isFinite(drift)) continue;
     entries.push({ t, set, drift });
@@ -184,8 +198,14 @@ function resolveCurrent(input = {}) {
  * `/signalk/...` path. On servers without the Weather API the client
  * stays idle and the §6.2 resolver never sees tier 3.
  *
- * The response is a WeatherDataModel array; `current.set` is radians
- * and `current.drift` m/s (converted here to deg/kn).
+ * The response is a WeatherData array (the Signal K Weather API model).
+ * Currents appear either as `water.surfaceCurrentDirection` (rad, the
+ * set, towards) / `water.surfaceCurrentSpeed` (m/s) — what real
+ * providers like signalk-weather-router-plus serve — or as the REST
+ * docs' `current.set` / `current.drift` (same units); both are
+ * converted here to deg/kn. Entries without current coverage (e.g.
+ * wind-only steps, or points outside the provider's current sources)
+ * simply lack the fields and are skipped.
  */
 class WeatherCurrentClient {
   /**

package/plugin/index.js

@@ -830,7 +830,7 @@ module.exports = (app) => {
           type: "boolean",
           title: "Use Signal K Weather API current (set/drift) for DR",
           description:
-            "Polls the server's weather provider (typically a GRIB another process downloaded) for the point-forecast current at the vessel position and integrates it into the DR solution (SPEC §6.2 tier 3). Falls back to the zero vector when unavailable.",
+            "Polls the server's weather provider (e.g. signalk-weather-router-plus, serving ECMWF wind and Copernicus SMOC currents) for the point-forecast current at the vessel position and integrates it into the DR solution (SPEC §6.2 tier 3). Falls back to the zero vector when unavailable.",
           default: DEFAULT_CONFIG.weatherCurrent.enabled,
         },
         "weatherCurrent.intervalMs": {
@@ -958,8 +958,9 @@ module.exports = (app) => {
           : seedTs;
 
       // §6.2 tier 3: poll the Signal K Weather API for the point-forecast
-      // current at the vessel position (typically a GRIB another process
-      // downloaded). Off the 1 Hz hot path: slow interval, cached read.
+      // current at the vessel position (a current-capable provider such as
+      // signalk-weather-router-plus). Off the 1 Hz hot path: slow interval,
+      // cached read.
       if (config.weatherCurrent.enabled) {
         // In-process Weather API (mirrors signalk-energy-predictor):
         // `app.weatherApi` is the same instance the REST routes wrap,

package/public/dr-app.js

@@ -256,7 +256,13 @@ template.innerHTML = /* html */ `
       gap: 0.5rem;
       align-items: flex-start;
     }
-    .dr-top-col > .dr-gps {
+    /* The right column is as wide as its widest child (usually the GPS
+       panel), so without this both the panel and the top-right widget
+       area would need explicit right edges: the panel rides the right
+       edge, and the top-right area must follow it — left-aligned it
+       would park under the panel's left half, reading as "top left". */
+    .dr-top-col > .dr-gps,
+    .dr-top-col > dr-ext-widget-area[anchor="top-right"] {
       align-self: flex-end;
     }
     /* Water-track readout: bottom-right panel — figures + the manual

package/public/dr-ext-host-core.js

@@ -151,6 +151,38 @@ export function occupiedCells(placements) {
   return occ;
 }
 
+/**
+ * Do placements in this anchor mirror their column axis? Right-hand
+ * anchors render mirrored: logical col 0 is the viewport corner (the
+ * RIGHT edge), so widgets pack corner-inward — symmetric with left
+ * anchors, where col 0 is the left edge. Without the mirror the first
+ * placed widget lands in the cell farthest from the corner and the
+ * free space sits between it and the edge.
+ *
+ * @param {string} anchor
+ * @returns {boolean}
+ */
+export function isMirroredAnchor(anchor) {
+  return typeof anchor === "string" && anchor.endsWith("-right");
+}
+
+/**
+ * CSS grid column line (1-based, physical left→right) where a
+ * placement's left edge renders for the given anchor. Mirrored
+ * (right-hand) anchors flip the column axis: a placement at logical
+ * column `col` spans lines `GRID_COLS - col - cols + 1 ..
+ * GRID_COLS - col`.
+ *
+ * @param {string} anchor
+ * @param {number} col - 0-based logical column (0 = the anchor corner)
+ * @param {number} cols - placement column span
+ * @returns {number} 1-based CSS grid column line
+ */
+export function gridColLine(anchor, col, cols) {
+  if (isMirroredAnchor(anchor)) return GRID_COLS - col - cols + 1;
+  return col + 1;
+}
+
 /**
  * First position (packing from the grid origin, row-major) where a
  * widget of this size fits without overlapping existing placements.

package/public/dr-ext-host.js

@@ -1032,7 +1032,11 @@ export class DrExtWidgetArea extends AreaBase {
         const size = core.parseSize(p.size) ?? { cols: 1, rows: 1 };
         cell = document.createElement("div");
         cell.className = "cell";
-        cell.style.gridArea = `${p.row + 1} / ${p.col + 1} / span ${size.rows} / span ${size.cols}`;
+        // Column line via core.gridColLine: right-hand anchors mirror
+        // the column axis so col 0 renders at the viewport corner.
+        cell.style.gridArea =
+          `${p.row + 1} / ${core.gridColLine(this.anchor, p.col, size.cols)} ` +
+          `/ span ${size.rows} / span ${size.cols}`;
         this.grid.appendChild(cell);
         this.cells.set(p.instanceId, cell);
       }

package/tests/current.test.js

@@ -16,11 +16,11 @@ const {
 
 const NOW = Date.UTC(2026, 6, 15, 12, 0, 0);
 
-/** forecast entry helper: set rad toward, drift m/s */
+/** forecast entry helper: set rad toward, drift m/s (SK Weather API shape) */
 function entry(offsetMin, setRad, driftMs) {
   return {
     date: new Date(NOW + offsetMin * 60000).toISOString(),
-    current: { set: setRad, drift: driftMs },
+    water: { surfaceCurrentDirection: setRad, surfaceCurrentSpeed: driftMs },
   };
 }
 
@@ -76,6 +76,47 @@ test("parseWeatherCurrent: null when no entry carries current data", () => {
   assert.strictEqual(parseWeatherCurrent(null, NOW), null);
 });
 
+test("parseWeatherCurrent: entries without current data are skipped, not fatal", () => {
+  // Real providers emit wind-only entries too (e.g. ECMWF steps outside
+  // SMOC coverage, or points the current sources do not reach). Those
+  // entries must be skipped so interpolation still brackets `nowMs`
+  // between the entries that carry current.
+  const r = parseWeatherCurrent(
+    [
+      {
+        date: new Date(NOW - 60 * 60000).toISOString(),
+        wind: { speedTrue: 5 },
+      },
+      entry(0, 0, 1),
+      entry(60, Math.PI / 2, 1),
+    ],
+    NOW + 30 * 60000,
+  );
+  assert.ok(r);
+  assert.ok(r.setTrue > 0 && r.setTrue < 90, `setTrue ${r?.setTrue}`);
+});
+
+test("parseWeatherCurrent: accepts the REST docs' current.set/drift shape too", () => {
+  // @signalk/server-api ships two conflicting models: the TypeBox
+  // WeatherDataModel behind the REST docs (top-level current.set/drift)
+  // and the TS WeatherData interface (water.surfaceCurrent*). The server
+  // serves provider data untransformed, so both must parse identically.
+  const docShape = (offsetMin, setRad, driftMs) => ({
+    date: new Date(NOW + offsetMin * 60000).toISOString(),
+    current: { set: setRad, drift: driftMs },
+  });
+  const fromWater = parseWeatherCurrent(
+    [entry(-60, 0, 1), entry(60, Math.PI / 2, 1)],
+    NOW,
+  );
+  const fromDoc = parseWeatherCurrent(
+    [docShape(-60, 0, 1), docShape(60, Math.PI / 2, 1)],
+    NOW,
+  );
+  assert.ok(fromDoc);
+  assert.deepStrictEqual(fromDoc, fromWater);
+});
+
 // --------------------------------------------------------------- resolver
 
 test("resolveCurrent: tier 5 zero vector when nothing better exists", () => {

package/tests/dr-ext-host-core.test.js

@@ -101,6 +101,23 @@ test("firstFreeSlot: packs row-major, skips overlaps", () => {
   assert.equal(core.firstFreeSlot(full, "1x1"), null);
 });
 
+test("gridColLine: right-hand anchors mirror the column axis", () => {
+  // Left anchor: col 0 is the leftmost CSS column (the corner).
+  assert.equal(core.isMirroredAnchor("top-left"), false);
+  assert.equal(core.gridColLine("top-left", 0, 1), 1);
+  assert.equal(core.gridColLine("top-left", 1, 1), 2);
+  // Right anchor: col 0 is the RIGHTMOST CSS column (the corner), so
+  // the first placed widget hugs the viewport edge and free space
+  // extends inward.
+  assert.equal(core.isMirroredAnchor("top-right"), true);
+  assert.equal(core.gridColLine("top-right", 0, 1), 2);
+  assert.equal(core.gridColLine("top-right", 1, 1), 1);
+  // Full-width spans cover both columns either way; a 1x2 in the
+  // corner column stays on the right edge for both rows.
+  assert.equal(core.gridColLine("top-right", 0, 2), 1);
+  assert.equal(core.gridColLine("top-right", 0, 1), core.GRID_COLS);
+});
+
 test("layoutFromJSON: validates shape, ignores unknown anchors", () => {
   const good = core.layoutFromJSON({
     version: 1,

package/tests/dr-ext-host.test.js

@@ -353,6 +353,16 @@ test("dr-app: widget areas sit below the app's own top panels", () => {
   );
   assert.ok(colRight, "top-right area follows the GPS panel in its column");
   assert.match(appSrc, /\.dr-top-col \{[^}]*flex-direction: column/s);
+  assert.match(
+    appSrc,
+    /\.dr-top-col > \.dr-gps,\s*\.dr-top-col > dr-ext-widget-area\[anchor="top-right"\]\s*\{\s*align-self: flex-end;/s,
+    "top-right area right-aligns with the GPS panel in its column",
+  );
+  assert.match(
+    hostSrc,
+    /core\.gridColLine\(this\.anchor, p\.col, size\.cols\)/,
+    "cell columns route through the anchor-aware mirroring helper",
+  );
   assert.match(
     appSrc,
     /new PlotterExtHost\(\{\s*stream: window\.drSignalkStream,\s*mount: root,/s,

package/tests/plugin.test.js

@@ -3854,12 +3854,18 @@ test("tick integrates the Weather API current into the DR solution (tier 3)", as
   const app = new FakeSignalKApp();
   app.dataPath = dir;
   // In-process Weather API (the same object signalk-energy-predictor
-  // calls): 1 m/s due east → set 90° true, drift ≈ 1.944 kn.
+  // calls): 1 m/s due east → set 90° true, drift ≈ 1.944 kn. The
+  // response uses the Signal K WeatherData shape (currents under
+  // `water`, as spec-conformant providers like
+  // signalk-weather-router-plus serve them).
   app.weatherApi = {
     getForecasts: async () => [
       {
         date: new Date().toISOString(),
-        current: { set: Math.PI / 2, drift: 1 },
+        water: {
+          surfaceCurrentDirection: Math.PI / 2,
+          surfaceCurrentSpeed: 1,
+        },
       },
     ],
   };