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

Kaiju: division by zero #2160

Closed
4 tasks done
kayobianco opened this issue Nov 5, 2023 · 1 comment · Fixed by #2179
Closed
4 tasks done

Kaiju: division by zero #2160

kayobianco opened this issue Nov 5, 2023 · 1 comment · Fixed by #2179
Labels
bug: module Bug in a MultiQC module

Comments

@kayobianco
Copy link

kayobianco commented Nov 5, 2023

Description of bug

Hi all,
I have the following problem.

(base) kayobianco@arqueas:~$ multiqc -m kaiju --outdir /home/kayobianco/Documentos/MiSeq_Arqueas_RBD/kaiju --filename kiaju_teste /home/kayobianco/Documentos/MiSeq_Arqueas_RBD/kaiju

  /// MultiQC 🔍 | v1.17

|           multiqc | Only using modules: kaiju
|           multiqc | Search path : /home/kayobianco/Documentos/MiSeq_Arqueas_RBD/kaiju
|         searching | ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 184/184  
|             kaiju | Found 46 reports
╭───────────────── Oops! The 'kaiju' 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:       │
│ /home/kayobianco/Documentos/MiSeq_Arqueas_RBD/kaiju/13_S13_L001_R1_001_kaiju │
│ .out.tsv                                                                     │
│                                                                              │
│ Traceback (most recent call last):                                           │
│   File "/home/kayobianco/miniconda3/lib/python3.8/site-packages/multiqc/mult │
│     output = mod()                                                           │
│   File "/home/kayobianco/miniconda3/lib/python3.8/site-packages/multiqc/modu │
│     self.kaiju_stats_table()                                                 │
│   File "/home/kayobianco/miniconda3/lib/python3.8/site-packages/multiqc/modu │
│     self.kaiju_sample_unclassified[s_name] * 100 / self.kaiju_sample_total_r │
│ ZeroDivisionError: division by zero                                          │
│                                                                              │
╰──────────────────────────────────────────────────────────────────────────────╯
|           multiqc | No analysis results found. Cleaning up..
|           multiqc | MultiQC complete

File that triggers the error

kaiju_teste.zip

MultiQC Error log

No response

Before submitting

  • I have read the troubleshooting documentation.
  • I am using the latest release of MultiQC.
  • I have included a full MultiQC log, not truncated.
  • I have attached an input file (.zip if necessary) that triggers the error.
@ewels ewels added the bug: module Bug in a MultiQC module label Nov 13, 2023
@ewels ewels added this to the MultiQC v1.18 milestone Nov 13, 2023
vladsavelyev added a commit to MultiQC/test-data that referenced this issue Nov 15, 2023
vladsavelyev added a commit to MultiQC/test-data that referenced this issue Nov 15, 2023
@vladsavelyev
Copy link
Member

Thank you @kayobianco for creating the issue,

I added a workaround for this in #2179

Note that your data doesn't seem to reproduce the error for me. The read numbers sum up to a non-zero count, and the module works fine without a division by zero error. So I added a made-up example into MultiQC_TestData: https://github.com/ewels/MultiQC_TestData/pull/303/files, but if you have a real example that triggers the error handy, please add it! Unless I'm misunderstanding something.

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

Successfully merging a pull request may close this issue.

3 participants