Repository navigation
fix: _from_vrs handle kwargs without error - #635
Merged
Merged
Conversation
This was referenced May 23, 2026
korikuzma
approved these changes
May 23, 2026
theferrit32
added a commit
to theferrit32/vrs-python
that referenced
this pull request
Oct 28, 2026
Forward-merge main into vrs/2.1 so it is no longer behind. Net change to vrs/2.1 is ga4gh#642 (is_intronic r. fix); the other main commits since the merge base (ga4gh#635, ga4gh#639, ga4gh#645, ga4gh#648) were already ported as parallel PRs. Conflict resolution: - src/ga4gh/core/models.py: conflict between ga4gh#645 (main) and vrs/2.1's restructured models (ga4gh#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 ga4gh#648. Took vrs/2.1's version (CORE_VERSION from ga4gh.core.version, 1.3.0).
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.
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_vrswill 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.