-
Notifications
You must be signed in to change notification settings - Fork 14
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
ENH: add action to estimate MAG abundance #177
Conversation
/q2ci |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #177 +/- ##
==========================================
- Coverage 98.47% 97.96% -0.52%
==========================================
Files 82 155 +73
Lines 4600 9298 +4698
Branches 0 409 +409
==========================================
+ Hits 4530 9109 +4579
- Misses 70 140 +70
- Partials 0 49 +49 ☔ View full report in Codecov by Sentry. |
This PR/issue depends on: |
For reviewers: this action was already tested against some real data so (unless you want to try it out) feel free to focus mostly on the code review. Thanks! 🙏 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @misialq, the code looks good to me! You can merge! 🚀
This PR adds an action to estimate abundances of dereplicated MAGs in the original samples. The estimation is done in the following way:
map-reads
action from q2-assembly).samtools coverage
command.Depends on bokulich-lab/q2-assembly#81.