-
Notifications
You must be signed in to change notification settings - Fork 4
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 stub and update test #16
Conversation
|
subworkflows/local/get_orthologs.nf
Outdated
@@ -131,20 +131,22 @@ workflow GET_ORTHOLOGS { | |||
ch_versions | |||
.mix(FETCH_INSPECTOR_GROUP_ONLINE.out.versions) | |||
.set { ch_versions } | |||
// TODO check that this is correct |
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.
I believe this if is not needed, this is inside the use_all
block, which uses all databases no matter the other parameters (a concept from Cedric). In this case, local_databases
controls preference for local databases, rather than using only local ones. This is a bit complicated, but I believe it is necessary.
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.
Reverted
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.
Thanks for the tweaks Jose. I have one remark about EggNOG, the rest looks fully reasonable. Do we want review from another person before accepting this PR?
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.
LGTM now
Add stub run to local modules.
Update
test.config
with correct samplesheet input.Get rid of parameters inside local modules.
Get rid of unused
validateInputParameters
function and other "genome" stuff.PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).