-
Notifications
You must be signed in to change notification settings - Fork 295
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
Add total number of reads to summary json/tsv #996
Conversation
Also apologies if jenkins or others complain about a commit to master, I accidentally changed to master between the above commits, and pushed the change. I've reverted it. |
jenkins retest this please |
Ah yes, one of these things:
|
189a890
to
a432d83
Compare
@kdmurray91 LGTM! Yes, please continue to @ mention us when you are ready for review. The full list is at http://khmer.readthedocs.org/en/latest/dev/getting-started.html#claiming-an-issue-and-starting-to-develop item number 12 (this was formalized during the coding sprint at PyCon). |
Add total number of reads to summary json/tsv
This PR adds a count of the total number of reads to the machine readable outputs of
load-into-counting.py
(TSV/JSONs).It takes the sum of the number of reads processed from the
khmer.ReadParser
object which is used for hashing across all read files.I've also bumped the "mrinfo_version" tag in the json output to 0.2.0, FWIW.
Cheers,
K