Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update BlastNFilterSummary to directly count number of targets found #187

Closed
coleslaw481 opened this issue Jun 14, 2018 · 0 comments
Closed

Update BlastNFilterSummary to directly count number of targets found #187

coleslaw481 opened this issue Jun 14, 2018 · 0 comments
Assignees
@coleslaw481
Milestone
1.11.0

Comments

@coleslaw481
Copy link
Contributor

coleslaw481 commented Jun 14, 2018 •
edited

Since Targets Found: line is no longer in summary.txt for the blastnfilter stage, modify the BlastNFilterSummary to directly calculate this value by counting number of .txt files as a fallback.
@coleslaw481 coleslaw481 added this to the 1.10.1 milestone Jun 14, 2018
@coleslaw481 coleslaw481 self-assigned this Jun 14, 2018
@coleslaw481 coleslaw481 changed the title Update BlastNFilterTask to directly count number of targets found Update BlastNFilterSummary to directly count number of targets found Jun 14, 2018
coleslaw481 added a commit that referenced this issue Jun 14, 2018
@coleslaw481
added tests to verify code is working properly to get number of targe…
796e3f5 
…ts by counting txt files if value is not in summary.txt. Fix for issue #187
@coleslaw481 coleslaw481 modified the milestones: 1.10.1, 1.11.0 Aug 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@coleslaw481 coleslaw481

Labels
None yet
Projects
None yet
Milestone
1.11.0
Development

No branches or pull requests
1 participant
@coleslaw481