Paper: reframe around resolution goal, fair comparisons, terminology - #128
Conversation
…#112 Address paper feedback (claude/paper_feedbac_2026-08-19.md): - Lead abstract/intro with the resolution goal (resolve as many real-world HGVS as possible) rather than "better UTA"; frame coverage, cleaning and version substitution as the three things serving that goal - Replace the unfair local-vs-remote throughput claim with like-for-like comparisons (local-to-local, remote-to-remote) in abstract and R3 - Recast the R2 Ensembl comparison as a capability point, not an accuracy headline; make RefSeq parity plus the historical submitted corpus the fair head-to-head, resolving the apparent Ensembl contradiction - Rename "version fallback"/"bumping" to "version substitution", "Tier 1/2" to "public/private data", and de-slang "a version bump can move a variant" - Cite the Ensembl VEP RefSeq alignment-gap limitation (ensembl-vep#1053) - Trim the JSON-format, benchmarking and TARK sections; lead the Discussion with the general-purpose transcript-source angle - Add claude/plans/new_analyses_plan.md capturing deferred analyses Paper-only prose and framing; no data or client-code changes.
davmlaw
left a comment
There was a problem hiding this comment.
Motivation still doesn't tell the story - Australian Genomics Shariant project had to collect historical c.HGVS from labs across the country. cdot is spun out of that work, and broad goal of the project is to fit
into ecosystem and help resolve as many HGVS as possible (so cleaning, and whole catalog of historical). I feel "What laboratories submitted" should be "ClinVar submissions as historical record of transcripts used" or something like
that. There should be vibepaper facts for: "the production corpus (N = 32,752) ... 1,721 rescued queriesthe production corpus (N = 32,752)." and "The 3.3% of the production corpus (1,075 queries; 826 unique strings)"
…PR feedback - #112 R2 (submitted-string resolution): rebuild the corpus from the VCV XML with per-pair submission dates and sample two ways, a whole-file random draw (recency-biased) and a time-bucketed draw balanced across submission-year eras (the fair historical picture). Report both, leading with the fair sample (cdot 98.0% vs UTA 79.9%; the fair draw widens the gap because older submissions cite superseded versions). Adopt the more complete self-contained XML corpus (3,198,528 pairs, all dated). R4 (cleaning residual): drop non-HGVS input (pasted URLs, prose) from the corpus before counting the residual, since it is a data-collection artifact, not a measure of tool correctness. Residual 3.3% -> 3.0%. Turn the hardcoded cleaning-corpus numbers into a vibepaper fact (cleaning_corpus.csv). PR feedback: lead the abstract motivation with the Shariant origin story; rename the R2 heading to "ClinVar submissions as a historical record of transcripts used".
|
🤖 Written by Claude Pushed Your PR comments
Deferred analyses folded in
Still open (awaiting your steer): warm-cache benchmark rerun (R3), full local-UTA over all ClinVar, and the submitter-attribution analysis. |
…es to CSVs - #112 Supplementary: replace remaining "Tier 1 / Tier 2" labels with "public data / private data" to match the main paper's provenance flags, and fix the stale [Tier 2] marker note in paper/README.md. Move the two number-dense hardcoded supplementary tables (S5 injection benchmark, S6 residual taxonomy) into committed CSVs rendered inline via the vibepaper include-csv directive, so the values live in data files rather than hardcoded markdown.
🤖 Written by Claude
Addresses the paper feedback in
claude/paper_feedbac_2026-08-19.md(#112). Paper-only prose and framing changes; no data-content or client-code changes, so no changelog entry.Framing (the "north star")
Fair comparisons
Terminology
Trims and citations
ensembl-vep#1053) in R1, precise to the c.-to-genomic direction.Deferred
Added
claude/plans/new_analyses_plan.mdcapturing the five feedback items that need re-running scripts or new data (time-bucketed submitted sampling, local UTA over all of ClinVar, submitter-attribution for missing versions, warm-cache benchmark rerun, dropping non-HGVS input from the R4 residual). These are not in this PR.Quick build renders cleanly (
snakemake -s paper/Snakefile), all template fact references resolve.