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

nf-core v2.14 update #99

Merged
merged 34 commits into from
Aug 1, 2024
Merged
Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
539a2fd
nf-core modules update and necessary integrations
tkchafin Jul 17, 2024
089f2b7
samtools:1.17 -> 1.20 in local modules
tkchafin Jul 17, 2024
eea3db1
installing new nf-core subworkflows
tkchafin Jul 17, 2024
6cfc646
schema updates
tkchafin Jul 17, 2024
bdfb8b9
nf_validation conversion
tkchafin Jul 18, 2024
28c683c
added tool citations; removed old samplesheet_check
tkchafin Jul 19, 2024
004e031
corrected citations
tkchafin Jul 19, 2024
33527b8
add compliant template name
tkchafin Jul 19, 2024
642ebab
Template update for nf-core/tools version 2.14.1
tkchafin Jul 19, 2024
260ad39
template update 2.14
tkchafin Jul 19, 2024
34731f8
revert nf-core sync changes
tkchafin Jul 19, 2024
b57595f
Restore .github directory from dev branch
tkchafin Jul 22, 2024
94a10f0
schema update
tkchafin Jul 22, 2024
26b3818
update .gitignore
tkchafin Jul 22, 2024
ed898a6
clean up
tkchafin Jul 22, 2024
20d2a51
clean up
tkchafin Jul 22, 2024
af0d8a3
nf-core linting
tkchafin Jul 22, 2024
14dc591
revert some template changes
tkchafin Jul 22, 2024
6404674
revert some template changes
tkchafin Jul 22, 2024
5224fdd
revert some template changes
tkchafin Jul 22, 2024
802aac2
cleanup
tkchafin Jul 22, 2024
8791fec
python black linting
tkchafin Jul 22, 2024
c769326
nf-core linting
tkchafin Jul 22, 2024
6660c19
nf-core linting
tkchafin Jul 22, 2024
fa3ae15
nf-core linting
tkchafin Jul 22, 2024
6d24111
nf-core linting
tkchafin Jul 22, 2024
81c32e0
Merge branch 'dev' into nf_core_update
tkchafin Jul 23, 2024
161f3f1
nf-core linting
tkchafin Jul 23, 2024
d36ab17
Merge branch 'nf_core_update' of https://github.com/tkchafin/readmapp…
tkchafin Jul 23, 2024
26589a6
nf-core lint
tkchafin Jul 28, 2024
77b8852
prettier lint
tkchafin Jul 28, 2024
3ffb042
revert projectDir change in nextflow_schema.json
tkchafin Jul 31, 2024
9bc9009
outdir for test profile
tkchafin Aug 1, 2024
816bd46
outdir for test profile
tkchafin Aug 1, 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
1 change: 1 addition & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ Learn more about contributing: [CONTRIBUTING.md](https://github.com/sanger-tol/r
- [ ] If you've added a new tool - have you followed the pipeline conventions in the [contribution docs](https://github.com/sanger-tol/readmapping/tree/master/.github/CONTRIBUTING.md)
- [ ] 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
68 changes: 14 additions & 54 deletions .github/workflows/linting.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,74 +11,34 @@ on:
types: [published]

jobs:
EditorConfig:
pre-commit:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@0ad4b8fadaa221de15dcec353f45205ec38ea70b # v4

- uses: actions/setup-node@v3

- name: Install editorconfig-checker
run: npm install -g editorconfig-checker

- name: Run ECLint check
run: editorconfig-checker -exclude README.md $(find .* -type f | grep -v '.git\|.py\|.md\|json\|yml\|yaml\|html\|css\|work\|.nextflow\|build\|nf_core.egg-info\|log.txt\|Makefile')

Prettier:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3

- uses: actions/setup-node@v3

- name: Install Prettier
run: npm install -g prettier

- name: Run Prettier --check
run: prettier --check ${GITHUB_WORKSPACE}

PythonBlack:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3

- name: Check code lints with Black
uses: psf/black@stable

# If the above check failed, post a comment on the PR explaining the failure
- name: Post PR comment
if: failure()
uses: mshick/add-pr-comment@v1
- name: Set up Python 3.12
uses: actions/setup-python@82c7e631bb3cdc910f68e0081d67478d79c6982d # v5
with:
message: |
## Python linting (`black`) is failing

To keep the code consistent with lots of contributors, we run automated code consistency checks.
To fix this CI test, please run:

* Install [`black`](https://black.readthedocs.io/en/stable/): `pip install black`
* Fix formatting errors in your pipeline: `black .`

Once you push these changes the test should pass, and you can hide this comment :+1:
python-version: "3.12"

We highly recommend setting up Black in your code editor so that this formatting is done automatically on save. Ask about it on Slack for help!
- name: Install pre-commit
run: pip install pre-commit

Thanks again for your contribution!
repo-token: ${{ secrets.GITHUB_TOKEN }}
allow-repeats: false
- name: Run pre-commit
run: pre-commit run --all-files

nf-core:
runs-on: ubuntu-latest
steps:
- name: Check out pipeline code
uses: actions/checkout@v3
uses: actions/checkout@0ad4b8fadaa221de15dcec353f45205ec38ea70b # v4

- name: Install Nextflow
uses: nf-core/setup-nextflow@v1
uses: nf-core/setup-nextflow@v2

- uses: actions/setup-python@v4
- uses: actions/setup-python@82c7e631bb3cdc910f68e0081d67478d79c6982d # v5
with:
python-version: "3.7"
python-version: "3.12"
architecture: "x64"

- name: Install dependencies
Expand All @@ -99,7 +59,7 @@ jobs:

- name: Upload linting log file artifact
if: ${{ always() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@65462800fd760344b1a7b4382951275a0abb4808 # v4
with:
name: linting-logs
path: |
Expand Down
14 changes: 8 additions & 6 deletions assets/schema_input.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,18 +14,20 @@
},
"datatype": {
"type": "string",
"pattern": "^\\S+$",
"errorMessage": "Data type, and must be one of: 'hic' or 'illumina' or 'ont' or 'pacbio'"
"enum": ["hic", "illumina", "pacbio", "pacbio_clr", "ont"],
"errorMessage": "Datatype must be one of: hic, illumina, pacbio, pacbio_clr, ont"
},
"datafile": {
"type": "string",
"format": "file-path",
"exists": true,
"pattern": "^\\S+$",
"errorMessage": "Data file for reads cannot contain spaces and must have extension 'cram' or 'bam' or '.fq.gz' or '.fastq.gz'"
"errorMessage": "Data file for reads cannot contain spaces and must have extension 'cram', 'bam', '.fq.gz' or '.fastq.gz'"
},
"library": {
"type": "string",
"pattern": "^\\S+$",
"errorMessage": "Library name to be assigned to read group (@RG) ID value"
"pattern": "^\\S*$",
"errorMessage": "Library information cannot contain spaces",
"default": ""
}
},
"required": ["sample", "datatype", "datafile"]
Expand Down
49 changes: 34 additions & 15 deletions modules.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,81 +7,100 @@
"nf-core": {
"blast/blastn": {
"branch": "master",
"git_sha": "911696ea0b62df80e900ef244d7867d177971f73",
"git_sha": "4262a04142431275e54e1f4b413628a2201ed6e6",
"installed_by": ["modules"]
},
"bwamem2/index": {
"branch": "master",
"git_sha": "911696ea0b62df80e900ef244d7867d177971f73",
"git_sha": "7081e04c18de9480948d34513a1c1e2d0fa9126d",
"installed_by": ["modules"]
},
"bwamem2/mem": {
"branch": "master",
"git_sha": "0460d316170f75f323111b4a2c0a2989f0c32013",
"git_sha": "3afb95b2e15fc4a2347470255a7ef654f650c8ec",
"installed_by": ["modules"]
},
"crumble": {
"branch": "master",
"git_sha": "911696ea0b62df80e900ef244d7867d177971f73",
"git_sha": "3f5420aa22e00bd030a2556dfdffc9e164ec0ec5",
"installed_by": ["modules"]
},
"custom/dumpsoftwareversions": {
"branch": "master",
"git_sha": "911696ea0b62df80e900ef244d7867d177971f73",
"git_sha": "82024cf6325d2ee194e7f056d841ecad2f6856e9",
"installed_by": ["modules"]
},
"gunzip": {
"branch": "master",
"git_sha": "5c460c5a4736974abde2843294f35307ee2b0e5e",
"git_sha": "4e5f4687318f24ba944a13609d3ea6ebd890737d",
"installed_by": ["modules"]
},
"minimap2/align": {
"branch": "master",
"git_sha": "efbf86bb487f288ac30660282709d9620dd6048e",
"git_sha": "e83b347b3e674de6bb1bb7bdf9b2674768c8e0fe",
"installed_by": ["modules"]
},
"samtools/faidx": {
"branch": "master",
"git_sha": "fd742419940e01ba1c5ecb172c3e32ec840662fe",
"git_sha": "04fbbc7c43cebc0b95d5b126f6d9fe4effa33519",
"installed_by": ["modules"]
},
"samtools/fastq": {
"branch": "master",
"git_sha": "911696ea0b62df80e900ef244d7867d177971f73",
"git_sha": "04fbbc7c43cebc0b95d5b126f6d9fe4effa33519",
"installed_by": ["modules"]
},
"samtools/flagstat": {
"branch": "master",
"git_sha": "911696ea0b62df80e900ef244d7867d177971f73",
"git_sha": "04fbbc7c43cebc0b95d5b126f6d9fe4effa33519",
"installed_by": ["modules"]
},
"samtools/idxstats": {
"branch": "master",
"git_sha": "911696ea0b62df80e900ef244d7867d177971f73",
"git_sha": "04fbbc7c43cebc0b95d5b126f6d9fe4effa33519",
"installed_by": ["modules"]
},
"samtools/merge": {
"branch": "master",
"git_sha": "0460d316170f75f323111b4a2c0a2989f0c32013",
"git_sha": "04fbbc7c43cebc0b95d5b126f6d9fe4effa33519",
"installed_by": ["modules"],
"patch": "modules/nf-core/samtools/merge/samtools-merge.diff"
},
"samtools/stats": {
"branch": "master",
"git_sha": "735e1e04e7e01751d2d6e97055bbdb6f70683cc1",
"git_sha": "04fbbc7c43cebc0b95d5b126f6d9fe4effa33519",
"installed_by": ["modules"]
},
"samtools/view": {
"branch": "master",
"git_sha": "3ffae3598260a99e8db3207dead9f73f87f90d1f",
"git_sha": "6c2309aaec566c0d44a6cf14d4b2d0c51afe2e91",
"installed_by": ["modules"]
},
"untar": {
"branch": "master",
"git_sha": "5c460c5a4736974abde2843294f35307ee2b0e5e",
"git_sha": "4e5f4687318f24ba944a13609d3ea6ebd890737d",
"installed_by": ["modules"]
}
}
},
"subworkflows": {
"nf-core": {
"utils_nextflow_pipeline": {
"branch": "master",
"git_sha": "5caf7640a9ef1d18d765d55339be751bb0969dfa",
"installed_by": ["subworkflows"]
},
"utils_nfcore_pipeline": {
"branch": "master",
"git_sha": "92de218a329bfc9a9033116eb5f65fd270e72ba3",
"installed_by": ["subworkflows"]
},
"utils_nfvalidation_plugin": {
"branch": "master",
"git_sha": "5caf7640a9ef1d18d765d55339be751bb0969dfa",
"installed_by": ["subworkflows"]
}
}
}
}
}
Expand Down
6 changes: 3 additions & 3 deletions modules/local/samtools_collatetofasta.nf
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@ process SAMTOOLS_COLLATETOFASTA {
tag "$meta.id"
label 'process_medium'

conda "bioconda::samtools=1.17"
conda "bioconda::samtools=1.20"
container "${ workflow.containerEngine == 'singularity' && !task.ext.singularity_pull_docker_container ?
'https://depot.galaxyproject.org/singularity/samtools:1.17--h00cdaf9_0' :
'biocontainers/samtools:1.17--h00cdaf9_0' }"
'https://depot.galaxyproject.org/singularity/samtools:1.20--h50ea8bc_0' :
'biocontainers/samtools:1.20--h50ea8bc_0' }"

