Antônio Camargo
Antônio Camargo
Hi @efratmuller, sorry for the late reply (and thanks for all the details!) It looks like the discrepancy is because the complete genomes of UHGV were added to the CheckV...
For the time being, you can put a link to this repository and cite the source databases (listed [here](https://github.com/snayfach/UHGV?tab=readme-ov-file#data-sources)). We plan to have proper reference that you can cite in...
Can you share the full command you used?
Hi @mkuhn, Sorry for the delay. I’ve been on post-congress vacation and will be back to work on Thursday. Thanks for the PR! Let's merge it into the main branch...
So, would `taxid2names` replace `taxid2name`? That's my assumption, since having both `taxid2names[taxid]["scientific name"]` and `taxid2name[taxid]` would be redundant. This change would cause an API break that I'd like to avoid...
> I find the code example a bit confusing. As a Python coder, I wouldn't expect a dictionary(-like object) to have an attribute that is then also a dictionary. Good...
Thanks you! I'll take a better look at the code in the next couple of days
Sorry that it took me so long to get to this, @mkuhn. I wasn't in a rush because you mentioned that you didn't need this available in the package to...
Now I feel less bad :) Here's the review: https://github.com/apcamargo/taxopy/pull/18/files/099eb5322f19d1591a9182c066ad142a1a88ed5d#diff-6047f6e01ec3eb44d31bc0b381e7b7771a860b12a1dce2452636673f2ca088acR249
Sorry, @mkuhn, I think I overcomplicated things and you might have missed my comment. In lines 249-252 of `taxopy/core.py`, you introduced an additional level of indentation to the following block:...