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

Initial release review [DO NOT MERGE] #29

Closed
wants to merge 609 commits into from
Closed
Show file tree
Hide file tree
Changes from 250 commits
Commits
Show all changes
609 commits
Select commit Hold shift + click to select a range
de1aad0
Adjust, fix bwa-mem2 index handling
scwatts Feb 29, 2024
7e0fb7f
Remove deprecated MarkDups `-multi_bam` argument
scwatts Feb 29, 2024
60a62eb
Template update for nf-core/tools version 2.13.1
nf-core-bot Feb 29, 2024
f47f33e
Fix AMBER subworkflow TN mode
scwatts Feb 29, 2024
08a58c8
Fix COBALT subworkflow TN mode
scwatts Feb 29, 2024
c53ac44
Adjust indentation
scwatts Feb 29, 2024
df1f5b7
Fix SAGE calling subworkflow TN mode
scwatts Feb 29, 2024
752f57a
Improving handling of 'no merge' RNA BAM scenarios
scwatts Feb 29, 2024
c648028
Set outputs for alignment workflows
scwatts Feb 29, 2024
db6bbf2
Add missing channel docs
scwatts Feb 29, 2024
c1f3142
Further work on RNA BAM handling
scwatts Feb 29, 2024
bd45719
Use explicit returns in .branch ops
scwatts Feb 29, 2024
8d06484
Do not index RNA BAMs prior to merge
scwatts Feb 29, 2024
361afb5
Update acknowledgements
scwatts Mar 2, 2024
124b79b
Bump PURPLE to 4.0.2
scwatts Mar 2, 2024
31a2d12
Correct comment capitalisation
scwatts Mar 2, 2024
22d3fd3
Handle backslashes in linxreport more consistently
scwatts Mar 3, 2024
8ced85b
Correct typo in PAVE somatic module
scwatts Mar 3, 2024
0881ae9
Bump version: 0.3.0 → 0.3.1
scwatts Mar 4, 2024
c7e87c2
Remove obsolete TODOs
mkcmkc Mar 5, 2024
1653daf
Fix Isofox singularity container URL
scwatts Mar 7, 2024
f618283
Bump TSO500 data bundle version
scwatts Mar 7, 2024
2bd379c
Remove -force_pathogenic_pass in PAVE somatic
scwatts Mar 7, 2024
0d151bd
Merge branch 'nf-core-template-merge-2.13.1' into dev
scwatts Mar 14, 2024
3196720
Apply prettier linting
scwatts Mar 14, 2024
37a4022
Fix indenting
scwatts Mar 14, 2024
4085b4f
Fix more linting failures
scwatts Mar 14, 2024
b4a37b3
More indenting fixes
scwatts Mar 14, 2024
05c3503
Update pipeline description
scwatts Mar 15, 2024
1b9a467
Update test config, adjust relevant arg schema
scwatts Mar 15, 2024
45ce699
Reorganise reference data and params
scwatts Mar 15, 2024
94fa17a
Fix creation of stub placeholders
scwatts Mar 15, 2024
d6a2134
Change ordering of HMF data config loading
scwatts Mar 15, 2024
2838170
Add test_stub profile
scwatts Mar 15, 2024
23f53ab
Run only stubs in CI job
scwatts Mar 15, 2024
47d551b
Merge pull request #3 from nf-core/nf-core-template-merge-2.13.1
scwatts Mar 15, 2024
f76f79b
Correct prepare reference panel data path lookup
scwatts Mar 16, 2024
07117c2
Merge branch 'dev' into alignment-subworkflow
scwatts Mar 16, 2024
9b012d0
Fix optional channel placeholders
scwatts Mar 16, 2024
b90c572
Update modules.json
scwatts Mar 16, 2024
77b8bce
Adjust indenting
scwatts Mar 16, 2024
bd6b304
Use standard container directive format for STAR
scwatts Mar 18, 2024
c7f1774
Add missing imports and subworkflow descriptions
scwatts Mar 18, 2024
177bf69
Use Bioconda/BioContainers for bwa-mem2 module
scwatts Mar 19, 2024
5c15bfa
Use fixed GRIDSS container for VIRUSBreakend
scwatts Mar 25, 2024
a2c6921
Apply linting
scwatts Mar 26, 2024
4e4a699
Use standard BioContainers base for VIRUSBreakend
scwatts Mar 26, 2024
f4703e5
Lint modules/local/virusbreakend/Dockerfile
scwatts Mar 26, 2024
9d61690
Improve naming for bwa-mem2 output BAMs
scwatts Mar 27, 2024
7adfda2
Apply @maxulysse recommendations
scwatts Apr 3, 2024
ae86ba9
Update VIRUSBreakend process container directive
scwatts Apr 3, 2024
7432c55
Do not qualify nf-core quay.io Docker URI
scwatts Apr 3, 2024
48d34aa
Merge pull request #12 from nf-core/nf-core-gridss-docker-image
scwatts Apr 3, 2024
066c8d2
Use BAM index created during alignment
scwatts Apr 15, 2024
0abc35f
Improve BAM index selection
scwatts Apr 22, 2024
eb95b4f
Include BAI in bwa-mem2/align stub
scwatts Apr 22, 2024
4039507
Adjust input selection logic
scwatts Apr 22, 2024
e8f6c99
Bump MarkDups to 1.1.5
scwatts Apr 23, 2024
ed8e1d1
Remove Sambamba index module file
scwatts Apr 23, 2024
e0da20a
Add new meta.yaml
scwatts Apr 23, 2024
101e987
Merge pull request #6 from nf-core/alignment-subworkflow
scwatts Apr 24, 2024
b982389
Adjust Hartwig genome data URLs
scwatts Apr 24, 2024
4a42093
Adjust other reference data URLs
scwatts Apr 24, 2024
d47abdc
Give GHA CI workflow step more accurate name
scwatts Apr 26, 2024
f8db2cf
Implement reference data writing
scwatts Apr 28, 2024
9c1adec
Merge pull request #14 from nf-core/update-reference-genome-urls
scwatts Apr 30, 2024
44cc52e
Merge pull request #15 from nf-core/rename-ci-job-step
scwatts Apr 30, 2024
5ab10af
Refactor reference data parameters
scwatts Apr 24, 2024
fe192f9
Fix variable names and config load order
scwatts Apr 30, 2024
a99b6ab
Correct malformed variable name
scwatts Apr 30, 2024
6c8ee3a
Merge pull request #16 from nf-core/refdata-param-refactor
scwatts Apr 30, 2024
63452bc
Update default bwa-mem2 and GRIDSS index URLs
scwatts Apr 30, 2024
906a81e
Add container and label to write refdata process
scwatts Apr 30, 2024
150d85f
Ensure ref_data_panel_data_path is defined
scwatts Apr 30, 2024
d61440c
Improve names for decompressed refdata output dirs
scwatts Apr 30, 2024
44a58b1
Add index name to decompressed refdata output dirs
scwatts Apr 30, 2024
1c2bd94
Correct mistake in assets/samplesheet.csv
scwatts Apr 30, 2024
b08399b
Ensure ref_data_virusbreakenddb_path is defined
scwatts Apr 30, 2024
78f60f6
Only write reference data require for analysis
scwatts Apr 30, 2024
776dca2
Fix panel data path default assignment
scwatts Apr 30, 2024
e9c00be
Use consistent spacing in prepare_reference.nf
scwatts Apr 30, 2024
18098c2
Set 'process_high_memory' label for bwa-mem2 index
scwatts Apr 30, 2024
fb709e5
Use bwa-mem2 for HLA read realignment
scwatts May 1, 2024
fe85940
Create bwa-mem1 index prior to GRIDSS indexing
scwatts May 1, 2024
48cd0b0
Update GRIDSS index module
scwatts May 1, 2024
b317fef
Fix GRIDSS index output
scwatts May 1, 2024
855837a
Include optional ALT file input for bwa-mem1
scwatts May 1, 2024
8ce2d6d
Do not add ALT file during GRIDSS indexing
scwatts May 1, 2024
065ec09
Merge pull request #18 from nf-core/fix-samplesheet-asset
scwatts May 1, 2024
93d6a92
Increase allocated memory for GRIDSS index process
scwatts May 2, 2024
bf26d4f
Output ALT file in GRIDSS index if present
scwatts May 2, 2024
c79cc5c
Rename STAR index output directory
scwatts May 2, 2024
7a0d9a0
Rename bwa-mem2 index output directory
scwatts May 2, 2024
f167cf6
Make PAVE genome version comparison more robust
scwatts May 2, 2024
45ddb0f
Use STAR 2.7.3a for indexing
scwatts May 2, 2024
88baaf6
Use bwa-mem2 to realign HLA reads
scwatts May 2, 2024
008666d
Clean up comments, whitespace, needed config
scwatts May 3, 2024
3adcfc6
Apply linting
scwatts May 3, 2024
fa95510
Merge pull request #19 from nf-core/improve-prepare-reference-workflow
scwatts May 3, 2024
103f2e7
Use nf-core indenting for subworkflows
scwatts May 2, 2024
766bcb2
Remove quay.io qualifier from container URLs
scwatts May 2, 2024
0335906
Remove MRD/longitudinal subworkflow placeholder
scwatts May 2, 2024
1419fb9
Improve consistency of local modules
scwatts May 2, 2024
e11a387
Rename local STAR module
scwatts May 2, 2024
d8bbe1c
Merge pull request #20 from nf-core/nfcore-style-adjustments
scwatts May 3, 2024
010c5cd
Set bwa-mem2 batch size for reproducibility
scwatts May 3, 2024
9e65de3
Template update for nf-core/tools version 2.14.0
nf-core-bot May 8, 2024
4e7eccf
Merge remote-tracking branch 'origin/dev' into nf-core-template-merge…
scwatts May 8, 2024
a354337
Fix template-introduced linting breakage
scwatts May 8, 2024
6ca7b3e
Do not apply linting fix manually
scwatts May 8, 2024
7334f62
Merge pull request #23 from nf-core/nf-core-template-merge-2.14.0
scwatts May 8, 2024
da76ec4
Template update for nf-core/tools version 2.14.1
nf-core-bot May 9, 2024
a6e0f85
Merge remote-tracking branch 'nf-core/dev' into nf-core-template-merg…
scwatts May 9, 2024
efe5185
Merge pull request #24 from nf-core/nf-core-template-merge-2.14.1
scwatts May 10, 2024
c02df3e
Bump SAGE to 3.4.3
scwatts May 10, 2024
7b41c49
Apply formatting fixes and other minor fixes
scwatts May 6, 2024
bb86c80
Merge pull request #26 from nf-core/bump-sage-version
scwatts May 10, 2024
18330f1
Merge pull request #25 from nf-core/further-formatting-improvements
scwatts May 10, 2024
558e728
First full pass of nf-core documentation
scwatts Mar 14, 2024
f0caef5
Fix Cloudflare R2 base URL
scwatts Mar 15, 2024
c610c0c
Apply prettier to docs
scwatts Mar 15, 2024
7955627
Link typo in docs/output.md
scwatts Mar 15, 2024
04ba068
Improve docs/usage.md
scwatts Mar 15, 2024
92d5285
Correct and clarify development/stable branches
scwatts Mar 19, 2024
afacc48
Update release parity versions
scwatts Apr 30, 2024
2a8b218
Update prepare reference data docs
scwatts May 3, 2024
7ece657
Various documentation edits
scwatts May 10, 2024
6726781
Update pipeline tests
scwatts May 10, 2024
7eedd64
Remove schema ignore param from test profiles
scwatts May 10, 2024
d7d8922
Adjust pipeline test config, comments
scwatts May 11, 2024
77f8074
Merge branch 'master' into fix-dev-root-commit
scwatts May 12, 2024
311c1bf
Add tool entries to CITATIONS.md
scwatts May 13, 2024
50fe58e
Merge pull request #28 from nf-core/fix-dev-root-commit
scwatts May 13, 2024
1945249
Merge pull request #27 from nf-core/update-pipeline-tests
scwatts May 14, 2024
7ed732c
Merge pull request #31 from nf-core/update-citations
scwatts May 15, 2024
1344a9e
Remove unused ref params from test_stub profile
scwatts May 15, 2024
24cc1ad
Use test_stub profile in download_pipline GHA wfl
scwatts May 15, 2024
a6b8d15
Remove non-stub run from download_pipeline CI
scwatts May 15, 2024
c12a0cc
Merge pull request #32 from nf-core/fix-download-pipeline-ci
scwatts May 15, 2024
322d5cb
Merge pull request #33 from nf-core/update-test-stub-genome-parameters
scwatts May 15, 2024
3c706c0
Remove duplicate tests
scwatts May 16, 2024
30b0e72
Do not split FASTQs during tests
scwatts May 16, 2024
b9b726f
Update docs/usage.md
scwatts May 16, 2024
6fb3d8d
Update docs/usage.md
scwatts May 16, 2024
5e4ccdb
Further clarify MarkDups function, requirements
scwatts May 17, 2024
051952e
Merge pull request #9 from nf-core/nf-core-documentation
scwatts May 17, 2024
f1fe44d
Rename bwa-mem2 index variables
scwatts May 16, 2024
5ea55f5
Use correct bwa-mem2 pointers in usage.md
scwatts May 17, 2024
112d239
Correct typo usage.md
scwatts May 17, 2024
a88b036
Merge pull request #34 from nf-core/improve-bwa-mem2-index-name
scwatts May 17, 2024
140094f
Merge pull request #35 from nf-core/remove-nf-test-tests
scwatts May 17, 2024
15ecbf4
Merge pull request #36 from nf-core/avoid-fastq-splitting-in-tests
scwatts May 17, 2024
521e2df
Change admonition style to fix rendering
scwatts May 18, 2024
2a0f2bf
Add known issue section to README.md
scwatts May 18, 2024
c46b9bb
Apply Prettier linting to usage.md
scwatts May 18, 2024
52610fe
Improve wording for known issue entry in README.md
scwatts May 18, 2024
7be3dda
Merge pull request #39 from nf-core/fix-usage-docs-rendering
scwatts May 20, 2024
67f8e65
Merge pull request #38 from nf-core/add-known-issues
scwatts May 20, 2024
b411125
Fix display of ORANGE report screencap on website
scwatts May 20, 2024
8e65844
Merge pull request #40 from nf-core/fix-website-image-rendering
scwatts May 20, 2024
6dc8459
Make README.md adminitions compatible with website
scwatts May 20, 2024
d5d64c4
Merge pull request #41 from nf-core/adjust-readme-admonition
scwatts May 20, 2024
437e72e
Only create bwa-mem2 index if FASTQ inputs present
scwatts May 21, 2024
a5b9846
Merge pull request #42 from nf-core/fix-conditional-bwamem2-index-cre…
scwatts May 21, 2024
d1bc96a
Use all available threads for fastp process
scwatts May 22, 2024
b0cf4ce
Merge pull request #43 from nf-core/set-fastp-threads
scwatts May 22, 2024
69f4d3c
Adjust test_stub.config usage comment
scwatts May 27, 2024
56f9058
Update .nf-core.yml lint rule skip list
scwatts May 27, 2024
138d900
Correct typo in docs/output.md
scwatts May 27, 2024
d925a44
Replace System.exit with Nextflow.exit
scwatts May 27, 2024
644927c
Further standardise variable naming
scwatts May 27, 2024
448a988
Make output tarball channel name more descriptive
scwatts May 27, 2024
84afa4d
Bump MarkDups 1.1.5 to Bioconda build hdfd78af_1
scwatts May 27, 2024
8295f1e
Bump SvPrep to 1.2.4
scwatts May 27, 2024
610ea8a
Bump bwa-mem2 container build and update Conda env
scwatts May 27, 2024
ab9518b
Bump Sambamba to 1.0.1
scwatts May 27, 2024
3a6ebfc
Bump version to 1.0.0 for initial release
scwatts May 27, 2024
be33799
Fix usage.md MarkDups link
scwatts May 29, 2024
c843c3b
Merge pull request #49 from nf-core/initial-release-review-changes
scwatts May 29, 2024
2f33d9e
Merge pull request #47 from nf-core/bump-svprep-version
scwatts May 29, 2024
31be68d
Merge pull request #46 from nf-core/bump-sambamba-version
scwatts May 29, 2024
ba366c9
Merge pull request #45 from nf-core/bump-markdups-package-build
scwatts May 29, 2024
c477a07
Merge pull request #48 from nf-core/bump-bwa_mem2-container-build
scwatts May 29, 2024
ce2bf00
Merge pull request #21 from nf-core/bwamem2-input-batch-size
scwatts May 31, 2024
177a26f
Bump MarkDups to 1.1.7
scwatts May 31, 2024
8d19bff
Update README.md known issues section
scwatts May 31, 2024
9abc114
Merge pull request #52 from nf-core/bump-markdups-version
scwatts May 31, 2024
85765b1
Quote absolute paths used in directory shadowing
scwatts Jun 3, 2024
67275f5
Merge pull request #53 from nf-core/fix-directory-shadowing-symlinks
scwatts Jun 4, 2024
fc04fef
symlink specific gridss index files
casslitch Jun 4, 2024
fb49fd0
Explicitly define all gridss index files to be symlinked
casslitch Jun 5, 2024
296f5d8
Fixed typo in find command
casslitch Jun 5, 2024
d0a920f
Fixed regex string in find command
casslitch Jun 5, 2024
e85c081
Add BAI placeholder in read processing out channel
scwatts Jun 6, 2024
c9fa9de
Merge pull request #56 from nf-core/fix-read-processing-optional-channel
scwatts Jun 6, 2024
b5c1daf
Merge pull request #54 from nf-core/casslitch-patch-1
casslitch Jun 6, 2024
9c81e3e
Bump AMBER to 4.0.1
scwatts Jun 7, 2024
6c0fbb6
Merge pull request #57 from nf-core/bump-amber-version
scwatts Jun 7, 2024
15a9810
Bump SAGE to 3.4.4
scwatts Jun 12, 2024
66ce9be
Merge pull request #58 from nf-core/bump-sage-version
scwatts Jun 12, 2024
3a701ec
Correct MarkDups version in Conda env and docs URL
scwatts Jun 13, 2024
b235996
Merge pull request #59 from nf-core/fix-markdups-version
scwatts Jun 13, 2024
c0b8155
Add type for isofox_read_length in schema
scwatts Jun 19, 2024
26abf3f
Merge pull request #61 from nf-core/fix-nextflow-schema
scwatts Jun 19, 2024
ecbf540
Include 1.0.0 version in release parity table
scwatts Jul 5, 2024
d36f8b0
Remove TODO in methods_description_template.yml
scwatts Jul 5, 2024
dcf919e
Adjust filepaths in assets/samplesheet.csv
scwatts Jul 5, 2024
dd4f868
Simplify target parameter definitions
scwatts Jul 5, 2024
882cb6c
Fix templated version in usage.md
scwatts Jul 5, 2024
204d75b
Fix grammar and clarify usage.md
scwatts Jul 5, 2024
02431ca
Add backticks to in text usage of oncoanalyser
scwatts Jul 5, 2024
bbf66f6
Update .bumpversion.cfg
scwatts Jul 5, 2024
1f30f98
Merge pull request #62 from nf-core/initial-release-review-changes
scwatts Jul 5, 2024
8479d69
Do not use hardcoded absolute path for ref genomes
scwatts Jul 10, 2024
1fc84ed
Remove obsolete NOTE
scwatts Jul 10, 2024
3a15fb9
Use more consistent SAGE calling subworkflow args
scwatts Jul 10, 2024
2e37b30
Use more readable format in targeted params config
scwatts Jul 10, 2024
2f86f87
Merge pull request #64 from nf-core/initial-release-review-changes
scwatts Jul 11, 2024
404efef
Move GRIDSS config input to last process input pos
scwatts Jul 17, 2024
584b3ef
Correct typo
scwatts Jul 17, 2024
e323400
Indicate NOTE author
scwatts Jul 17, 2024
4d9b223
Provide more informative error for bad run mode
scwatts Jul 17, 2024
989fbb2
Remove obsolete TODOs
scwatts Jul 17, 2024
feca359
Apply typo correction
scwatts Jul 17, 2024
b9ed09e
Merge pull request #67 from nf-core/initial-release-review-changes
scwatts Jul 17, 2024
0d8faa2
Demote double underscores to single underscores
scwatts Jul 18, 2024
4e9a212
Use better halt/error in PAVE process
scwatts Jul 18, 2024
879d7d5
Update params YAML example in usage.md
scwatts Jul 18, 2024
828c8ee
Update indicated location of software versions
scwatts Jul 18, 2024
7545745
Minor improvements to nextflow_schema.json
scwatts Jul 18, 2024
b438878
Merge pull request #68 from nf-core/initial-release-review-changes
scwatts Jul 18, 2024
a779453
Fix grammar in usage.md
scwatts Jul 18, 2024
b4523d1
Add note on sequencing depth to usage.md
scwatts Jul 18, 2024
1b087b6
Merge pull request #69 from nf-core/update-usage
scwatts Jul 18, 2024
c1b1fce
Improve fenced codeblock syntax highlight
scwatts Jul 19, 2024
2b4e521
Use correct AMBER version in conda env file
scwatts Jul 19, 2024
58b75d4
Pin Tar version in extracttarball conda env
scwatts Jul 19, 2024
8f8ffa2
Merge pull request #70 from nf-core/initial-release-review-changes
scwatts Jul 24, 2024
f142fa6
Add summary note on pipeline function in README.md
scwatts Jul 31, 2024
c0d6b07
Correct Conda GRIDSS build version
scwatts Jul 31, 2024
7394aa2
Correct Conda SAGE version
scwatts Jul 31, 2024
b660853
Comment out any hint of citations
scwatts Jul 31, 2024
a6b5cc2
Use the HMF abbreviation more in README.md
scwatts Jul 31, 2024
76cb780
Update input samplesheet URL for tests
scwatts Jul 31, 2024
3a835c6
Fix MarkDups URLs
scwatts Jul 31, 2024
3f7d818
Use stub samplesheet for test_stub profile
scwatts Jul 31, 2024
f876e18
Switch to BAM inputs for test_stub samplesheet
scwatts Jul 31, 2024
41010dd
Merge pull request #72 from nf-core/initial-release-review-changes
scwatts Aug 1, 2024
7ec0f39
Allow RNA BAM from samplesheet as input to LILAC
scwatts Aug 6, 2024
49872fc
Merge pull request #76 from nf-core/lilac-samplesheet-bam-input
scwatts Aug 6, 2024
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
25 changes: 25 additions & 0 deletions .bumpversion.cfg
mashehu marked this conversation as resolved.
Show resolved Hide resolved
scwatts marked this conversation as resolved.
Show resolved Hide resolved
scwatts marked this conversation as resolved.
Show resolved Hide resolved
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
[bumpversion]
current_version = 1.0.0
commit = True
tag = False
parse = (?P<major>\d+)\.(?P<minor>\d+)\.(?P<patch>[a-z0-9+]+)

