-
Notifications
You must be signed in to change notification settings - Fork 0
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
add CRAM input handling #41
Comments
@yashpatel6 any thoughts regarding PipeVal handling? Is there a way to provide it a reference file through the pipeline? |
I think the way forward will be to generalize the PipeVal process used within Nextflow to allow additional inputs and options |
Sorry I don't understand, are you recommending changes to the external PipeVal tool independent of pipeline-targeted-coverage or are you recommending changes internal to pipeline-targeted-coverage, e.g. modifying this code? |
I mean the Nextflow process we have for PipeVal (ie. the one being imported here: https://github.com/uclahs-cds/pipeline-targeted-coverage/blob/f3a5959d24538b445f83b1438786a2b9e92f13b8/main.nf#L6) needs to be generalized to allow for additional options |
CRAM formatted inputs require some modifications of the pipeline:
The text was updated successfully, but these errors were encountered: