ga4gh / ga4gh/vrs-python

Add support for switching RLE subunit between largest and smallest

Open
#637 1 comment 0 reactions 1 assignee Claimed by @theferrit32 View on GitHub
Dominant language
Python
Stars
62
Forks
42
Avg merge
1h 2m
Merged PRs (30d)
1

Description

The code right now uses the largest subunit obtained after trimming common sequence.

The goal of this issue is to add support for instead using the smallest. This will then be made the default when we update to a forthcoming spec version, and will enable us to support creating digests using the prior 'largest subunit' method optionally in the code, when that is requested.

https://github.com/ga4gh/vrs-python/blob/a50b9fb189e777241aefd789abbd480ec9df4351/src/ga4gh/vrs/models.py#L688

https://github.com/ga4gh/vrs-python/blob/a50b9fb189e777241aefd789abbd480ec9df4351/src/ga4gh/vrs/normalize.py#L207

See https://github.com/ga4gh/vrs-python/issues/549#issuecomment-2810682997 for prior discussion.

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.