Skip to content
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

Module VEP broke #1597

Closed
quentin67100 opened this issue Nov 26, 2021 · 4 comments
Closed

Module VEP broke #1597

quentin67100 opened this issue Nov 26, 2021 · 4 comments
Labels
bug: module Bug in a MultiQC module

Comments

@quentin67100
Copy link

Description of bug

I tried to use MultiQC with singularity. But it failled to add VEP stats. I also tried with the conda version of multiqc but it failled too.

File that triggers the error

Pologne.stat.vep.html.zip
,

MultiQC Error log

/// MultiQC 🔍 | v1.11

|           multiqc | Search path : /shared/projects/gentaumix/Pologne
|          qualimap | Found 4 BamQC reports
|    MarkDuplicates | Skipping MarkDuplicates sample 'multiqc_data' as missing essential fields
|    MarkDuplicates | Skipping MarkDuplicates sample 'multiqc_data' as missing essential fields
|    MarkDuplicates | Skipping MarkDuplicates sample 'multiqc_data' as missing essential fields
|    MarkDuplicates | Skipping MarkDuplicates sample 'multiqc_data' as missing essential fields
|    MarkDuplicates | Skipping MarkDuplicates sample 'multiqc_data' as missing essential fields
|            picard | Found 4 MarkDuplicates reports
|            picard | Found 8 ValidateSamFile reports
╭────────────────── Oops! The 'vep' MultiQC module broke... ───────────────────╮
│ Please copy this log and report it at                                        │
│ https://github.com/ewels/MultiQC/issues                                      │
│ Please attach a file that triggers the error. The last file found was:       │
│ /shared/projects/gentaumix/Pologne//06_VEP/Pologne.stat.vep.html             │
│                                                                              │
│ Traceback (most recent call last):                                           │
│   File "/usr/lib/python3.8/site-packages/multiqc/multiqc.py", line 624, in r │
│     output = mod()                                                           │
│   File "/usr/lib/python3.8/site-packages/multiqc/modules/vep/vep.py", line 3 │
│     self.parse_vep_html(f)                                                   │
│   File "/usr/lib/python3.8/site-packages/multiqc/modules/vep/vep.py", line 1 │
│     existing = values[1].split("(")[0].replace(" ", "")                      │
│ IndexError: list index out of range                                          │
│                                                                              │
╰──────────────────────────────────────────────────────────────────────────────╯
|      fastq_screen | Found 8 reports
|            fastqc | Found 8 reports
|           multiqc | Compressing plot data
|           multiqc | Report      : ../../../../../projects/gentaumix/Pologne/Pologne_MultiQC_global.html
|           multiqc | Data        : ../../../../../projects/gentaumix/Pologne/Pologne_MultiQC_global_data
|           multiqc | MultiQC complete
@ewels
Copy link
Member

ewels commented Dec 13, 2021

@maxulysse - are you interested in taking a look at this?

@ewels ewels added the bug: module Bug in a MultiQC module label Dec 13, 2021
@project-defiant
Copy link

project-defiant commented Jan 4, 2022

I had similar issue with v.1.11 from bioconda but with the output from annotating empty vcf file

================================================================================
Traceback (most recent call last):
  File "/home/szszyszkowski/.local/lib/python3.8/site-packages/multiqc/multiqc.py", line 651, in run
    output = mod()
  File "/home/szszyszkowski/.local/lib/python3.8/site-packages/multiqc/modules/vep/vep.py", line 37, in __init__
    self.parse_vep_html(f)
  File "/home/szszyszkowski/.local/lib/python3.8/site-packages/multiqc/modules/vep/vep.py", line 131, in parse_vep_html
    existing = values[1].split("(")[0].replace(" ", "")
IndexError: list index out of range

Similar output is produced if using *.vep.txt files as input. I expect it to be possibly due to lack of some values in the [General Statistics] section

...
[General statistics]
Lines of input read
Variants processed
Variants filtered out	0
Novel / existing variants	-
Overlapped genes	0
Overlapped transcripts	0
Overlapped regulatory features	-
...

@maxulysse
Copy link
Contributor

@ewels I'd be happy to have a look, sorry for the late reply

ewels added a commit to MultiQC/test-data that referenced this issue Jan 27, 2022
@ewels ewels closed this as completed in 1977df6 Jan 27, 2022
@ewels
Copy link
Member

ewels commented Jan 27, 2022

Hi all,

This should now be fixed in 1977df6 - table cells containing - are not reported and will be empty in the MultiQC table now (the entire column will be missing if all samples had this). The module should no longer crash.

Thanks for reporting and providing an example report 👍🏻

Phil

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug: module Bug in a MultiQC module
Projects
None yet
Development

No branches or pull requests

4 participants