From 8ba66ba689fe71c720dc1d0bb97179d53d174b6c Mon Sep 17 00:00:00 2001
From: "Dylan M. Taylor" <dylan@dylanmtaylor.com>
Date: Wed, 4 Nov 2026 00:51:51 +0000
Subject: [PATCH] address review: stop the descriptive builder once the grid is
 too large

---
 crates/engine/src/cmd/analysis.rs | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/crates/engine/src/cmd/analysis.rs b/crates/engine/src/cmd/analysis.rs
index 9d6318433..7f66954e5 100644
--- a/crates/engine/src/cmd/analysis.rs
+++ b/crates/engine/src/cmd/analysis.rs
@@ -332,6 +332,10 @@ fn descriptive(s: &Session, p: &Json) -> Result<Grid> {
     }
     let mut g = Grid::default();
     for (i, v) in vars.iter().enumerate() {
+        // The report is refused once the grid is too large: the rest would only build discarded labels.
+        if g.too_large {
+            break;
+        }
         let x = numbers(&v.vals, true)?;
         let c = idx(i).saturating_mul(2);
         g.txt(0, c, &v.name);