Repository navigation
Conversation
commit: |
There was a problem hiding this comment.
🔵 Needs a closer look
Cached harmonic-only datums can survive when a new synthesis omits them.
0 open findings
What changed in this PR
Corrects eight TICON constituent phases and regenerates dependent harmonic datums.
Changes:
- Applies constituent-specific phase offsets during import.
- Re-synthesizes cached harmonic datums when constituents change.
- Regenerates 4,640 stations and quality warnings.
| File | Description |
|---|---|
sources/ticon/import.ts |
Adds phase conversion and datum refresh logic. |
sources/ticon/README.md |
Documents offsets and validation method. |
data/ticon/*.json (4,640 files) |
Updates phases and synthesized datums. |
quality.json |
Refreshes resulting warning text. |
🧠 Review effort: Balanced
Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.
|
you don't have to review, the only the question i have here for @bkeepers is this
|
…ment TICON-4 phases M3, sigma1, T3, S3, R3, 3L2, 2MK5 and 2MO5 against an equilibrium argument that differs from the engine's by a constant (180°, -90° or +90°). Scored against the raw GESLA-4 record at each constituent's largest gauges, every one matches its offset (7/8 to 20/20 gauges; the rest are river gauges at noise), and S1, P1, T2, R2, S4, RHO1, 2SM2, 2MS6, 3N2 and MKS2 need none. The importer subtracts the offset from TICON's published phase; re-fit wsv/rws stations are fit against the engine's arguments and are unaffected. Cached datums went stale whenever the constituents changed, since only the observed means come from the record. The importer now re-synthesizes the harmonic datums when a station's constituents differ from its cached file, keeping the observed means. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Phases of the eight converted constituents change at all 4,640 TICON stations that use TICON's published phases. Re-synthesis moves HAT, LAT or LLWLT by 1 cm or more at about 2,400 of them, up to 39 cm (LAT at Ashland Ave). It also applies engine changes since the datums were cached, such as MKS2 moving to its IHO line. quality.json changes only warning text; no station changes score or acceptance. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
86e1ba1 to
db454d6
Compare
|
What's the question there? |
The datum question is really three decisions bundled into a phase fix:
|
TICON-4 phases eight constituents against a different equilibrium argument than the engine's, so the engine predicts them at the wrong phase. The importer now converts them, and datums are re-synthesized to match.
Each offset comes from scoring TICON's constituent against the raw GESLA-4 record at its largest gauges, with the published phase shifted by 0°, +90°, −90° and 180°. +1 is a match, −1 unrelated, −3 anti-phase.
Most of the gauges that disagree are river gauges (Hedel, Gorinchem) that score near −1 at every offset. The exception is S3 at L'Isle-aux-Grues in the St. Lawrence estuary, which scores 0.66 at 0°. After conversion, every constituent scores best at 0° at its top gauges, apart from the same river gauges. S1, P1, T2, R2, S4, RHO1, 2SM2, 2MS6, 3N2 and MKS2 already score best at 0°, so they need no conversion.
Six of the eight are among the ten constituents new in TICON-4; M3 and S3 are not. The conversion happens at import (
TICON_ARGUMENT_OFFSETinsources/ticon/import.ts), so the engine's IHO definitions stay as they are. NOAA's phases for M3, sigma1, 2MK5 and 2MO5 already match those definitions. The re-fitwsvandrwsstations are fit against the engine's arguments and are unchanged. The TICON README has the table and the method.Datums. The importer reused cached datums even when a station's constituents changed. HAT, LAT, LLWLT and the other harmonic datums come from synthesis, so they went stale. The importer now re-synthesizes those whenever the constituents differ from the cached file, and keeps the observed means (MHHW through MLLW), which come from the record. Re-synthesis moves HAT, LAT or LLWLT by 1 cm or more at about 2,400 stations, and by up to 39 cm (LAT at Ashland Ave).
Re-synthesis also picks up engine changes made since the datums were cached. About 120 stations move 1 cm or more beyond what the phase conversion explains. At Aiguillon-sur-Mer, LAT moves 15 cm and the phases account for 7 cm; the rest comes from the 3N2 and MKS2 lines moving in openwatersio/slackwater#365. Aiguillon has 13.5 cm of 3N2 and 5.2 cm of MKS2. One near-tideless lake gauge, Tecumseh (Lake St. Clair), had an LLWLT half a metre below MSL and now has one above it. Its tides are under a centimetre, so both values are artifacts. It joins the 19 TICON stations whose datums are already out of order on main (LAT above MLLW, HAT below MHHW, or LLWLT above MLLW), which need a separate fix.
quality.jsonchanges only in warning text; no station changes score or acceptance.npm test(including the station tests),npm run validate:database,npm run lint,npm run test -w swiftCloses #239
🤖 Generated with Claude Code