Skip to content

report: record curator decisions; fix analysis index placement - #49

Merged
hyphaltip merged 1 commit into
mainfrom
report-decisions
Oct 8, 2026
Merged

hyphaltip merged 1 commit into
mainfrom
report-decisions

Conversation

@hyphaltip

Copy link
Copy Markdown
Member

Follow-up to #48. These two changes were pushed to report-genome after #48 was merged, so they never reached main.

  • Records the three design decisions from 2026-10-07 in the spec, analysis/2026-10-07_report-design-review.md and the analysis index:
    • detect writes report.html by default, with --no-html / MATPREDICT_HTML=0 to opt out;
    • WeasyPrint is the PDF engine;
    • withheld loci stay in an always-present section, with the table collapsed on screen and expanded in print.
  • Fixes analysis/INDEX.md. The merge of main into Per-genome HTML/PDF report, written by detect by default #48 had put the report row inside the "Reads-only typing" table, marked "open". It now has its own "Reports and tooling" section, marked "decided; PR Per-genome HTML/PDF report, written by detect by default #48". The updated Fola and A. fumigatus reads rows are kept as they are.

Docs only; no code changes. The tree is identical to the final report-genome.

🤖 Generated with Claude Code

…ld loci collapsed)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@hyphaltip
hyphaltip merged commit a932dfe into main Oct 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant