-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
qiime2 plugin? #31
Comments
Merged and yes, there would be a lot of interest to see a q2-plugin here |
I was able to get t2t to work, even though it required a bit of massaging to convert from qiime2 taxonomy to a suitable taxonomy that t2t accepts. I can get a PR started in ~2 weeks. |
Thanks! Q2 does not ensure all ranks are represented IIRC
… On Apr 12, 2021, at 11:04 AM, Jamie Morton ***@***.***> wrote:
I was able to get t2t to work, even though it required a bit of massaging to convert from qiime2 taxonomy to a suitable taxonomy that t2t accepts. I can get a PR started in ~2 weeks.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub <#31 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AADTZMTLVNFRTLSJNLDWEELTIMY4NANCNFSM42ZSJ7WA>.
|
Correct, I had to hack that in as well. import pandas as pd
RANK_ORDER = ['d', 'p', 'c', 'o', 'f', 'g', 's']
taxa = pd.read_table('consistency.tsv', index_col=0, header=None)
def f(x):
tabs = x.split('; ')
n = len(RANK_ORDER) - len(tabs)
if n > 0:
return x + '; ' + '__; '.join(RANK_ORDER[-n:]) + '__'
else:
return x
data = taxa[1].apply(f)
data.to_csv('t2t_consistency.tsv', sep='\t', header=None) |
Would there be any interest in attaching a qiime2 plugin to this repository?
Of course, it would have to be resolved after merging #29
But it could be incredibly useful to have internal node relabeling in place; particularly for interpreting ILR coordinates.
The text was updated successfully, but these errors were encountered: