Repository navigation
Conversation
GKS-Core Release: https://github.com/ga4gh/gks-core/releases/tag/1.1.0 VRS Release: https://github.com/ga4gh/vrs/releases/tag/2.1.0-snapshot.2026-02.2 (changes can easily be seen ga4gh/vrs@2.0.1...2.1.0-snapshot.2026-02.2)
the root cause of biocommons/anyvar#467 If you have a valid variant expression, you can attach some extra kwargs like "assembly_name" to help ground certain kinds of translation. However, if you happen to have an invalid expression, the translator will attempt each different format of translation, including `_from_vrs`. If you have attached kwargs, `_from_vrs` will raise an unexpected TypeError because it, unlike the other `_from_` methods, doesn't include kwargs in its type signature. I can't think of a reason why this might be the case. I also added a test to show this.
close #638 * Unsupported `fmt` in `translate_to` should raise `NotImplementedError`
close #638 * Unsupported `fmt` in `translate_to` should raise `NotImplementedError`
Pull in patches from 2.3.x branch
## Changes * update models vrs 2.1.1-ballot.2026-09.1 and gkm-core 1.3.0-ballot.2026-09.1 * Added `_abstract` classvar * GKS -> GKM (deprecating classes that start with GKS) * Adds missing `RelativeAllele` to Variation root ## Schema changes _note: Use /json rather than *-source.yaml_ * Core changes: https://github.com/ga4gh/gkm-core/compare/1.2.0..1.3.0-ballot.2026-09.1 * VRS changes: https://github.com/ga4gh/vrs/compare/2.1.0..2.1.1-ballot.2026-09.1 --------- Co-authored-by: Larry Babb <lbabb@broadinstitute.org>
_abstract is metadata for MSP (to allow additional properties)
* GKM-Core version 1.3.0 * Also fixes constraints for `ConceptSet.concepts`
`is_intronic` only recognized introns on coding (c.) coordinates because it gated on the interval being a `BaseOffsetInterval`. RNA (r.) coordinates parse to a plain `Interval` whose endpoints are still `BaseOffsetPosition`, so intronic r. variants slipped past the "Intronic HGVS variants are not supported" guard inspect the interval endpoints instead of the interval type. g./p. coordinates have no base-offset positions and stay non-intronic closes #628 --------- Co-authored-by: Kyle Ferriter <kferrite@broadinstitute.org>
Forward-merge main into vrs/2.1 so it is no longer behind. Net change to vrs/2.1 is #642 (is_intronic r. fix); the other main commits since the merge base (#635, #639, #645, #648) were already ported as parallel PRs. Conflict resolution: - src/ga4gh/core/models.py: conflict between #645 (main) and vrs/2.1's restructured models (#644 + gkm-core updates). Took vrs/2.1's version, which already includes the int Extension.value fix. - src/ga4gh/core/__init__.py: auto-merged without conflict but incorrectly, duplicating main's __version__ block (undefined `version`) and CORE_VERSION = "1.0.0" from #648. Took vrs/2.1's version (CORE_VERSION from ga4gh.core.version, 1.3.0).
Contributor
Author
|
Will do a merge commit for this |
korikuzma
marked this pull request as ready for review
October 19, 2026 18:28
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
close #672