-
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
Update parameters #12
Conversation
…ions of various parameters for clarity
This PR is against the
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really good. Just minor suggestions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. A few comments. Once PR #11 is merged into dev
, can you merge dev
into this branch, which should remove some of the extra changes displayed here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good to me. Good to merge after PR #11
This PR updates various
profile_dists
andgas_call
parameters and adjusts their corresponding defaults based on requirements of the gasnomenclature pipeline.It adds additional definitions and pattern requirements to the nextflow_schema.json.
Documentation for the pipeline has also been updated.
PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).