[bumpversion:file:nextflow.config]
search = version = '{current_version}'
replace = version = '{new_version}'

[bumpversion:file:README.md]
search = -revision {current_version}
replace = -revision {new_version}

[bumpversion:file (example commands):docs/usage.md]
search = -revision {current_version}
replace = -revision {new_version}

[bumpversion:file (urls):docs/usage.md]
search = /{current_version}/
replace = /{new_version}/

[bumpversion:file (templated example):docs/usage.md]
search = {current_version}`
replace = {new_version}`
20 changes: 20 additions & 0 deletions .devcontainer/devcontainer.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
{
"name": "nfcore",
"image": "nfcore/gitpod:latest",
"remoteUser": "gitpod",
"runArgs": ["--privileged"],

// Configure tool-specific properties.
"customizations": {
// Configure properties specific to VS Code.
"vscode": {
// Set *default* container specific settings.json values on container create.
"settings": {
"python.defaultInterpreterPath": "/opt/conda/bin/python"
},

// Add the IDs of extensions you want installed when the container is created.
"extensions": ["ms-python.python", "ms-python.vscode-pylance", "nf-core.nf-core-extensionpack"]
}
}
}
13 changes: 11 additions & 2 deletions .editorconfig
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ trim_trailing_whitespace = true
indent_size = 4
indent_style = space

