Skip to content

Paper: add missing Supplementary Tables S5 and S6 - #126

Closed
davmlaw wants to merge 1 commit into
paper-feedback-responsefrom
supplementary-tables-s5-s6
Closed

Paper: add missing Supplementary Tables S5 and S6#126
davmlaw wants to merge 1 commit into
paper-feedback-responsefrom
supplementary-tables-s5-s6

Conversation

@davmlaw

@davmlaw davmlaw commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

🤖 Written by Claude

Results referenced Supplementary Tables S5 and S6, but supplementary.md jumped from Table S4 to Table S7; neither table existed.

  • Table S5 (new): per-category breakdown of the reproducible injection cleaning benchmark (inject_and_clean.py), including the LOVD syntax checker head-to-head columns landed on the base branch (lovd_head_to_head.py), with synthesised NM_000059.4 examples, weighted totals, and the false-correction check.
  • Table S6 (moved): the residual-error taxonomy table and its method/limitation note move from Results into the supplement (the Results heading already labelled it "(Table S6)"); the Results prose now points at the table instead of inlining it.

Stacked on paper-feedback-response (PR #125) because Table S5 uses the LOVD comparison facts introduced there. Both the main and supplementary docx build cleanly via the quick Snakemake target with all template values substituted.

🤖 Generated with Claude Code

https://claude.ai/code/session_015662nbf1iezxdL9EMcckU7

Results referenced Table S5 (injection benchmark detail) and Table S6
(residual error taxonomy) but the supplement jumped from S4 to S7.
Add Table S5 with the per-category injection recovery plus the LOVD
head-to-head breakdown, and move the residual-class table and its
classification method note from Results into the supplement as Table
S6, leaving the Results prose pointing at it.
davmlaw added a commit that referenced this pull request Aug 18, 2026
Paper: add missing Supplementary Tables S5 and S6
@davmlaw

davmlaw commented Aug 18, 2026

Copy link
Copy Markdown
Contributor Author

🤖 Written by Claude

Merged into main directly (commit 60d1165) rather than into the paper-feedback-response base, which had already been merged via #125.

GitHub would not let the base be retargeted to main ("There are no new commits between base branch 'main' and head branch 'supplementary-tables-s5-s6'") because the commits already landed there, so this PR is being closed manually. The change is in main; nothing is lost.

@davmlaw davmlaw closed this Aug 18, 2026
@davmlaw
davmlaw deleted the supplementary-tables-s5-s6 branch August 18, 2026 00:58
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