From 06a8a9d143826d7af341ebca5fc7e49bb722f58b 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 d980a530..107adc2e 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; }