Conpair concordance: account for different format when run with --outfile

CHANGELOG.md

@@ -14,6 +14,8 @@
 
 ### Module updates
 
+- **Conpair**
+  - Bugfix: allow to find and proprely parse the `concordance` output of Conpair, which may output 2 kinds of format for `concordance` depending if it's ran with or without `--outfile` ([#1851](https://github.com/ewels/MultiQC/issues/1851))
 - **Cell Ranger**
   - Bugfix: avoid multiple `KeyError` exceptions when parsing Cell Ranger 7.x `web_summary.html` ([#1853](https://github.com/ewels/MultiQC/issues/1853), [#1871](https://github.com/ewels/MultiQC/issues/1871))
 - **DRAGEN**

multiqc/modules/conpair/conpair.py

@@ -55,21 +55,25 @@ def parse_conpair_logs(self, f):
         One parser to rule them all."""
 
         conpair_regexes = {
-            "concordance_concordance": r"Concordance: ([\d\.]+)%",
-            "concordance_used_markers": r"Based on (\d+)/\d+ markers",
-            "concordance_total_markers": r"Based on \d+/(\d+) markers",
-            "concordance_marker_threshold": r"\(coverage per marker threshold : (\d+) reads\)",
-            "concordance_min_mapping_quality": r"Minimum mappinq quality: (\d+)",
-            "concordance_min_base_quality": r"Minimum base quality: (\d+)",
-            "contamination_normal": r"Normal sample contamination level: ([\d\.]+)%",
-            "contamination_tumor": r"Tumor sample contamination level: ([\d\.]+)%",
+            "concordance_concordance": [r"Concordance: ([\d\.]+)%\nBased.*", r"(\d\.\d+)\nBased.*"],
+            "concordance_used_markers": [r"Based on (\d+)/\d+ markers"],
+            "concordance_total_markers": [r"Based on \d+/(\d+) markers"],
+            "concordance_marker_threshold": [r"\(coverage per marker threshold : (\d+) reads\)"],
+            "concordance_min_mapping_quality": [r"Minimum mappinq quality: (\d+)"],
+            "concordance_min_base_quality": [r"Minimum base quality: (\d+)"],
+            "contamination_normal": [r"Normal sample contamination level: ([\d\.]+)%"],
+            "contamination_tumor": [r"Tumor sample contamination level: ([\d\.]+)%"],
         }
 
         parsed_data = {}
-        for k, r in conpair_regexes.items():
-            match = re.search(r, f["f"])
-            if match:
-                parsed_data[k] = float(match.group(1))
+        for k, r_arr in conpair_regexes.items():
+            for r in r_arr:
+                match = re.search(r, f["f"])
+                if match:
+                    parsed_data[k] = float(match.group(1))
+                    if k == "concordance_concordance" and not "Concordance" in r:
+                        parsed_data[k] = 100 * parsed_data[k]
+                    break
 
         def _cp_type(data):
             if "concordance_concordance" in parsed_data:

multiqc/utils/search_patterns.yaml

@@ -229,7 +229,7 @@ clusterflow/runfiles:
   contents: "Cluster Flow Run File"
   num_lines: 2
 conpair/concordance:
-  contents: "markers (coverage per marker threshold : "
+  contents_re: '.*markers \(coverage per marker threshold\s?: .*'
   num_lines: 3
 conpair/contamination:
   contents: "Tumor sample contamination level: "