Skip to content

feat: Reverse Translation Mapping Support#108

Draft
bencap wants to merge 3 commits into
feature/bencap/vrs-correctnessfrom
feature/bencap/target-variant-projection
Draft

feat: Reverse Translation Mapping Support#108
bencap wants to merge 3 commits into
feature/bencap/vrs-correctnessfrom
feature/bencap/target-variant-projection

Conversation

@bencap

@bencap bencap commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

bencap added 3 commits June 9, 2026 23:58
Add MappingOutcome so every (variant, level) record carries a typed outcome,
distinguishing a benign absence (intronic, no protein consequence) from a
genuine failure that error_message alone cannot convey. Derive the preferred
(authoritative) layer from the target's assay level rather than always
preferring genomic.
…c projection

Select a coding transcript for NC_ protein-coding targets via Ensembl locus
overlap plus MANE, so they are no longer silently skipped by reverse
translation. Project each measured variant onto its deterministically
reachable layers (g<->c, nucleotide->p), emitted as typed-outcome records and
routed by preferred_layer_only.
…ferred-layer records

- Track represented variant IDs at the preferred layer; re-attribute
  only variants that have no preferred-layer record, avoiding duplicate
  mapped_scores for variants with both a dead genomic attempt and a
  measured protein record (e.g. codon-optimised targets).
- Synthesize a preferred-layer failure for variants that mapped only at
  a non-preferred layer (e.g. wild-type p.= on a genomic-preferred
  target) so every input variant always has exactly one output record.
- Extract _map_protein_layer in vrs_map to return (mapping, reason)
  instead of an ad-hoc error MappedScore; a row that maps at no layer
  is failed once, layer-agnostically, carrying the detailed reason.
- Add TestNullFailureDedup and TestMapProteinLayerReason test coverage.
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.

feat: Redesign mapper output to produce MappingRecord + Allele rows at all levels

1 participant