You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am routinely building grafted / adjusted taxonomies with taxonkit and was wondering if the changes in the fork below could be pulled into the next release / Bioconda release, so we can test using Metabuli in production?
FYI the fix works for other taxonkit adjusted taxonomies like the NCBI/GTDB grafts outlined in the Tutorials.
Currently, I am using this version of Metabuli: Metabuli Version 1.0.8. I have a question: is the version I downloaded different from the one available in this directory? -> https://github.com/jaebeom-kim/Metabuli
Yes, please use https://github.com/jaebeom-kim/Metabuli when you try the ICTV database. Sorry for this inconvenience. Taxonkit uses the full range of 32 bit integer for taxonomy ID, but Metabuli used only 31 bits, so I made a quick fix in my fork.
Another question: Should I download the nodes.dmp and names.dmp files from here > https://ftp.ncbi.nlm.nih.gov/pub/taxonomy/new_taxdump/ or do I need to build my own taxonomy files using TaxonKit and create an ICTV taxdump file?
Hey @jaebeom-kim,
really enjoying Metabuli, fantastic work!
I am routinely building grafted / adjusted taxonomies with
taxonkit
and was wondering if the changes in the fork below could be pulled into the next release / Bioconda release, so we can test using Metabuli in production?FYI the fix works for other
taxonkit
adjusted taxonomies like the NCBI/GTDB grafts outlined in the Tutorials.Yes, please use
https://github.com/jaebeom-kim/Metabuli
when you try the ICTV database. Sorry for this inconvenience. Taxonkit uses the full range of 32 bit integer for taxonomy ID, but Metabuli used only 31 bits, so I made a quick fix in my fork.You don't need to download any dmp files to try the ICTV database.
But, I just shared dmp files in https://hulk.mmseqs.com/jaebeom/vmr39.2/ictv-taxdump/ just in case.
Thanks again!
Originally posted by @jaebeom-kim in #96 (comment)
The text was updated successfully, but these errors were encountered: