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

Verify the user inputs are correct #42

Closed
mfoll opened this issue Oct 6, 2015 · 3 comments · Fixed by #45 or #57
Closed

Verify the user inputs are correct #42

mfoll opened this issue Oct 6, 2015 · 3 comments · Fixed by #45 or #57
Assignees
Milestone

Comments

@mfoll
Copy link
Member

mfoll commented Oct 6, 2015

For parameters:

  • sample_names
  • do_plots
  • all_sites

See here for example

@mfoll
Copy link
Member Author

mfoll commented Oct 7, 2015

It would be good to also check that the input files exists (fasta, index and bed) and that the BAM folder also exists (see http://www.nextflow.io/docs/latest/script.html#check-file-attributes). I think that you can do something like:

assert params.fasta_ref.exists()

@mfoll
Copy link
Member Author

mfoll commented Oct 14, 2015

values of 0 for map_qual and base_qualshould be valid (means no filter).

@mfoll
Copy link
Member Author

mfoll commented Oct 19, 2015

It was not necessary a good idea for everything: some files might only be accessible during the execution of the processes (for example if you put the fasta ref on the scratch of the nodes of a cluster). We could either move the checks to later points (in the processes themselves) or add an option to prevent the checks, or even remove them?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants