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

read_merger.pl to support named pipes as inputs #106

Closed
wants to merge 1 commit into from

Conversation

clintval
Copy link

@clintval clintval commented Jan 5, 2018

Would you be willing to support named pipes when classifying with Kraken?

I store paired-end reads in BAM and, instead of storing the intermediate FASTQ on disk, stream the paired sequences into Kraken with named pipes. This offers good speedups as I am not writing to the filesystem when using BAM as a source file. Thanks!

Example:

❯ mkfifo /tmp/fastq{1,2}
❯ picard SamToFastq \
    INPUT=input.bam \
    FASTQ=/tmp/fastq1 \
    SECOND_END_FASTQ=/tmp/fastq2 \
  | kraken \
    --fastq-input \
    --paired \
    --db "${DBNAME}" \
    /tmp/fastq1 \
    /tmp/fastq2 \
    > output.txt

@clintval
Copy link
Author

clintval commented Jan 5, 2018

Actually. I just realized this is already in PR #89. I will wait for that!

@clintval clintval closed this Jan 5, 2018
@clintval clintval deleted the cv_named_pipes_support branch January 5, 2018 23:18
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.

1 participant