nf-core / nf-core/modules

Fix sentieon dedup metrics now that multiqc can handle it

Open
#8,927 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Nextflow
Stars
429
Forks
1.1k
Avg merge
1d 6h
Merged PRs (30d)
153

Description

Fairly sure we don't need this anymore, I think that multiqc can do this now.

Originally posted by @SPPearce in https://github.com/nf-core/modules/pull/8923#discussion_r2287537715

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with PR #8923 and its linked discussion, since no file or test is named. Establish which Sentieon dedup metric handling remains and what MultiQC already covers; done means the obsolete handling is no longer needed, with relevant checks passing.

Written by the indexing model from the issue text.

Assessment

Domain
bioinformatics
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.