input:
tuple val(meta), path(input)
Expand Down
6 changes: 3 additions & 3 deletions modules/local/samtools_filtertofastq.nf
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@ process SAMTOOLS_FILTERTOFASTQ {
tag "$meta.id"
label 'process_low'

conda "bioconda::samtools=1.17"
conda "bioconda::samtools=1.20"
container "${ workflow.containerEngine == 'singularity' && !task.ext.singularity_pull_docker_container ?
'https://depot.galaxyproject.org/singularity/samtools:1.17--h00cdaf9_0' :
'biocontainers/samtools:1.17--h00cdaf9_0' }"
'https://depot.galaxyproject.org/singularity/samtools:1.20--h50ea8bc_0' :
'biocontainers/samtools:1.20--h50ea8bc_0' }"

input:
tuple val(meta), path(input), path(index)
Expand Down
6 changes: 3 additions & 3 deletions modules/local/samtools_replaceheader.nf
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@ process SAMTOOLS_REHEADER {
tag "$meta.id"
label 'process_single'

conda "bioconda::samtools=1.17"
conda "bioconda::samtools=1.20"
container "${ workflow.containerEngine == 'singularity' && !task.ext.singularity_pull_docker_container ?
'https://depot.galaxyproject.org/singularity/samtools:1.17--h00cdaf9_0' :
'biocontainers/samtools:1.17--h00cdaf9_0' }"
'https://depot.galaxyproject.org/singularity/samtools:1.20--h50ea8bc_0' :
'biocontainers/samtools:1.20--h50ea8bc_0' }"

input:
tuple val(meta), path(file)
Expand Down
6 changes: 3 additions & 3 deletions modules/local/samtools_sormadup.nf
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@ process SAMTOOLS_SORMADUP {
tag "$meta.id"
label 'process_medium'

conda "bioconda::samtools=1.17"
conda "bioconda::samtools=1.20"
container "${ workflow.containerEngine == 'singularity' && !task.ext.singularity_pull_docker_container ?
'https://depot.galaxyproject.org/singularity/samtools:1.17--h00cdaf9_0' :
'biocontainers/samtools:1.17--h00cdaf9_0' }"
'https://depot.galaxyproject.org/singularity/samtools:1.20--h50ea8bc_0' :
'biocontainers/samtools:1.20--h50ea8bc_0' }"

input:
tuple val(meta), path(input)
Expand Down
7 changes: 7 additions & 0 deletions modules/nf-core/blast/blastn/environment.yml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

49 changes: 37 additions & 12 deletions modules/nf-core/blast/blastn/main.nf

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading