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

merge dsl2 into dev #417

Merged
merged 559 commits into from
Aug 3, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
559 commits
Select commit Hold shift + click to select a range
159fe57
fix: simplify markduplicates subworkflow
maxulysse Mar 23, 2021
86e76e5
fix: --input tests with folder
maxulysse Mar 23, 2021
6d48b9f
fix: extension for samtools_merge module
maxulysse Mar 23, 2021
e1abc27
fix: fix haplotypecaller
maxulysse Mar 23, 2021
64ea14f
feat: update bwa/bwamem2 containers
maxulysse Mar 23, 2021
6b57759
Merge pull request #358 from MaxUlysse/dsl2_modules
maxulysse Mar 23, 2021
11bcd92
fix: correct hash for mulled containers to fix #364
maxulysse Mar 29, 2021
656bb85
Merge pull request #366 from MaxUlysse/dsl2_bwa_bwamem2
maxulysse Mar 30, 2021
85d6c0d
feat: remove social preview image to use GitHub OpenGraph
maxulysse Apr 26, 2021
cfe37e4
Merge pull request #369 from maxulysse/dsl2
ewels Apr 26, 2021
fa5576c
feat: add pytest workflow for default and aligner
maxulysse Apr 27, 2021
db6a6cf
ci: Add local modules pytest-workflow workflow
edmundmiller Apr 27, 2021
96b012b
test: Add infrastructure
edmundmiller Apr 27, 2021
f517e06
test(markduplicates): Add initial test
edmundmiller Apr 27, 2021
6c8397e
test: Add modules config for default settings
edmundmiller Apr 27, 2021
737cedf
Merge pull request #6 from Emiller88/dsl2-subworkflow-tests
maxulysse Apr 28, 2021
00fc1e7
Apply suggestions from code review
maxulysse May 11, 2021
645bdc5
feat: immprove tests
maxulysse May 11, 2021
b22e28a
fix: comments
maxulysse May 11, 2021
39d19f8
feat: add gatk4 tests
maxulysse May 11, 2021
b3b82a4
fix: no need for gatk4 tests here
maxulysse May 11, 2021
a2a0818
feat: more tests, and more complete
maxulysse May 11, 2021
4c5e0a8
feat: add tests for variant calling
maxulysse May 11, 2021
765d017
feat: more tests incoming with pytest
maxulysse May 11, 2021
c016b59
feat: clean up tests
maxulysse May 12, 2021
13f61c1
Merge pull request #370 from maxulysse/dsl2_tests
maxulysse May 19, 2021
465ceae
Merge remote-tracking branch 'upstream/dev' into dsl2
maxulysse May 19, 2021
7d90b4d
fix: Nextflow version
maxulysse May 19, 2021
d8a5722
Merge remote-tracking branch 'upstream/dev' into dsl2
maxulysse May 19, 2021
617f407
Apply suggestions from code review
maxulysse May 20, 2021
1b7432c
Merge pull request #377 from maxulysse/dsl2
maxulysse May 20, 2021
1e0edf8
feat: update main script and lib from DSL2 Best Practices
maxulysse May 20, 2021
b120e90
fix: add more file to lint ignore
maxulysse May 20, 2021
b5fc96c
feat: strip out unused functions
maxulysse May 20, 2021
34a0b4d
feat: update some modules
maxulysse May 20, 2021
14d5b97
fix: HaplotypeCaller
maxulysse May 20, 2021
bcc8385
feat: update msisensor
maxulysse May 25, 2021
f68fa78
feat: update msisensor_msi
maxulysse May 25, 2021
b86fc4f
fix: Gstring.toString()
maxulysse May 25, 2021
7f55703
fix: output
maxulysse May 25, 2021
7be61c8
remove unused file
maxulysse May 25, 2021
6291ec5
remove white space
maxulysse May 25, 2021
8f6e461
fix: output
maxulysse May 25, 2021
ce7b5ee
fix: markduplicates subworkflow test
maxulysse May 26, 2021
4141f64
fix: subworkflow md finally fixed
maxulysse May 26, 2021
99dd4ea
fix: nextflow version
maxulysse May 26, 2021
b1d14e0
moving files and functions around
maxulysse May 27, 2021
8db1c6f
feat: update modules and move around functions, and possibly break ev…
maxulysse May 28, 2021
151d600
fix: add .gitignore to .nf-core-lint.yml
maxulysse May 28, 2021
712e29d
fix: add new csv files
maxulysse May 28, 2021
b83e31f
feat: simplify mapping script, update module, default test working
maxulysse May 28, 2021
ea0b43e
feat: cleanup files
maxulysse May 28, 2021
266772b
cleanup
maxulysse May 28, 2021
c371418
feat: simplify variant_calling
maxulysse May 28, 2021
6c3299d
fix: output
maxulysse May 30, 2021
77a5c65
feat: simplify tests
maxulysse May 30, 2021
4fbbf80
fix: tests
maxulysse May 30, 2021
7750ce9
feat: simplify test profiles
maxulysse Jun 3, 2021
55852ba
feat: refactor + reorganize modules
maxulysse Jun 3, 2021
d140635
feat: refactor - forgot to commit file
maxulysse Jun 3, 2021
7e6b232
fix: gatk4/markduplicatesspark
maxulysse Jun 4, 2021
ad88088
fix: subworfklow tests
maxulysse Jun 7, 2021
980b702
fix: markduplicats tests
maxulysse Jun 7, 2021
4332455
feat: update bwa/bwamem2 modules
maxulysse Jun 7, 2021
2b3b55c
feat: update trimgalore
maxulysse Jun 7, 2021
7ed1a92
feat: update tabix/tabix
maxulysse Jun 7, 2021
7b2a7a3
feat: update and add strelka modules
maxulysse Jun 7, 2021
61eee21
fix: typo
maxulysse Jun 7, 2021
b214c5d
fix: path to module
maxulysse Jun 7, 2021
0f1fba5
fix: typo
maxulysse Jun 7, 2021
04d97f4
feat: update samtools/merge
maxulysse Jun 7, 2021
b4a903c
feat: update multiqc modules
maxulysse Jun 7, 2021
7dc1c86
feat: replace msisensor by msisensor-pro
maxulysse Jun 7, 2021
8d10fc4
fix: path to modules
maxulysse Jun 7, 2021
54bd1d4
fix: msisensor-pro test
maxulysse Jun 7, 2021
7d4547b
fix: msisensorpro execution7
maxulysse Jun 7, 2021
28b232b
feat: code polishing
maxulysse Jun 7, 2021
214a28b
fix: improve csv files generation
maxulysse Jun 7, 2021
6b23e8e
feat: code polishing
maxulysse Jun 7, 2021
f52c318
fix: update CI
maxulysse Jun 7, 2021
dacc326
fix: add csv for recalibrated bam files
maxulysse Jun 7, 2021
b092ea3
feat: restart from all steps
maxulysse Jun 7, 2021
0dd07ea
fix: gatk4spark tests
maxulysse Jun 8, 2021
68e1cc4
feat: code polishing
maxulysse Jun 8, 2021
a7b829f
feat: remove no longer used files
maxulysse Jun 8, 2021
2b705c9
feat: move modules from local to nf-core
maxulysse Jun 8, 2021
39fd254
Merge pull request #379 from maxulysse/dsl2_modules_update
maxulysse Jun 11, 2021
81bcdcb
Merge branch 'dsl2' into dsl2_annotation
maxulysse Jun 11, 2021
e8570cb
fix: restore samtools to 1.12
maxulysse Jun 11, 2021
531eecc
Merge branch 'dsl2_annotation' of github.com:maxulysse/nf-core_sarek …
maxulysse Jun 11, 2021
6ce647d
Add cram support, read splitting
FriederikeHanssen Jun 15, 2021
21a695a
Add estimate library complexity if spark is used
FriederikeHanssen Jun 16, 2021
a634a79
Fixes resume problem, but is losing the file name...
FriederikeHanssen Jun 16, 2021
1632d4b
Add tmp dir to gatk processes so tmp files are written to the proper …
FriederikeHanssen Jun 16, 2021
ab0c1c8
Fix filename display + resume for TABIX
FriederikeHanssen Jun 16, 2021
e8290a1
Try to get spark to work
FriederikeHanssen Jun 16, 2021
809d321
Add MDSpark back in
FriederikeHanssen Jun 16, 2021
6ba8720
try with runoptions
FriederikeHanssen Jun 16, 2021
46dcdc2
The newest gatk container is not working for me with spark, 4.1.9.0 is
FriederikeHanssen Jun 16, 2021
41649cc
Add docker.userEmulation back in
FriederikeHanssen Jun 16, 2021
1aecc44
fix: pipeline reports filename
maxulysse Jun 17, 2021
5ace72f
fix: remove default: 'null'
maxulysse Jun 17, 2021
96341f9
fix: json schema
maxulysse Jun 17, 2021
a838bca
fix: pattern
maxulysse Jun 17, 2021
29318b5
fix: json
maxulysse Jun 17, 2021
9594ae7
fix: test
maxulysse Jun 17, 2021
f4450f7
fix: msisensor -> msisensorpro
maxulysse Jun 17, 2021
06d9585
Update nextflow_schema.json
maxulysse Jun 18, 2021
df65564
Merge pull request #390 from maxulysse/dsl2_fixes
maxulysse Jun 18, 2021
8205da4
Merge remote-tracking branch 'upstream/dsl2' into dsl2_annotation
maxulysse Jun 18, 2021
30bf2f4
feat: add annotation
maxulysse Jun 18, 2021
d79d39b
fix: update json
maxulysse Jun 18, 2021
cca46d7
feat: add tests for annotation
maxulysse Jun 18, 2021
df4fb5a
fix: output
maxulysse Jun 18, 2021
b4dd4ca
Add more spark things
FriederikeHanssen Jun 21, 2021
147d6b4
Fix module params
FriederikeHanssen Jun 21, 2021
53d69fa
Publish ref
FriederikeHanssen Jun 21, 2021
5b4fc53
try with path instead of file
FriederikeHanssen Jun 21, 2021
1158010
try with path instead of file
FriederikeHanssen Jun 21, 2021
9b1768f
try with fromFile instead
FriederikeHanssen Jun 21, 2021
6323fbe
file
FriederikeHanssen Jun 21, 2021
838932f
whole fixownership seems to work
FriederikeHanssen Jun 21, 2021
dd4f783
Merge remote-tracking branch 'upstream/dsl2' into dsl2
FriederikeHanssen Jun 21, 2021
39403ec
Add numLanes to meta sheet to deal with blocked mapping output channels
FriederikeHanssen Jun 22, 2021
a62acb1
Add channel dumping to check for missing id
FriederikeHanssen Jun 22, 2021
cd51cbc
use groupKey instead
FriederikeHanssen Jun 22, 2021
21c168e
not sure if this works, but run a bigger test for this
FriederikeHanssen Jun 22, 2021
d4354d6
Simplify mapping epression
FriederikeHanssen Jun 23, 2021
d6a6f03
fix: merge + add tests
maxulysse Jun 23, 2021
7f886cb
feat: containers à la carte
maxulysse Jun 23, 2021
01a0293
feat: move args to conf/modules.config
maxulysse Jun 23, 2021
26448a6
remove unused sw & add ref to samtools stats
FriederikeHanssen Jun 23, 2021
87b6831
Add skip_fastqc in again
FriederikeHanssen Jun 23, 2021
8b9812e
fix: get rid of warning message
maxulysse Jun 23, 2021
a3ab368
fix: code polishing
maxulysse Jun 23, 2021
5184462
try with exporting ref path and cache
FriederikeHanssen Jun 23, 2021
46effa4
Remove quotes
FriederikeHanssen Jun 23, 2021
70d28c0
add mutect2 somatic module
FriederikeHanssen Jun 23, 2021
a920909
Try to circumvent stats issues with view
FriederikeHanssen Jun 23, 2021
2047dc8
Add ref to cram merge
FriederikeHanssen Jun 24, 2021
112aaf0
Add ref to cram merge
FriederikeHanssen Jun 24, 2021
52632e6
Use double quotes for output
FriederikeHanssen Jun 24, 2021
144fef7
Add ref to stats
FriederikeHanssen Jun 24, 2021
44c5bdf
fix logic for bam to cram conversion
FriederikeHanssen Jun 24, 2021
cc749c5
feat: add tests for subworkflow annotate
maxulysse Jun 24, 2021
1d5b395
Simplify if
FriederikeHanssen Jun 24, 2021
44594c5
fix: annotate tests
maxulysse Jun 24, 2021
6a77b3c
Add memory overhead for gatk based tools
FriederikeHanssen Jun 24, 2021
5ab6078
Add mutect2 somatic
FriederikeHanssen Jun 24, 2021
3da89f7
code polishing
maxulysse Jun 24, 2021
11f39a6
feat: add test for snpeff_annotate subworkflow
maxulysse Jun 24, 2021
b0982ac
feat: add test for vep_annotate subworkflow
maxulysse Jun 24, 2021
bf2a182
fix: update meta.yml file for snpEff
maxulysse Jun 25, 2021
1a9d53c
fix: update meta.yml file for VEP
maxulysse Jun 25, 2021
e1e0d34
add conf
FriederikeHanssen Jun 25, 2021
b7781ea
chores: update files
maxulysse Jun 25, 2021
dbe4bdb
remove failing dumo statement
FriederikeHanssen Jun 25, 2021
951b78d
select spark tools
FriederikeHanssen Jun 25, 2021
f08901f
change use_gatk_spark in bwamem2
FriederikeHanssen Jun 26, 2021
d44e522
change use_gatk_spark in bwamem2
FriederikeHanssen Jun 26, 2021
dda8b13
add dump tag to figure out why bqsr is not working
FriederikeHanssen Jun 27, 2021
db3d98a
try withotu clone to get to work on aws
FriederikeHanssen Jun 27, 2021
287146f
remove meta.id
FriederikeHanssen Jun 27, 2021
2732291
USe channels for known sites
FriederikeHanssen Jun 27, 2021
c398e01
Try to fix known_sites channel
FriederikeHanssen Jun 27, 2021
a665e66
add groupTuple back in
FriederikeHanssen Jun 27, 2021
bccdb7d
change dbsnp/knownindels channel
FriederikeHanssen Jun 27, 2021
118f9c1
fix multiple knwonindels input
FriederikeHanssen Jun 27, 2021
c2e73c6
add dump statements, why are the intervals not working for humans
FriederikeHanssen Jun 27, 2021
d5cec16
sth of when providing multiple indices
FriederikeHanssen Jun 27, 2021
777f7a2
add tbi back in
FriederikeHanssen Jun 27, 2021
286bbe3
concat seems to fix this channel madness
FriederikeHanssen Jun 27, 2021
3fd8078
hardcode number of intervals for tests
FriederikeHanssen Jun 27, 2021
493a147
fix docker image tag, can't find singularity one
FriederikeHanssen Jun 27, 2021
dd88aa3
add haplotypecalelr back in
FriederikeHanssen Jun 27, 2021
ec7a2d5
count num intervals with map oprator
FriederikeHanssen Jun 28, 2021
1be7416
collect dbsnp tbi to avoid consumption of channel
FriederikeHanssen Jun 28, 2021
206db89
add gvcf back in
FriederikeHanssen Jun 28, 2021
9ce70a0
Add bamqc after bqsr with crams
FriederikeHanssen Jun 29, 2021
047e5cf
Use docker image for htslib + singularity
FriederikeHanssen Jun 29, 2021
88ac395
Merge pull request #386 from maxulysse/dsl2_annotation
maxulysse Jun 29, 2021
19c911a
add dbsnp back in
FriederikeHanssen Jun 29, 2021
2342971
add dbsnp back in
FriederikeHanssen Jun 29, 2021
5e8f2cc
Resolve merge conflicts
FriederikeHanssen Jun 29, 2021
4f974f6
add step/tools to indices wf
FriederikeHanssen Jun 29, 2021
d766d44
Resolve remaining merge conflicts/fix problems
FriederikeHanssen Jun 29, 2021
3a37778
add try/catch to figure out why module conf is not loaded
FriederikeHanssen Jun 29, 2021
22d55ab
Split by num reads instead of parts to generate similar sized files
FriederikeHanssen Jul 2, 2021
0fa42ea
Add the basic stubs for deepvariant
abhi18av Jul 6, 2021
f038a67
Integrate stubs into the germline caller subworkflow
abhi18av Jul 7, 2021
0312af1
Update the modules conf for deepvariant [ci skip]
abhi18av Jul 12, 2021
f360c13
Tweak the output channel [ci skip]
abhi18av Jul 12, 2021
b0bd8e9
Link the output of deepvariant to germline variant caller [ci skip]
abhi18av Jul 12, 2021
1e94be3
Update version scraper [ci skip]
abhi18av Jul 12, 2021
cfdb91c
Update dv for cpus and minor fixes [ci skip]
abhi18av Jul 12, 2021
dc14d75
Template update for nf-core/tools version 2.0
KevinMenden Jul 13, 2021
9e5fadb
Template update for nf-core/tools version 2.0.1
nf-core-bot Jul 13, 2021
b446514
Accomodate the code review
abhi18av Jul 15, 2021
18d5b61
Merge pull request #394 from abhi18av/abhinav/deepvariant
maxulysse Jul 15, 2021
3f26535
Code clean up
FriederikeHanssen Jul 15, 2021
069a4f1
Fix merge conflicts
FriederikeHanssen Jul 15, 2021
f5bf99c
Merge remote-tracking branch 'origin/dsl2' into nf-core-template-merg…
maxulysse Jul 15, 2021
1866022
apply suggestions from code review
FriederikeHanssen Jul 15, 2021
26a1e23
Merge pull request #388 from FriederikeHanssen/dsl2
maxulysse Jul 15, 2021
32c65e2
Merge branch 'dsl2' into nf-core-template-merge-2.0.1
maxulysse Jul 15, 2021
e8da52a
fix: markdownlint
maxulysse Jul 15, 2021
993802f
fix: ECLint
maxulysse Jul 15, 2021
341a9ea
fix: ECLint [skip ci]
maxulysse Jul 15, 2021
5032791
fix: rename file
maxulysse Jul 15, 2021
ffd83f1
fix: json
maxulysse Jul 15, 2021
dd1b8e8
fix: modules
maxulysse Jul 16, 2021
02aa6cc
fix: more nf-core lint
maxulysse Jul 16, 2021
08a40c2
fix: files unchanged
maxulysse Jul 16, 2021
1ab34c4
fix: more nf-core lint
maxulysse Jul 16, 2021
dfcd96e
fix: nf-core lint files unchanged
maxulysse Jul 16, 2021
708fe16
fix: modules ensemblvep
maxulysse Jul 16, 2021
8adc6c7
fix: modules tabix/bgziptabix
maxulysse Jul 16, 2021
a6db228
fix: modules snpeff
maxulysse Jul 16, 2021
17dc608
fix: modules gatk4/createsequencedictionary
maxulysse Jul 16, 2021
b4fd1b6
fix: modules gatk4/createsequencedictionary
maxulysse Jul 16, 2021
e1700e4
fix: update seqkit/split2
maxulysse Jul 16, 2021
7ee0a46
fix: update samtools/merge
maxulysse Jul 16, 2021
da74c55
fix: modules go to local
maxulysse Jul 16, 2021
3e14948
fix: more modules to local
maxulysse Jul 16, 2021
092af71
fix: EClint again
maxulysse Jul 16, 2021
c39fb2f
fix: EClint again
maxulysse Jul 16, 2021
84af6f8
fix: nf-core lint
maxulysse Jul 16, 2021
94951d6
fix: workflow is back
maxulysse Jul 16, 2021
2f69572
fix: bwa/mem and bwamem2/mem modules
maxulysse Jul 16, 2021
7844ee8
fix: update modules
maxulysse Jul 16, 2021
472e721
fix: genome loading in profiles
maxulysse Jul 16, 2021
b0d2798
fix: multiqc args
maxulysse Jul 16, 2021
2ad6bcd
fix: bam -> cram
maxulysse Jul 19, 2021
675bca9
fix: merge output names
maxulysse Jul 19, 2021
fa99d8e
Merge pull request #408 from maxulysse/nf-core-template-merge-2.0.1
maxulysse Jul 19, 2021
c41a127
fix: update CI tests
maxulysse Jul 19, 2021
6a270cb
fix: remove variant_calling test
maxulysse Jul 19, 2021
9c35d4d
fix: annotate
maxulysse Jul 19, 2021
dd7bb00
fix: multiqc output for CI
maxulysse Jul 19, 2021
fb97387
fix: remove only msi tests
maxulysse Jul 19, 2021
05194ed
Merge pull request #410 from maxulysse/dsl2_CI
maxulysse Jul 19, 2021
09c4b75
fix: ban -> cram in CSV
maxulysse Jul 19, 2021
c301e74
fix: CSV back in CI tests
maxulysse Jul 19, 2021
2fae194
Merge pull request #411 from maxulysse/dsl2_updates
maxulysse Jul 19, 2021
8244911
Template update for nf-core/tools version 2.1
nf-core-bot Jul 27, 2021
5f5330c
Merge branch 'dsl2' into nf-core-template-merge-2.1
maxulysse Jul 28, 2021
8f95fd8
Merge pull request #416 from nf-core/nf-core-template-merge-2.1
maxulysse Jul 28, 2021
c17aeb5
Merge remote-tracking branch 'upstream/dsl2' into dev
maxulysse Jul 28, 2021
f8ba00d
fix: Mardownlint in CHANGELOG
maxulysse Jul 28, 2021
62b5f7a
fix: code polish
maxulysse Aug 3, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
109 changes: 0 additions & 109 deletions .circleci/config.yml

