-
-
Notifications
You must be signed in to change notification settings - Fork 89
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
Post AUC beta run dplyr lesson updates #161
Post AUC beta run dplyr lesson updates #161
Conversation
Did not know you could "draft" a pull request to indicate it is not yet ready for merging! Very cool! |
All issues listed under "Data wrangling and analysis with tidyverse" in our mega issue: #142 have been addressed and are now ready for review. Pinging @JasonJWilliamsNY @naupaka and @poterlowicz-lab (whoever has a moment) to review and merge. |
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.
Nice work @ytakemon! I spotted one minor thing to fix.
Fixed typo
Post AUC beta run dplyr lesson updates
Currently, we filter on INDEL which is a logical column:
filter(variants, sample_id == "SRR2584863", (INDEL | QUAL >= 100))
and INDEL is logical. Instead of INDEL, we will now filter on MQ:filter(variants, sample_id == "SRR2584863", (MQ >= 50 | QUAL >= 100))