Skip to content

Commit

Permalink
fix pep8 issues
Browse files Browse the repository at this point in the history
  • Loading branch information
jethror1 committed Sep 20, 2024
1 parent fb926e1 commit 88e58a2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/vcf_qc.py
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ def get_het_hom_counts(vcf) -> dict:
print(f"Missing field(s)")
continue

if sample_fields['GT'] == (0,0):
if sample_fields['GT'] == (0, 0):
continue

# using the sum of all allele depths instead of the format AD
Expand Down
5 changes: 2 additions & 3 deletions tests/test_vcf_qc.py
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@ def test_het_and_hom_ratios_calculated_correctly(self):
variants
"""
counts = {
'het': [ 0.5, 0.45, 0.55],
'het': [0.5, 0.45, 0.55],
'hom': [1.0, 1.0, 1.0, 1.0, 0.95],
'x_het': [],
'x_hom': []
Expand All @@ -207,7 +207,7 @@ def test_het_and_hom_ratios_calculated_correctly(self):

def test_x_het_hom_calculated_correctly(self):
counts = {
'het': [ 0.5, 0.45, 0.55],
'het': [0.5, 0.45, 0.55],
'hom': [1.0, 1.0, 1.0, 1.0, 0.95],
'x_het': [0.45, 0.55],
'x_hom': [1.0, 0.98]
Expand Down Expand Up @@ -313,4 +313,3 @@ def test_expected_dxlink_format_returned(self, mock_job, mock_upload):
vcf_qc.upload_output_file('test.vcf.qc'),
{'output_file': {'$dnanexus_link': 'file-xxx'}}
)

0 comments on commit 88e58a2

Please sign in to comment.