Skip to content

Commit

Permalink
DEVELOP-721: removing checkqc threshold check (#55)
Browse files Browse the repository at this point in the history
* DEVELOP-721: removing checkqc threshold check

* DEVELOP-721: Updated unit tests
  • Loading branch information
nkongenelly authored Mar 6, 2024
1 parent d37769f commit d88e32b
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
5 changes: 0 additions & 5 deletions bin/get_qc_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,11 +31,6 @@ def __init__(self):
multiqc_mapping="percent_Q30",
compare_direction="lt",
),
ValueHandlerMapper(
handler_name="ReadsPerSampleHandler",
multiqc_mapping="mqc-generalstats-bcl2fastq-total",
compare_direction="lt",
),
]

self.mapping = self._convert_to_mappings(self._mapper_list)
Expand Down
1 change: 0 additions & 1 deletion tests/unit_tests/test_get_qc_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,5 @@ def test_convert_to_multiqc_config(checkqc_config_dict):
multiqc_config = convert_to_multiqc_config(checkqc_config_dict)
multiqc_section = multiqc_config["table_cond_formatting_rules"]
assert multiqc_section["Error"]["warn"] == [{"gt": 2}]
assert multiqc_section["mqc-generalstats-bcl2fastq-total"]["fail"] == [{"lt": 13.5}]
assert multiqc_section["percent_Q30"]["warn"] == [{"lt": 80}]
assert multiqc_section["total"]["warn"] == [{"lt": 18}]

0 comments on commit d88e32b

Please sign in to comment.