-
Notifications
You must be signed in to change notification settings - Fork 266
Implement ANCOM, perhaps as part of differential_abundance.py #2090
Comments
It's currently in an open PR scikit-bio/scikit-bio#1114; agree it would be
|
Excellent! The implementation itself does make more sense as part of skikit-bio, and this function can be added as a method in differential_abundance.py. You can close this issue if you want. |
There is a problem here unfortunately: QIIME 1 won't be updated to use a post 0.2.x version of scikit-bio, so this method won't be available. Unfortunately I think we'll have to wait for QIIME 2 to get moving if we're going to use the scikit-bio implementation (but it should be easy to use the scikit-bio implementation outside of QIIME once that PR is merged). |
Technically speaking, there is no reason we couldn't backport ANCOM to a 0.2.4 release, but that isn't me volunteering... |
Yep, it all comes down to what we have the bandwidth for. I'm really excited to start using this method, but I think the way to get there is to start focusing on QIIME 2 development. |
I'm comfortable with this being incorporated into QIIME 2. If I need it before then, a colleague or I can run it through scikit-bio. This is off topic, but what's the timeline for QIIME 2 release? Is your team planning a 1.10.0 release before 2.0.0? Just curious... |
No, we're not planning a 1.10.0 release, but we will do bug fix 1.9.x On Tue, Oct 13, 2015 at 3:32 PM, Colin Brislawn notifications@github.com
|
In case folks haven't seen it, the blog post went up here. |
ANCOM is now merged in scikit-bio, so will be included in the upcoming 0.4.1 release. |
This recent paper presents Analysis of Composition of Microbiomes (ANCOM)
http://www.ncbi.nlm.nih.gov/pmc/articles/PMC4450248/
Congrats @wdwvt1, @rob-knight! I would love to see this method in qiime!
The text was updated successfully, but these errors were encountered: