Skip to content

feat(variants): show tree branch(es) a SNP is assigned to on the deta…#18

Merged
JamesKane merged 1 commit into
mainfrom
feat/variant-tree-branches
Jul 3, 2026
Merged

feat(variants): show tree branch(es) a SNP is assigned to on the deta…#18
JamesKane merged 1 commit into
mainfrom
feat/variant-tree-branches

Conversation

@JamesKane

Copy link
Copy Markdown
Owner

…il view

Restores a feature lost in the Rust rewrite: the Variant Browser detail card now lists the haplogroup branch(es) the variant defines, linking each to the Y/mt tree rooted at that node.

Matched by genomic site (the tree is hs1-native), not just the row's own links: a catalog-named SNP (e.g. V3739) surfaces the placement carried by its de-novo coordinate-named sibling (chrY:), and recurrence — the same site defining multiple branches — shows all of them. When the defining row differs from the viewed one, its name is shown as "via ". Low-confidence links are badged.

  • du_db::variant::tree_branches — GIN-indexed coordinates @> site containment.
  • migration 0055: partial index haplogroup_variant(variant_id) WHERE valid_until IS NULL (the existing unique key leads with haplogroup_id, so the reverse lookup had no support and seq-scanned; 46ms -> 2ms).

…il view

Restores a feature lost in the Rust rewrite: the Variant Browser detail card now
lists the haplogroup branch(es) the variant defines, linking each to the Y/mt tree
rooted at that node.

Matched by genomic site (the tree is hs1-native), not just the row's own links: a
catalog-named SNP (e.g. V3739) surfaces the placement carried by its de-novo
coordinate-named sibling (chrY:<pos><change>), and recurrence — the same site
defining multiple branches — shows all of them. When the defining row differs from
the viewed one, its name is shown as "via <name>". Low-confidence links are badged.

- du_db::variant::tree_branches — GIN-indexed coordinates @> site containment.
- migration 0055: partial index haplogroup_variant(variant_id) WHERE valid_until IS
  NULL (the existing unique key leads with haplogroup_id, so the reverse lookup had
  no support and seq-scanned; 46ms -> 2ms).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@JamesKane JamesKane merged commit 2c36321 into main Jul 3, 2026
1 check failed
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