[*.{md,yml,yaml,html,css,scss,js,cff}]
[*.{md,yml,yaml,html,css,scss,js}]
indent_size = 2

# These files are edited and tested upstream in nf-core/modules
Expand All @@ -18,7 +18,16 @@ end_of_line = unset
insert_final_newline = unset
trim_trailing_whitespace = unset
indent_style = unset
indent_size = unset
[/subworkflows/nf-core/**]
charset = unset
end_of_line = unset
insert_final_newline = unset
trim_trailing_whitespace = unset
indent_style = unset

[/assets/email*]
indent_size = unset

# ignore python and markdown
[*.{py,md}]
indent_style = unset
1 change: 1 addition & 0 deletions .gitattributes
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
*.config linguist-language=nextflow
*.nf.test linguist-language=nextflow
modules/nf-core/** linguist-generated
subworkflows/nf-core/** linguist-generated
24 changes: 23 additions & 1 deletion .github/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ Please use the pre-filled template to save time.
However, don't be put off by this template - other more general issues and suggestions are welcome!
Contributions to the code are even more welcome ;)

> [!NOTE]
> If you need help using or modifying nf-core/oncoanalyser then the best place to ask is on the nf-core Slack [#oncoanalyser](https://nfcore.slack.com/channels/oncoanalyser) channel ([join our Slack here](https://nf-co.re/join/slack)).

## Contribution workflow
Expand All @@ -25,6 +26,12 @@ If you're not used to this workflow with git, you can start with some [docs from

## Tests

You have the option to test your changes locally by running the pipeline. For receiving warnings about process selectors and other `debug` information, it is recommended to use the debug profile. Execute all the tests with the following command:

```bash
nf-test test --profile debug,test,docker --verbose
```

When you create a pull request with changes, [GitHub Actions](https://github.com/features/actions) will run automatic tests.
Typically, pull-requests are only fully reviewed when these tests are passing, though of course we can help out before then.

Expand Down Expand Up @@ -85,7 +92,7 @@ Once there, use `nf-core schema build` to add to `nextflow_schema.json`.

Sensible defaults for process resource requirements (CPUs / memory / time) for a process should be defined in `conf/base.config`. These should generally be specified generic with `withLabel:` selectors so they can be shared across multiple processes/steps of the pipeline. A nf-core standard set of labels that should be followed where possible can be seen in the [nf-core pipeline template](https://github.com/nf-core/tools/blob/master/nf_core/pipeline-template/conf/base.config), which has the default process as a single core-process, and then different levels of multi-core configurations for increasingly large memory requirements defined with standardised labels.

The process resources can be passed on to the tool dynamically within the process with the `${task.cpu}` and `${task.memory}` variables in the `script:` block.
The process resources can be passed on to the tool dynamically within the process with the `${task.cpus}` and `${task.memory}` variables in the `script:` block.

### Naming schemes

Expand All @@ -101,3 +108,18 @@ If you are using a new feature from core Nextflow, you may bump the minimum requ
### Images and figures

For overview images and other documents we follow the nf-core [style guidelines and examples](https://nf-co.re/developers/design_guidelines).

## GitHub Codespaces

This repo includes a devcontainer configuration which will create a GitHub Codespaces for Nextflow development! This is an online developer environment that runs in your browser, complete with VSCode and a terminal.

To get started:

- Open the repo in [Codespaces](https://github.com/nf-core/oncoanalyser/codespaces)
- Tools installed
- nf-core
- Nextflow

Devcontainer specs:

- [DevContainer config](.devcontainer/devcontainer.json)
4 changes: 2 additions & 2 deletions .github/ISSUE_TEMPLATE/bug_report.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,9 +42,9 @@ body:
attributes:
label: System information
description: |
* Nextflow version _(eg. 21.10.3)_
* Nextflow version _(eg. 23.04.0)_
* Hardware _(eg. HPC, Desktop, Cloud)_
* Executor _(eg. slurm, local, awsbatch)_
* Container engine: _(e.g. Docker, Singularity, Conda, Podman, Shifter or Charliecloud)_
* Container engine: _(e.g. Docker, Singularity, Conda, Podman, Shifter, Charliecloud, or Apptainer)_
* OS _(eg. CentOS Linux, macOS, Linux Mint)_
* Version of nf-core/oncoanalyser _(eg. 1.1, 1.5, 1.8.2)_
4 changes: 3 additions & 1 deletion .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,11 @@ Learn more about contributing: [CONTRIBUTING.md](https://github.com/nf-core/onco

- [ ] 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 - have you followed the pipeline conventions in the [contribution docs](https://github.com/nf-core/oncoanalyser/tree/master/.github/CONTRIBUTING.md)- [ ] If necessary, also make a PR on the nf-core/oncoanalyser _branch_ on the [nf-core/test-datasets](https://github.com/nf-core/test-datasets) repository.
- [ ] If you've added a new tool - have you followed the pipeline conventions in the [contribution docs](https://github.com/nf-core/oncoanalyser/tree/master/.github/CONTRIBUTING.md)
- [ ] If necessary, also make a PR on the nf-core/oncoanalyser _branch_ on the [nf-core/test-datasets](https://github.com/nf-core/test-datasets) repository.
- [ ] Make sure your code lints (`nf-core lint`).
- [ ] Ensure the test suite passes (`nextflow run . -profile test,docker --outdir <OUTDIR>`).
- [ ] Check for unexpected warnings in debug mode (`nextflow run . -profile debug,test,docker --outdir <OUTDIR>`).
- [ ] Usage Documentation in `docs/usage.md` is updated.
- [ ] Output Documentation in `docs/output.md` is updated.
- [ ] `CHANGELOG.md` is updated.
Expand Down
19 changes: 12 additions & 7 deletions .github/workflows/awsfulltest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,27 +8,32 @@ on:
types: [published]
workflow_dispatch:
jobs:
run-tower:
run-platform:
name: Run AWS full tests
if: github.repository == 'nf-core/oncoanalyser'
runs-on: ubuntu-latest
steps:
- name: Launch workflow via tower
uses: nf-core/tower-action@v3
- name: Launch workflow via Seqera Platform
uses: seqeralabs/action-tower-launch@v2
# 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
with:
workspace_id: ${{ secrets.TOWER_WORKSPACE_ID }}
access_token: ${{ secrets.TOWER_ACCESS_TOKEN }}
compute_env: ${{ secrets.TOWER_COMPUTE_ENV }}
revision: ${{ github.sha }}
workdir: s3://${{ secrets.AWS_S3_BUCKET }}/work/oncoanalyser/work-${{ github.sha }}
parameters: |
{
"hook_url": "${{ secrets.MEGATESTS_ALERTS_SLACK_HOOK_URL }}",
"outdir": "s3://${{ secrets.AWS_S3_BUCKET }}/oncoanalyser/results-${{ github.sha }}"
}
profiles: test_full,aws_tower
- uses: actions/upload-artifact@v3
profiles: test_full

- uses: actions/upload-artifact@v4
with:
name: Tower debug log file
path: tower_action_*.log
name: Seqera Platform debug log file
path: |
seqera_platform_action_*.log
seqera_platform_action_*.json
20 changes: 12 additions & 8 deletions .github/workflows/awstest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,25 +5,29 @@ name: nf-core AWS test
on:
workflow_dispatch:
jobs:
run-tower:
run-platform:
name: Run AWS tests
if: github.repository == 'nf-core/oncoanalyser'
runs-on: ubuntu-latest
steps:
# Launch workflow using Tower CLI tool action
- name: Launch workflow via tower
uses: nf-core/tower-action@v3
# Launch workflow using Seqera Platform CLI tool action
- name: Launch workflow via Seqera Platform
uses: seqeralabs/action-tower-launch@v2
with:
workspace_id: ${{ secrets.TOWER_WORKSPACE_ID }}
access_token: ${{ secrets.TOWER_ACCESS_TOKEN }}
compute_env: ${{ secrets.TOWER_COMPUTE_ENV }}
revision: ${{ github.sha }}
workdir: s3://${{ secrets.AWS_S3_BUCKET }}/work/oncoanalyser/work-${{ github.sha }}
parameters: |
{
"outdir": "s3://${{ secrets.AWS_S3_BUCKET }}/oncoanalyser/results-test-${{ github.sha }}"
}
profiles: test,aws_tower
- uses: actions/upload-artifact@v3
profiles: test

- uses: actions/upload-artifact@v4
with:
name: Tower debug log file
path: tower_action_*.log
name: Seqera Platform debug log file
path: |
seqera_platform_action_*.log
seqera_platform_action_*.json
4 changes: 2 additions & 2 deletions .github/workflows/branch.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,13 +13,13 @@ jobs:
- name: Check PRs
if: github.repository == 'nf-core/oncoanalyser'
run: |
{ [[ ${{github.event.pull_request.head.repo.full_name }} == nf-core/oncoanalyser ]] && [[ $GITHUB_HEAD_REF = "dev" ]]; } || [[ $GITHUB_HEAD_REF == "patch" ]]
{ [[ ${{github.event.pull_request.head.repo.full_name }} == nf-core/oncoanalyser ]] && [[ $GITHUB_HEAD_REF == "dev" ]]; } || [[ $GITHUB_HEAD_REF == "patch" ]]

# If the above check failed, post a comment on the PR explaining the failure
# NOTE - this doesn't currently work if the PR is coming from a fork, due to limitations in GitHub actions secrets
- name: Post PR comment
if: failure()
uses: mshick/add-pr-comment@v1
uses: mshick/add-pr-comment@b8f338c590a895d50bcbfa6c5859251edc8952fc # v2
with:
message: |
## This PR is against the `master` branch :x:
Expand Down
22 changes: 13 additions & 9 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,29 +11,33 @@ on:
env:
NXF_ANSI_LOG: false

concurrency:
group: "${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}"
cancel-in-progress: true

jobs:
test:
name: Run pipeline with test data
name: Run pipeline stubs
# Only run on push if this is the nf-core dev branch (merged PRs)
if: "${{ github.event_name != 'push' || (github.event_name == 'push' && github.repository == 'nf-core/oncoanalyser') }}"
runs-on: ubuntu-latest
strategy:
matrix:
NXF_VER:
- "21.10.3"
- "23.04.0"
- "latest-everything"
steps:
- name: Check out pipeline code
uses: actions/checkout@v2
uses: actions/checkout@0ad4b8fadaa221de15dcec353f45205ec38ea70b # v4

- name: Install Nextflow
uses: nf-core/setup-nextflow@v1
uses: nf-core/setup-nextflow@v2
with:
version: "${{ matrix.NXF_VER }}"

- name: Run pipeline with test data
# 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
- name: Disk space cleanup
uses: jlumbroso/free-disk-space@54081f138730dfa15788a46383842cd2f914a1be # v1.3.1

- name: Run pipeline stubs only
run: |
nextflow run ${GITHUB_WORKSPACE} -profile test,docker --outdir ./results
nextflow run ${GITHUB_WORKSPACE} -stub -profile test_stub --outdir ./results
24 changes: 24 additions & 0 deletions .github/workflows/clean-up.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
name: "Close user-tagged issues and PRs"
on:
schedule:
- cron: "0 0 * * 0" # Once a week

jobs:
clean-up:
runs-on: ubuntu-latest
permissions:
issues: write
pull-requests: write
steps:
- uses: actions/stale@28ca1036281a5e5922ead5184a1bbf96e5fc984e # v9
with:
stale-issue-message: "This issue has been tagged as awaiting-changes or awaiting-feedback by an nf-core contributor. Remove stale label or add a comment otherwise this issue will be closed in 20 days."
stale-pr-message: "This PR has been tagged as awaiting-changes or awaiting-feedback by an nf-core contributor. Remove stale label or add a comment if it is still useful."
close-issue-message: "This issue was closed because it has been tagged as awaiting-changes or awaiting-feedback by an nf-core contributor and then staled for 20 days with no activity."
days-before-stale: 30
days-before-close: 20
days-before-pr-close: -1
any-of-labels: "awaiting-changes,awaiting-feedback"
exempt-issue-labels: "WIP"
exempt-pr-labels: "WIP"
repo-token: "${{ secrets.GITHUB_TOKEN }}"
79 changes: 79 additions & 0 deletions .github/workflows/download_pipeline.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,79 @@
name: Test successful pipeline download with 'nf-core download'

# Run the workflow when:
# - dispatched manually
# - when a PR is opened or reopened to master branch
# - the head branch of the pull request is updated, i.e. if fixes for a release are pushed last minute to dev.
on:
workflow_dispatch:
inputs:
testbranch:
description: "The specific branch you wish to utilize for the test execution of nf-core download."
required: true
default: "dev"
pull_request:
types:
- opened
- edited
- synchronize
branches:
- master
pull_request_target:
branches:
- master

env:
NXF_ANSI_LOG: false

jobs:
download:
runs-on: ubuntu-latest
steps:
- name: Install Nextflow
uses: nf-core/setup-nextflow@v2

- name: Disk space cleanup
uses: jlumbroso/free-disk-space@54081f138730dfa15788a46383842cd2f914a1be # v1.3.1

- uses: actions/setup-python@82c7e631bb3cdc910f68e0081d67478d79c6982d # v5
with:
python-version: "3.12"
architecture: "x64"
- uses: eWaterCycle/setup-singularity@931d4e31109e875b13309ae1d07c70ca8fbc8537 # v7
with:
singularity-version: 3.8.3

- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install git+https://github.com/nf-core/tools.git@dev

- name: Get the repository name and current branch set as environment variable
run: |
echo "REPO_LOWERCASE=${GITHUB_REPOSITORY,,}" >> ${GITHUB_ENV}
echo "REPOTITLE_LOWERCASE=$(basename ${GITHUB_REPOSITORY,,})" >> ${GITHUB_ENV}
echo "REPO_BRANCH=${{ github.event.inputs.testbranch || 'dev' }}" >> ${GITHUB_ENV}

- name: Download the pipeline
env:
NXF_SINGULARITY_CACHEDIR: ./
run: |
nf-core download ${{ env.REPO_LOWERCASE }} \
--revision ${{ env.REPO_BRANCH }} \
--outdir ./${{ env.REPOTITLE_LOWERCASE }} \
--compress "none" \
--container-system 'singularity' \
--container-library "quay.io" -l "docker.io" -l "ghcr.io" \
--container-cache-utilisation 'amend' \
--download-configuration

- name: Inspect download
run: tree ./${{ env.REPOTITLE_LOWERCASE }}

- name: Run the downloaded pipeline (stub)
id: stub_run_pipeline
continue-on-error: true
env:
NXF_SINGULARITY_CACHEDIR: ./
NXF_SINGULARITY_HOME_MOUNT: true
run: nextflow run ./${{ env.REPOTITLE_LOWERCASE }}/$( sed 's/\W/_/g' <<< ${{ env.REPO_BRANCH }}) -stub -profile test_stub,singularity --outdir ./results
Loading
Loading