-
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
Template update #71
Template update #71
Conversation
conf/test_full.config
Outdated
sam2lca_identity = 0.8 | ||
sam2lca_acc2tax = 'adnamap' //'adnamap' for db build, 'nucl' for default | ||
kraken2_db = 'https://github.com/nf-core/test-datasets/raw/coproid/kraken.tar.gz' | ||
sp_sources = 'https://github.com/nf-core/test-datasets/raw/coproid/sourcepredict/test_sources.csv' | ||
sp_labels = 'https://github.com/nf-core/test-datasets/raw/coproid/sourcepredict/test_labels.csv' | ||
sp_sources = 'https://raw.githubusercontent.com/maxibor/sourcepredict/master/data/modern_gut_microbiomes_sources.csv' |
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.
These will need to be hosted on the coproID repository itself. Or in the nf-core test datasets repo.
conf/test_full.config
Outdated
sp_sources = 'https://github.com/nf-core/test-datasets/raw/coproid/sourcepredict/test_sources.csv' | ||
sp_labels = 'https://github.com/nf-core/test-datasets/raw/coproid/sourcepredict/test_labels.csv' | ||
sp_sources = 'https://raw.githubusercontent.com/maxibor/sourcepredict/master/data/modern_gut_microbiomes_sources.csv' | ||
sp_labels = 'https://raw.githubusercontent.com/maxibor/sourcepredict/master/data/modern_gut_microbiomes_labels.csv' |
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.
same as above
nextflow.config
Outdated
// nf-core: Optionally, you can add a pipeline-specific nf-core config at https://github.com/nf-core/configs | ||
includeConfig !System.getenv('NXF_OFFLINE') && params.custom_config_base ? "${params.custom_config_base}/pipeline/demo.config" : "/dev/null" | ||
// Load nf-core/coproid custom profiles from different institutions. | ||
// TODO nf-core: Optionally, you can add a pipeline-specific nf-core config at https://github.com/nf-core/configs |
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.
Need to remove TODO comments
main.nf
Outdated
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
*/ | ||
|
||
// TODO nf-core: Remove this line if you don't need a FASTA file |
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.
Need to remove todo comments
nf-core/coproid pull request
Many thanks for contributing to nf-core/coproid!
Please fill in the appropriate checklist below (delete whatever is not relevant).
These are the most common things requested on pull requests (PRs).
PR checklist
nextflow run . -profile test,singularity
).nf-core lint .
).docs
is updatedCHANGELOG.md
is updatedREADME.md
is updatedLearn more about contributing: CONTRIBUTING.md