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

BAM as input? #41

Closed
jfy133 opened this issue Oct 11, 2018 · 3 comments
Closed

BAM as input? #41

jfy133 opened this issue Oct 11, 2018 · 3 comments
Assignees
Labels
enhancement New feature or request feature minor

Comments

@jfy133
Copy link
Member

jfy133 commented Oct 11, 2018

Is your feature request related to a problem? Please describe.
Sometimes we receive/download only BAM files (e.g. see Slon et al. 2017 data on ENA), that we wish remap/reprocess in a different context. For example if the data is metagenomic, and we want to map to a particular bacterial genome. Currently we would have to manually re-convert the BAM to FASTQ, which leads to unnecessary data redundancy.

Describe the solution you'd like
An option to provide BAM as an input, rather than FASTQ. One solution would with a -bam flag, which would (turn-off a clip/merge module? and then) pipe stdout from samtools fastq into the mapper itself.

Describe alternatives you've considered
One could just manually re-convert the BAM.

@apeltzer apeltzer self-assigned this Oct 11, 2018
@apeltzer apeltzer added enhancement New feature or request feature labels Oct 11, 2018
@apeltzer apeltzer added this to the V2.1 "Ulm" milestone Oct 11, 2018
@apeltzer
Copy link
Member

Yup, that sounds good! I guess having a step to run on BAM directly would be quite easy to establish...

@apeltzer apeltzer added the minor label Oct 13, 2018
@apeltzer apeltzer changed the title BAM as input? [minor] BAM as input? Oct 13, 2018
@jfy133
Copy link
Member Author

jfy133 commented Dec 15, 2018

This may also require a skip adapter_removal step of some kind, in case you want to re-run already preprocessed data.

@apeltzer apeltzer mentioned this issue Jan 21, 2019
7 tasks
@apeltzer
Copy link
Member

This is addressed in #129

jfy133 added a commit that referenced this issue May 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature minor
Projects
None yet
Development

No branches or pull requests

2 participants