SACGF / SACGF/variantgrid

Display resolved p.HGVS in classification and variant tables

Open
#19 0 comments 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
Python
Stars
30
Forks
3
Avg merge
9h 28m
Merged PRs (30d)
42

Description

Depends on #1383.

Consistent request from users: add p.HGVS to tables. The protein change is how most people interpret variants, especially in cancer/somatic contexts where shorthand like "KRAS G12C" is the common language.

**Tables to update:**
- Classification datatables (`render_c_hgvs()` — switch from `published_evidence__p_hgvs__value` to `ResolvedVariantInfo.p_hgvs`)
- Classification grouping datatables (likewise)
- Classification template tags (likewise)
- Classified variant tables
- Patient tagged variants
- Sample tagged variants
- Gene tagged variants

**Display requirements:**
- Reorder default columns so p.HGVS is adjacent to c.HGVS (currently easy to miss — ENSP IDs take up space)
- Show short-form in compact table cells (e.g., `V600E`), long-form on hover/tooltip — important for cancer/somatic use
- Respect existing `CLASSIFICATION_GRID_SHOW_PHGVS` setting toggle

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.