Enabling stdinput in the style of FastQC#89
Open
kerrycobb wants to merge 1 commit intosmithlabcode:masterfrom
Open
Enabling stdinput in the style of FastQC#89kerrycobb wants to merge 1 commit intosmithlabcode:masterfrom
kerrycobb wants to merge 1 commit intosmithlabcode:masterfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A feature of FastQC that is missing from falco is the ability to use stdin as input. I made a few changes to allow it to be done in the style of FastQC like
cat <fastq files> | falco stdin:<prefix>.I haven't tested it extensively but at a glance it looks like it's working with fastqc files. It assumes fastq format. I'm not sure how FastQC accomodates other formats as stdin. Perhaps with the --format flag.
I don't have much time right now to keep working on this. My fork is doing what I needed. Maybe this can inspire or help you to implement this feature. If you are receptive, I may be able to put some more effort in at some point to make this more robust and add some tests.