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

Auto-Save Index Files Alongside BAMs #446

Open
wants to merge 298 commits into
base: main
Choose a base branch
from

Conversation

shadizaheri
Copy link
Collaborator

@shadizaheri shadizaheri commented Apr 11, 2024

Overview:
This branch is built on top of jts_kvg_sp_malaria branch. This PR improves our BAM file indexing process by ensuring index files are automatically saved alongside BAM files.

Key Updates:

  • outdir parameter in SRIndexBam workflow is now optional. The workflow places .bai files next to their .bam files if outdir isn't provided.

  • Adjusted FinalizeBamIndex to automatically detect and use the BAM file directory for storing .bai files.

Benefits:
Ensures .bai files are easily accessible for analysis.

Testing:
Tests confirm correct .bai file placement in specified and derived outdir cases, ensuring reliability across various scenarios.

jonn-smith and others added 30 commits January 10, 2023 12:33
…ally too large to benefit from Steve's ComputeAllowedLocalSSD optimization.
jonn-smith and others added 27 commits December 4, 2023 16:22
@shadizaheri shadizaheri marked this pull request as ready for review April 11, 2024 19:46
- Local disk to SSD

- samtools index -@ 2 ~{basename(prefix)}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants