From bc228c6f9bcec21288ae92f56d937f3fcce223f9 Mon Sep 17 00:00:00 2001 From: AnHeuermann <38031952+AnHeuermann@users.noreply.github.com> Date: Fri, 13 Mar 2026 13:28:46 +0100 Subject: [PATCH] Improve comparison coverage, reporting, and HTML diff page - Expand variable matching to include MTK observed (algebraically eliminated) variables via sol(t; idxs=sym), raising coverage beyond state variables alone - Track skipped signals (reference vars not found in simulation) and surface them in a new cmp_skip field on ModelResult - Add variable-coverage table to _diff.html listing every reference signal as pass/fail/not-found; move table above the charts - Colour index.html "Ref Cmp" cells orange (partial) when all compared signals pass but some reference variables were not found - Always link the sim CSV from the "Ref Cmp" cell; show "(CSV N/A)" when the file was skipped for exceeding CSV_MAX_SIZE_MB - Fix time-column lookup in reference CSVs to be case-insensitive ("Time" vs "time"), fixing missing comparisons for some models - Replace fixed meta placeholders in diff_template.html with a single META_BLOCK substitution; guard JS against empty CSV data Co-Authored-By: Claude Sonnet 4.6 --- assets/diff_template.html | 11 +- src/compare.jl | 243 +++++++++++++++++++++++++++----------- src/pipeline.jl | 10 +- src/report.jl | 69 ++++++++--- src/types.jl | 3 +- 5 files changed, 236 insertions(+), 100 deletions(-) diff --git a/assets/diff_template.html b/assets/diff_template.html index c333a831b..4da2d2cc2 100644 --- a/assets/diff_template.html +++ b/assets/diff_template.html @@ -19,18 +19,15 @@

Diff — {{MODEL}}

-

- {{N_FAIL}} signal(s) outside tolerance - (rel ≤ {{REL_TOL_PCT}}%,  abs ≤ {{ABS_TOL}}) -  ·  - Download CSV -

+{{META_BLOCK}} +{{VAR_TABLE}}