From 7c42439af8a252d3b5d00996f0b05cf01aa8d361 Mon Sep 17 00:00:00 2001 From: Filip Leonarski Date: Thu, 27 Aug 2026 16:28:11 +0200 Subject: [PATCH] rugnux --model: say why validation did not run when the data carry no resolution The d_min <= 0 return was the one failure path that left failure_reason empty. The report's MODEL_VALIDATION= NOT_PERFORMED line and the WARNING that goes with it are both gated on that string, so this case printed a blank reason and no warning - the "indistinguishable from a run that was never given --model" outcome the field was added to prevent. All four failure returns now set it. Co-Authored-By: Claude Opus 5 (1M context) Claude-Session: https://claude.ai/code/session_016L1qig74oYQzfUJJZbbxFh --- rugnux/ModelValidation.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/rugnux/ModelValidation.cpp b/rugnux/ModelValidation.cpp index d980a530a..107adc2e3 100644 --- a/rugnux/ModelValidation.cpp +++ b/rugnux/ModelValidation.cpp @@ -137,7 +137,8 @@ ModelValidationResult ValidateAgainstModel(const std::vector & if (r.d > 0 && (d_min == 0.0 || r.d < d_min)) d_min = r.d; if (d_min <= 0.0) { - logger.Error("Model validation: merged reflections carry no resolution"); + result.failure_reason = "the merged reflections carry no resolution"; + logger.Error("Model validation: {}", result.failure_reason); return result; }