Fix QualiMap MEAN/MEDIAN

[apeltzer]

Hi,

this fixes a bug introduced in MultiQC v1.8 while adding the mean coverage computation to the QualiMap module output. Unfortunately, the break condition caused some errors and I had to split up the loop over the dictionary to two loops now to get this done correctly. I verified using the data in the #1082 issue and it looks like this fixed the problem.

multiqc_report.html.zip

If this PR is not a new module

• This comment contains a description of changes (with reason)
• CHANGELOG.md has been updated
• (optional but recommended): https://github.com/ewels/MultiQC_TestData contains test data for this change

[ewels]

This is maybe a little late to be asking this, but is there a reason that we can’t just take the value for mean coverage from the Qualimap output?

https://github.com/ewels/MultiQC_TestData/blob/310327cf2bbaa28b713f9b3e2e7c5b60b2e0ae74/data/modules/qualimap/bam_qc/ERR089819.qc/genome_results.txt#L61

[apeltzer]

Sure, can do that - makes probably sense to do that. The median isn't there so its fine to compute that "by hand".

[apeltzer]

done and adjusted 👍

[ewels]

Tested and works. Thanks again @apeltzer!