This file was deleted.

27 changes: 27 additions & 0 deletions .editorconfig
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
root = true

[*]
charset = utf-8
end_of_line = lf
insert_final_newline = true
trim_trailing_whitespace = true
indent_size = 4
indent_style = space

[*.{yml,yaml}]
indent_size = 2

[*.json]
insert_final_newline = unset

# These files are edited and tested upstream in nf-core/modules
[/modules/nf-core/**]
charset = unset
end_of_line = unset
insert_final_newline = unset
trim_trailing_whitespace = unset
indent_style = unset
indent_size = unset

[/assets/email*]
indent_size = unset
2 changes: 1 addition & 1 deletion .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -1 +1 @@
* @MaxUlysse
* @maxulysse
8 changes: 4 additions & 4 deletions .github/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ If you'd like to write some code for nf-core/sarek, the standard workflow is as
* If there isn't one already, please create one so that others know you're working on this
2. [Fork](https://help.github.com/en/github/getting-started-with-github/fork-a-repo) the [nf-core/sarek repository](https://github.com/nf-core/sarek) to your GitHub account
3. Make the necessary changes / additions within your forked repository following [Pipeline conventions](#pipeline-contribution-conventions)
4. Use `nf-core schema build .` and add any new parameters to the pipeline JSON schema (requires [nf-core tools](https://github.com/nf-core/tools) >= 1.10).
4. Use `nf-core schema build` and add any new parameters to the pipeline JSON schema (requires [nf-core tools](https://github.com/nf-core/tools) >= 1.10).
5. Submit a Pull Request against the `dev` branch and wait for the code to be reviewed and merged

If you're not used to this workflow with git, you can start with some [docs from GitHub](https://help.github.com/en/github/collaborating-with-issues-and-pull-requests) or even their [excellent `git` resources](https://try.github.io/).
Expand Down Expand Up @@ -69,12 +69,12 @@ If you wish to contribute a new step, please use the following coding standards:
2. Write the process block (see below).
3. Define the output channel if needed (see below).
4. Add any new flags/options to `nextflow.config` with a default (see below).
5. Add any new flags/options to `nextflow_schema.json` with help text (with `nf-core schema build .`).
5. Add any new flags/options to `nextflow_schema.json` with help text (with `nf-core schema build`).
6. Add any new flags/options to the help message (for integer/text parameters, print to help the corresponding `nextflow.config` parameter).
7. Add sanity checks for all relevant parameters.
8. Add any new software to the `scrape_software_versions.py` script in `bin/` and the version command to the `scrape_software_versions` process in `main.nf`.
9. Do local tests that the new code works properly and as expected.
10. Add a new test command in `.github/workflow/ci.yaml`.
10. Add a new test command in `.github/workflow/ci.yml`.
11. If applicable add a [MultiQC](https://https://multiqc.info/) module.
12. Update MultiQC config `assets/multiqc_config.yaml` so relevant suffixes, name clean up, General Statistics Table column order, and module figures are in the right order.
13. Optional: Add any descriptions of MultiQC report sections and output files to `docs/output.md`.
Expand All @@ -83,7 +83,7 @@ If you wish to contribute a new step, please use the following coding standards:

Parameters should be initialised / defined with default values in `nextflow.config` under the `params` scope.

Once there, use `nf-core schema build .` to add to `nextflow_schema.json`.
Once there, use `nf-core schema build` to add to `nextflow_schema.json`.

### Default processes resource requirements

Expand Down
5 changes: 1 addition & 4 deletions .github/ISSUE_TEMPLATE/bug_report.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
---
name: Bug report
about: Report something that is broken or incorrect
title: "[BUG]"
labels: bug
assignees: maxulysse
---

<!--
Expand Down Expand Up @@ -53,13 +51,12 @@ Have you provided the following extra information/files:

## Nextflow Installation

- Version: <!-- [e.g. 19.10.0] -->
- Version: <!-- [e.g. 21.04.0] -->

## Container engine

- Engine: <!-- [e.g. Conda, Docker, Singularity, Podman, Shifter or Charliecloud] -->
- version: <!-- [e.g. 1.0.0] -->
- Image tag: <!-- [e.g. nfcore/sarek:2.7] -->

## Additional context

Expand Down
2 changes: 0 additions & 2 deletions .github/ISSUE_TEMPLATE/feature_request.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
---
name: Feature request
about: Suggest an idea for the nf-core/sarek pipeline
title: "[FEATURE]"
labels: enhancement
assignees: maxulysse
---

<!--
Expand Down
3 changes: 1 addition & 2 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,9 @@ Learn more about contributing: [CONTRIBUTING.md](https://github.com/nf-core/sare

- [ ] This comment contains a description of changes (with reason).
- [ ] If you've fixed a bug or added code that should be tested, add tests!
- [ ] If you've added a new tool - add to the software_versions process and a regex to `scrape_software_versions.py`
- [ ] If you've added a new tool - have you followed the pipeline conventions in the [contribution docs](https://github.com/nf-core/sarek/tree/master/.github/CONTRIBUTING.md)
- [ ] If necessary, also make a PR on the nf-core/sarek _branch_ on the [nf-core/test-datasets](https://github.com/nf-core/test-datasets) repository.
- [ ] Make sure your code lints (`nf-core lint .`).
- [ ] Make sure your code lints (`nf-core lint`).
- [ ] Ensure the test suite passes (`nextflow run . -profile test,docker`).
- [ ] Usage Documentation in `docs/usage.md` is updated.
- [ ] Output Documentation in `docs/output.md` is updated.
Expand Down
32 changes: 16 additions & 16 deletions .github/RELEASE_CHECKLIST.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,26 +3,26 @@
> This checklist is for our own reference, to help us prepare a new release

1. Check that everything is ready to go
- Desired [PRs](https://github.com/nf-core/sarek/pulls) are merged
- [GitHub Actions](https://github.com/nf-core/sarek/actions?query=workflow%3A%22nf-core+CI%22) are passing on `dev`
- [nf-core linting](https://github.com/nf-core/sarek/actions?query=workflow%3A%22nf-core+linting%22) are passing on `dev`
- Desired [PRs](https://github.com/nf-core/sarek/pulls) are merged
- [GitHub Actions](https://github.com/nf-core/sarek/actions?query=workflow%3A%22nf-core+CI%22) are passing on `dev`
- [nf-core linting](https://github.com/nf-core/sarek/actions?query=workflow%3A%22nf-core+linting%22) are passing on `dev`
2. Increase version number following [semantic versioning](http://semver.org/spec/v2.0.0.html)
3. Choose an appropriate codename for the release (if major or minor)
- i.e. Peaks in [Sarek National Park](https://en.wikipedia.org/wiki/Sarek_National_Park#Topography)
- i.e. Peaks in [Sarek National Park](https://en.wikipedia.org/wiki/Sarek_National_Park#Topography)
4. Sync `dev` and checkout a new branch for the release
5. Bump version:
- `nf-core bump-version . <VERSION>`
- edit `.circleci/config.yml`
- edit `.github/workflows/ci.yml`
- edit `conf/base.config`
- edit `conf/test.config`
- edit `containers/snpeff/Dockerfile`
- edit `containers/snpeff/environment.yml`
- edit `containers/vep/Dockerfile`
- edit `containers/vep/environment.yml`
- edit `docs/images/sarek_workflow.svg`
- generate a new `docs/images/sarek_workflow.png`
- edit `CHANGELOG`
- `nf-core bump-version . <VERSION>`
- edit `.circleci/config.yml`
- edit `.github/workflows/ci.yml`
- edit `conf/base.config`
- edit `conf/test.config`
- edit `containers/snpeff/Dockerfile`
- edit `containers/snpeff/environment.yml`
- edit `containers/vep/Dockerfile`
- edit `containers/vep/environment.yml`
- edit `docs/images/sarek_workflow.svg`
- generate a new `docs/images/sarek_workflow.png`
- edit `CHANGELOG`
6. Make a PR to `master`
7. Wait for reviews
8. Merge said PR
Expand Down
52 changes: 20 additions & 32 deletions .github/workflows/awsfulltest.yml
Original file line number Diff line number Diff line change
@@ -1,46 +1,34 @@
name: nf-core AWS full size tests
# This workflow is triggered on published releases.
# It can be additionally triggered manually with GitHub actions workflow dispatch.
# It can be additionally triggered manually with GitHub actions workflow dispatch button.
# It runs the -profile 'test_full' on AWS batch

on:
workflow_run:
workflows: ["nf-core Docker push (release)"]
types: [completed]
release:
types: [published]
workflow_dispatch:


env:
AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }}
AWS_SECRET_ACCESS_KEY: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
TOWER_ACCESS_TOKEN: ${{ secrets.AWS_TOWER_TOKEN }}
AWS_JOB_DEFINITION: ${{ secrets.AWS_JOB_DEFINITION }}
AWS_JOB_QUEUE: ${{ secrets.AWS_JOB_QUEUE }}
AWS_S3_BUCKET: ${{ secrets.AWS_S3_BUCKET }}


jobs:
run-awstest:
run-tower:
name: Run AWS full tests
if: github.repository == 'nf-core/sarek'
runs-on: ubuntu-latest
steps:
- name: Setup Miniconda
uses: conda-incubator/setup-miniconda@v2
with:
auto-update-conda: true
python-version: 3.7
- name: Install awscli
run: conda install -c conda-forge awscli
- name: Start AWS batch job
- name: Launch workflow via tower
uses: nf-core/tower-action@master
# TODO nf-core: You can customise AWS full pipeline tests as required
# Add full size test data (but still relatively small datasets for few samples)
# on the `test_full.config` test runs with only one set of parameters
# Then specify `-profile test_full` instead of `-profile test` on the AWS batch command
run: |
aws batch submit-job \
--region eu-west-1 \
--job-name nf-core-sarek \
--job-queue $AWS_JOB_QUEUE \
--job-definition $AWS_JOB_DEFINITION \
--container-overrides '{"command": ["nf-core/sarek", "-r '"${GITHUB_SHA}"' -profile test --outdir s3://'"${AWS_S3_BUCKET}"'/sarek/results-'"${GITHUB_SHA}"' -w s3://'"${AWS_S3_BUCKET}"'/sarek/work-'"${GITHUB_SHA}"' -with-tower"], "environment": [{"name": "TOWER_ACCESS_TOKEN", "value": "'"$TOWER_ACCESS_TOKEN"'"}]}'

with:
workspace_id: ${{ secrets.TOWER_WORKSPACE_ID }}
bearer_token: ${{ secrets.TOWER_BEARER_TOKEN }}
compute_env: ${{ secrets.TOWER_COMPUTE_ENV }}
pipeline: ${{ github.repository }}
revision: ${{ github.sha }}
workdir: s3://${{ secrets.AWS_S3_BUCKET }}/work/sarek/work-${{ github.sha }}
parameters: |
{
"outdir": "s3://${{ secrets.AWS_S3_BUCKET }}/sarek/results-${{ github.sha }}"
}
profiles: '[ "test_full", "aws_tower" ]'

50 changes: 18 additions & 32 deletions .github/workflows/awstest.yml
Original file line number Diff line number Diff line change
@@ -1,42 +1,28 @@
name: nf-core AWS test
# This workflow is triggered on push to the master branch.
# It can be additionally triggered manually with GitHub actions workflow dispatch.
# It runs the -profile 'test' on AWS batch.
# This workflow can be triggered manually with the GitHub actions workflow dispatch button.
# It runs the -profile 'test' on AWS batch

on:
workflow_dispatch:


env:
AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }}
AWS_SECRET_ACCESS_KEY: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
TOWER_ACCESS_TOKEN: ${{ secrets.AWS_TOWER_TOKEN }}
AWS_JOB_DEFINITION: ${{ secrets.AWS_JOB_DEFINITION }}
AWS_JOB_QUEUE: ${{ secrets.AWS_JOB_QUEUE }}
AWS_S3_BUCKET: ${{ secrets.AWS_S3_BUCKET }}


jobs:
run-awstest:
run-tower:
name: Run AWS tests
if: github.repository == 'nf-core/sarek'
runs-on: ubuntu-latest
steps:
- name: Setup Miniconda
uses: conda-incubator/setup-miniconda@v2
- name: Launch workflow via tower
uses: nf-core/tower-action@master

with:
auto-update-conda: true
python-version: 3.7
- name: Install awscli
run: conda install -c conda-forge awscli
- name: Start AWS batch job
# TODO nf-core: You can customise CI pipeline run tests as required
# For example: adding multiple test runs with different parameters
# Remember that you can parallelise this by using strategy.matrix
run: |
aws batch submit-job \
--region eu-west-1 \
--job-name nf-core-sarek \
--job-queue $AWS_JOB_QUEUE \
--job-definition $AWS_JOB_DEFINITION \
--container-overrides '{"command": ["nf-core/sarek", "-r '"${GITHUB_SHA}"' -profile test --outdir s3://'"${AWS_S3_BUCKET}"'/sarek/results-'"${GITHUB_SHA}"' -w s3://'"${AWS_S3_BUCKET}"'/sarek/work-'"${GITHUB_SHA}"' -with-tower"], "environment": [{"name": "TOWER_ACCESS_TOKEN", "value": "'"$TOWER_ACCESS_TOKEN"'"}]}'
workspace_id: ${{ secrets.TOWER_WORKSPACE_ID }}
bearer_token: ${{ secrets.TOWER_BEARER_TOKEN }}
compute_env: ${{ secrets.TOWER_COMPUTE_ENV }}
pipeline: ${{ github.repository }}
revision: ${{ github.sha }}
workdir: s3://${{ secrets.AWS_S3_BUCKET }}/work/sarek/work-${{ github.sha }}
parameters: |
{
"outdir": "s3://${{ secrets.AWS_S3_BUCKET }}/sarek/results-${{ github.sha }}"
}
profiles: '[ "test", "aws_tower" ]'

Loading