nf-core / nf-core/modules

Update index handling in bam subworkflows

Open
#2,978 3 comments 1 reaction 1 assignee View on GitHub

@vucka is already working on this.

Since Mar 28, 2023.

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

Description

Is your feature request related to a problem? Please describe

In the bam_dedup_stats_samtools_umitools subworkflow some joins are used on line 31-32. These joins are dependent on the output of samtools_index. I think it would be better if the samtools_index module only emits one channel called index which contains the index created (with suffix bai, csi or crai as possible outputs). This would solve the issue with having to use remainder:true which causes bottlenecks in the pipeline run.

Describe the solution you'd like

No response

Describe alternatives you've considered

No response

Additional context

No response

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.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.