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

Mosdepth: Add additional summaries to general stats #2257

Merged
merged 10 commits into from Jan 8, 2024

Conversation

Joon-Klaps
Copy link
Contributor

@Joon-Klaps Joon-Klaps commented Jan 5, 2024

  • This comment contains a description of changes (with reason)

Mosdepth uses the file *.mosdepth.summary.txt for one variable mean_coverage. However more data is available in this file, and should be up to the user & not developer to include this in the general stats table or not. These extra statistics are:

  • Total coverage of bases
  • Total length of genome
  • Minimum coverage
  • Maxium coverage

@Joon-Klaps Joon-Klaps changed the title Add additional mosdepth summaries to General Stats Mosdepth: Add additional summaries to General Stats Jan 5, 2024
@Joon-Klaps Joon-Klaps changed the title Mosdepth: Add additional summaries to General Stats Mosdepth: Add additional summaries to general stats Jan 5, 2024
Copy link
Member

@vladsavelyev vladsavelyev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice addition! Thanks for the contribution

A few tweaks from me for the sake of column formatting and standardizing, and merging

@vladsavelyev vladsavelyev merged commit b05b4a3 into MultiQC:main Jan 8, 2024
7 checks passed
@jchorl jchorl mentioned this pull request Feb 18, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants