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
Write per-virus per-sample fasta in proteins-to-viruses.py #471
Conversation
Maintain a cache of virus/sample FASTA file names, creating the FASTA | ||
on demand. | ||
""" | ||
def __init__(self, proteinGrouper): |
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.
You don't have an explanation for proteinGrouper
.
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.
Fixed.
@@ -286,7 +336,7 @@ def toHTML(self): | |||
|
|||
# Write all samples (with viruses (with proteins)). | |||
append('<h1>Samples by virus</h1>') | |||
for sampleName in sorted(sampleNames): | |||
for sampleName in sampleNames: | |||
|
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.
You don't need the newline.
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.
OK.
@@ -264,8 +271,9 @@ def testOneLineInEachOfTwoFilesSameVirusTitle(self): | |||
|
|||
def testOneLineInEachOfTwoFilesDifferentViruses(self): | |||
""" | |||
If a protein grouper is given two files, each with one line from the | |||
different viruses, its virusTitles dict must be as expected. | |||
If a protein grouper is given two files intwo different directories, |
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.
s/intwo/in two
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.
Fixed.
OK, just the trivial comments. |
Fixes #465.