James Stevenson
James Stevenson
Ah, yeah, foiled by repository rules. We'll be able to once @matentzn is able to re-review.
Identified by Otto Jolanki. Other instances: ``` Failed to generate SPDI for chr21, pos: 5033800, ref: TGGGTCCAGGCACCGGCGCCCAGCCCCCGTGGGGTGTCCAGGGC alt: T AttributeError("'NoneType' object has no attribute 'root'") Failed to generate SPDI for...
I was thinking about how to test this. It doesn't make sense to me to fail if the values are inequal but warnings in GitHub actions are fairly obscure so...
Maybe this is a bit too out there, but for those publishing work to Jupyter notebooks (probably a lot of us), you can also supply [special __repr__ methods](https://ipython.readthedocs.io/en/stable/config/integrating.html#rich-display) that incorporate...
Agree with this, although the complication is that the translator module currently bins all translator sources together (i.e. you gotta import `hgvs`). Might be good to refactor to separate those...
👍 this is great. I think I'd _like_ to see the output sorted, although it might be worth checking if it's faster/easier to just run the output through `bcftools sort`.
👍 I'd like to see the DataProxy interface add some kind of health check (is the REST server responding? is the seqrepo file system available?) to be called by `create_dataproxy`...
Is there a 'nice to have' tag? I would characterize it as 'nice to have'
Hi @melissacline -- did you fork the repo (line above this one)? I think that `ga4gh/vrs-python` works for you because you're part of the ga4gh GitHub org and have maintainer...
Either way, we could make this easier and clearer by adding a [hyperlink to fork](https://github.com/ga4gh/vrs-python/fork) onto that text. edit: made a PR -- https://github.com/ga4gh/vrs-python/pull/458#pullrequestreview-2410737354. Happy to modify language further for...