Skip to content

Commit 955e983

Browse files
rdhyeeclaude
andauthored
Fix isamples_explorer: declare facetSummariesError as mutable (#54)
OJS requires `mutable` keyword for cross-cell variable assignment. The undeclared assignment was killing the facetSummaries cell, cascading failures to all downstream filters and queries. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
1 parent a2c677d commit 955e983

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

tutorials/isamples_explorer.qmd

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -335,17 +335,22 @@ async function runQuery(sql) {
335335
}
336336
```
337337

338+
```{ojs}
339+
//| code-fold: true
340+
mutable facetSummariesError = null
341+
```
342+
338343
```{ojs}
339344
//| code-fold: true
340345
// Tier 1: Load pre-computed facet summaries (2KB, instant)
341346
facetSummaries = {
342-
facetSummariesError = null;
347+
mutable facetSummariesError = null;
343348
try {
344349
const rows = await runQuery(`SELECT * FROM read_parquet('${facet_summaries_url}')`);
345350
return rows;
346351
} catch (e) {
347352
console.error("Facet summaries load error:", e);
348-
facetSummariesError = e;
353+
mutable facetSummariesError = e;
349354
return [];
350355
}
351356
}

0 commit comments

Comments
 (0)