nf-core / nf-core/atacseq

bam_csi_index parameter for ATACSeq

Open
#343 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Nextflow
Stars
229
Forks
141
Avg merge
13h 41m
Merged PRs (30d)
1

Description

Description of feature

I had success when running atacseq (-r 2.1.2) for samples with small genome. When I tried to run other samples with big genome and one of the chromosome size is larger than 500Mb. I set the "--bam_csi_index". However, I still got error message "....cannot be stored in a bai index. Try using a csi index". It seems the atacseq pipeline does not implement the bam_csi_index parameter. It would be greatly appreciated if this parameter can be implemented in atacseq pipeline.

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 at the atacseq pipeline entry point and search for the existing bam index parameter and the step that creates or expects a BAI index. Trace how parameters are passed through the workflow, then verify that setting bam_csi_index selects CSI indexing for genomes with chromosomes larger than 500 Mb.

Written by the indexing model from the issue text.

Assessment

Domain
bioinformatics
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.