-
Notifications
You must be signed in to change notification settings - Fork 6
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
cram input necessary? #88
Comments
Hi @gbdias . You're absolutely right |
@gbdias We use CRAM files as input for several reasons: |
hi @yumisims thanks for the info. I can appreciate the advantage of streaming CRAM to bwa mem. I was just confused because the current code doesn't seem to use CRAM like that, in which case taking fastq directly would be faster. |
You're right @gbdias, the pipeline here doesn't implement the parallel aligner @yumisims is talking about, which requires CRAM, and is currently implemented in https://github.com/sanger-tol/treeval |
I was wondering if the cram input requirement for HiC data alignment makes sense since the first step of the pipeline is to convert back to fastq. Would it be ok to also accept fastq as input and skip this step?
readmapping/subworkflows/local/align_short.nf
Line 21 in a27e9ce
The text was updated successfully, but these errors were encountered: