Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ewels
Copy link
Member

@ewels ewels commented Jan 7, 2019

Continuation of #174 on a branch for group-work on the code.


New subcommand to interactively prompt the user to set workflow flags before launching.

For example:

$  nf-core launch methylseq

                                          ,--./,-.
          ___     __   __   __   ___     /,-._.--~\
    |\ | |__  __ /  ` /  \ |__) |__         }  {
    | \| |       \__, \__/ |  \ |___     \`-._,-`-,
                                          `._,._,'


INFO: Launching nf-core/methylseq


INFO: Main nextflow options:

Config profile to use [standard]:
Unique name for this nextflow run [False]: Phil's test run
Work directory for intermediate files [work]:
Resume a previous workflow run [False]: true
Release / revision to use [False]: 1.0

INFO: Pipeline specific parameters:

--mindepth [0]:
--nodedup [false]: true
--reads ['data/*_R{1,2}.fastq.gz']:
--numMismatches [0.6]:
--igenomes_base ['s3://ngi-igenomes/igenomes/']:
--container ['nfcore/methylseq:1.0']:
--notrim [false]: false
--max_memory ['128 GB']: 64 GB
--version ['1.0']:
--saveReference [false]:
--unmapped [false]: false
--max_cpus [16]:
--saveTrimmed [false]:
--non_directional [false]:
--comprehensive [false]:
--aligner ['bismark']:
--outdir ['./results']:
--relaxMismatches [false]:
--singleEnd [false]:
--saveAlignedIntermediates [false]:
--nf_required_version ['0.27.6']:
--ignoreFlags [false]:
--max_time ['10d']:
--multiqc_config ['/Users/ewels/.nextflow/assets/nf-core/methylseq/conf/multiqc_config.yaml']:

INFO: Nextflow command:
  nextflow run nf-core/methylseq -name "Phil's test run" -resume -r "1.0" --nodedup --max_memory "64 GB"


Do you want to run this command now? [y/N]: y

INFO: Launching!

This is a WIP - need to fill in the PR checklist still. It's a little crude, so any feedback welcome!

NB: This only works with local paths or workflows that have already been pulled (see nextflow-io/nextflow#889). For anything else, it does not ask for any params and just builds the basic command. This should probably be handled in a better way (a warning at least).

PR checklist

  • This comment contains a description of changes (with reason)
  • If you've fixed a bug or added code that should be tested, add tests!
  • Documentation in docs is updated
  • CHANGELOG.md is updated
  • README.md is updated

ewels added 2 commits October 5, 2018 23:57
New subcommand to interactively prompt the user to set workflow flags before launching.
nf-core launch: new subcommand
@ewels ewels added WIP Work in progress command line tools Anything to do with the cli interfaces labels Jan 7, 2019
@ewels ewels changed the title Nf core launch New CLI tool: nf-core launch Jan 7, 2019
@codecov-io
Copy link

codecov-io commented Jan 8, 2019

Codecov Report

Merging #241 into dev will decrease coverage by 6.36%.
The diff coverage is 57.39%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #241      +/-   ##
==========================================
- Coverage   91.21%   84.85%   -6.37%     
==========================================
  Files           8       12       +4     
  Lines         945     1208     +263     
==========================================
+ Hits          862     1025     +163     
- Misses         83      183     +100
Impacted Files Coverage Ξ”
nf_core/launch.py 0% <0%> (ΓΈ)
nf_core/workflow/workflow.py 0% <0%> (ΓΈ)
nf_core/workflow/validation.py 86% <86%> (ΓΈ)
nf_core/workflow/parameters.py 92.7% <92.7%> (ΓΈ)
nf_core/download.py 93.61% <0%> (+0.06%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data
Powered by Codecov. Last update fe5465c...66e13a0. Read the comment docs.

@ewels ewels changed the base branch from master to dev January 8, 2019 14:50
@ewels ewels removed the WIP Work in progress label Feb 8, 2019
@ewels
Copy link
Member Author

ewels commented Feb 8, 2019

@sven1103 - are we waiting on anything else to go in to this PR now? Can it be merged?

@sven1103
Copy link
Member

sven1103 commented Feb 8, 2019

If you are happy, I am happy with it :)

@sven1103 sven1103 merged commit 3310f7d into dev Feb 8, 2019
@ewels
Copy link
Member Author

ewels commented Feb 9, 2019

yay

@ewels ewels deleted the nf-core-launch branch February 9, 2019 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

command line tools Anything to do with the cli interfaces

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants