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

New release #198

Merged
merged 230 commits into from
Nov 8, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
230 commits
Select commit Hold shift + click to select a range
458d1d8
drugz first commit
Feb 28, 2024
8a21317
drugz second commit
Feb 28, 2024
4a14dee
DrugZ_template_lint
Feb 28, 2024
d2eb3b1
foldchange_added
Feb 29, 2024
f9fd4fd
drugz removed
Jul 4, 2024
4f3b6ac
Merge remote-tracking branch 'upstream/master'
Jul 4, 2024
e95e0bc
Merge branch 'dev' into master
LaurenceKuhl Jul 4, 2024
cd42df3
Add removal of genes in drugZ with an extra parameter
LaurenceKuhl Jul 6, 2024
15f4a6d
Add option of removal of genes
LaurenceKuhl Jul 10, 2024
979638b
Create clonality_classifier.nf
alan-tracey Jul 22, 2024
d11ef15
Update clonality_classifier.nf
alan-tracey Jul 22, 2024
f0d94c1
Create indel_classifier.py
alan-tracey Jul 22, 2024
1d0e5c3
Update indel_classifier.py
alan-tracey Jul 22, 2024
e30b89b
Update clonality_classifier.nf
alan-tracey Jul 22, 2024
63bc7d4
Update clonality_classifier.nf
alan-tracey Jul 22, 2024
b53f628
Update clonality_classifier.nf
alan-tracey Jul 22, 2024
fb37547
Update clonality_classifier.nf
alan-tracey Jul 22, 2024
b66ea4b
Merge branch 'dev' into master
mirpedrol Jul 22, 2024
e6d1f87
Update modules/local/clonality_classifier.nf
alan-tracey Jul 22, 2024
d9decb8
Update templates/indel_classifier.py
alan-tracey Jul 22, 2024
9e5c469
Update templates/indel_classifier.py
alan-tracey Jul 22, 2024
baa901a
Update templates/indel_classifier.py
alan-tracey Jul 22, 2024
03a6cf9
Update crisprseq_targeted.nf
alan-tracey Jul 22, 2024
4d43db7
add @alan-tracey to contributors list
mirpedrol Jul 22, 2024
40063e0
publish classify_clonality versions and output file to publish dir
mirpedrol Jul 22, 2024
37c75c1
add parameter skip_clonality
mirpedrol Jul 23, 2024
f779bf6
Apply suggestions from code review
mirpedrol Jul 23, 2024
e70b118
[automated] Fix code linting
nf-core-bot Jul 23, 2024
adab60a
fix module name and container
mirpedrol Jul 23, 2024
8f9312f
more fixes
mirpedrol Jul 23, 2024
5a883fd
remove specific publishDir to use the default one
mirpedrol Jul 23, 2024
f8f8778
Merge branch 'dev' into master
mirpedrol Jul 23, 2024
70071bf
Fix No such variable: CUTADAPT error by replacing CUTADAPT.out with C…
Jul 23, 2024
a4763fc
bump version to 2.3.0dev
mirpedrol Jul 23, 2024
a5ffb81
Merge branch 'dev' into bump-2.3.0dev
mirpedrol Jul 23, 2024
3b834a1
Merge pull request #181 from mirpedrol/bump-2.3.0dev
mirpedrol Jul 23, 2024
9528a3b
Merge branch 'dev' into master
mirpedrol Jul 23, 2024
438c368
update CHANGELOG.md
mirpedrol Jul 23, 2024
8b5dfa5
Merge pull request #178 from alan-tracey/master
mirpedrol Jul 23, 2024
b00808f
Add drugZ as a parameter with contrast file
LaurenceKuhl Jul 24, 2024
dba723b
Merge branch 'nf-core:master' into pr-122
LaurenceKuhl Jul 24, 2024
ad1b023
Start doc
LaurenceKuhl Jul 24, 2024
56d1032
Merge branch 'pr-122' of https://github.com/LaurenceKuhl/crisprseq in…
LaurenceKuhl Jul 24, 2024
bbfa401
Update documentation
LaurenceKuhl Jul 24, 2024
e34da36
Ran pre commit
LaurenceKuhl Jul 24, 2024
7a9fbf8
Fix typo
LaurenceKuhl Jul 24, 2024
00a12ff
Merge branch 'dev' into pr-122
LaurenceKuhl Jul 24, 2024
6ef1df3
add control_sgrna option
LaurenceKuhl Jul 24, 2024
c1c3d03
Fix empty channel
LaurenceKuhl Jul 24, 2024
472d8d9
Ran prettier
LaurenceKuhl Jul 24, 2024
680b541
Update docs/output/screening.md
LaurenceKuhl Jul 25, 2024
e6c4478
Update docs/usage/screening.md
LaurenceKuhl Jul 25, 2024
d843f86
Update nextflow_schema.json
LaurenceKuhl Jul 25, 2024
5b5042f
Ran pre commit
LaurenceKuhl Jul 25, 2024
ab5e0de
run pipeline tests with singularity and conda too
mirpedrol Jul 25, 2024
7149626
update changelog
mirpedrol Jul 25, 2024
24188cb
set up conda and singularity
mirpedrol Jul 25, 2024
3b6ea5b
run all tests even if one fails
mirpedrol Jul 25, 2024
a6fd18d
Comments from reviewer
LaurenceKuhl Jul 25, 2024
315819e
Specify documentation
LaurenceKuhl Jul 25, 2024
5cfc4ee
Merge branch 'master' into dev
mirpedrol Jul 25, 2024
d0b2b53
fix libjpeg.so.9: No such file or directory error in conda container
mirpedrol Jul 25, 2024
bc2a8ba
Merge branch 'dev' into pr-122
LaurenceKuhl Jul 25, 2024
e593c46
fix umi clustering error caused by updating vsearch/cluster module
mirpedrol Jul 25, 2024
d0918a1
Merge branch 'dev' into fix-cutadapt-error
joannakraw Jul 25, 2024
e7cf03a
add suggestions
LaurenceKuhl Jul 25, 2024
27e218e
Merge branch 'pr-122' of https://github.com/LaurenceKuhl/crisprseq in…
LaurenceKuhl Jul 25, 2024
1da7769
Add contributor in readme and change in changelog
LaurenceKuhl Jul 25, 2024
4f5282e
update medaca version in conda to solve tensorflow not found error
mirpedrol Jul 25, 2024
71676e6
Merge pull request #187 from joannakraw/fix-cutadapt-error
LaurenceKuhl Jul 25, 2024
899a844
fix medaka process by passing the model as a file
mirpedrol Jul 25, 2024
ca3bb8c
Merge branch 'dev' into pr-122
LaurenceKuhl Jul 26, 2024
30e8519
Try fixes
LaurenceKuhl Jul 26, 2024
d7e6d61
fix typo
LaurenceKuhl Jul 26, 2024
2fdc47d
Merge branch 'dev' into add-more-tests
LaurenceKuhl Jul 26, 2024
90da200
Try different conda version for matrice creation
LaurenceKuhl Jul 26, 2024
40d23e6
Merge branch 'add-more-tests' of https://github.com/mirpedrol/crisprs…
LaurenceKuhl Jul 26, 2024
4df0815
Update mageckflute conda recipe for test_screening_rra
LaurenceKuhl Jul 26, 2024
8c191df
Merge branch 'dev' into control-sgrna
LaurenceKuhl Aug 1, 2024
3961eda
Ran pre commit with new hooks
LaurenceKuhl Aug 1, 2024
915dccc
set NXF_SINGULARITY_HOME_MOUNT to true for singularity
mirpedrol Aug 1, 2024
944d988
Update bin/BAGEL.py
LaurenceKuhl Aug 1, 2024
52dd1df
Update .pre-commit-config.yaml
LaurenceKuhl Aug 1, 2024
730e7b3
Update bin/drugz.py
LaurenceKuhl Aug 1, 2024
b0e45e0
Update bin/BAGEL.py
LaurenceKuhl Aug 1, 2024
3e9c21e
Update .pre-commit-config.yaml
LaurenceKuhl Aug 1, 2024
5c7d79f
Merge pull request #168 from LaurenceKuhl/pr-122
LaurenceKuhl Aug 1, 2024
dde4ce1
try fixing corrupted cache from mageck flute Singularity container
mirpedrol Aug 1, 2024
f1cb399
Merge branch 'dev' into control-sgrna
LaurenceKuhl Aug 1, 2024
11b61b5
Fix design matrix bug
LaurenceKuhl Aug 13, 2024
1a2c142
Fix FluteMLE channels
LaurenceKuhl Aug 13, 2024
8fd0285
Update nextflow.config
LaurenceKuhl Aug 13, 2024
256d548
Update workflows/crisprseq_screening.nf
LaurenceKuhl Aug 13, 2024
6e37c0d
Commit suggestions, add the initialisation in the initialisation subw…
LaurenceKuhl Aug 13, 2024
499f8cb
patch mageck mle
LaurenceKuhl Aug 13, 2024
0844e0c
Add a small clarification in the doc
LaurenceKuhl Aug 13, 2024
53aabc2
Update modules/local/mageck/flutemle.nf
LaurenceKuhl Aug 14, 2024
64266f7
Add explanations
LaurenceKuhl Aug 14, 2024
724c897
hit selection first draft
LaurenceKuhl Aug 16, 2024
87ba3ce
Merge pull request #190 from LaurenceKuhl/fix_design
LaurenceKuhl Aug 19, 2024
f032bb6
Merge branch 'dev' of https://github.com/nf-core/crisprseq into contr…
LaurenceKuhl Aug 20, 2024
f1a9f88
Merge branch 'dev' into control-sgrna
LaurenceKuhl Aug 20, 2024
4f20186
Ran pre commit
LaurenceKuhl Aug 20, 2024
56e07c0
Test
LaurenceKuhl Aug 20, 2024
22a90af
Merge branch 'control-sgrna' of https://github.com/nf-core/crisprseq …
LaurenceKuhl Aug 20, 2024
70b9959
Fix definition of mle_control_sgrna
LaurenceKuhl Aug 20, 2024
7acb16b
Merge pull request #184 from nf-core/control-sgrna
LaurenceKuhl Aug 20, 2024
4673818
Test
LaurenceKuhl Aug 20, 2024
b52ee8f
Update PR number
LaurenceKuhl Aug 20, 2024
0d6cfc4
Update README.md
LaurenceKuhl Aug 20, 2024
671ca2f
Alphabetically sort the order
LaurenceKuhl Aug 21, 2024
bdb7682
Merge pull request #192 from LaurenceKuhl/dev
LaurenceKuhl Aug 21, 2024
093e5c1
Test
LaurenceKuhl Aug 23, 2024
88118c9
First running hitselection
LaurenceKuhl Aug 27, 2024
47476a7
Merge branch 'dev' into hit_selection
LaurenceKuhl Aug 27, 2024
6077809
Add smaller steps for the test
LaurenceKuhl Aug 27, 2024
ae842d5
Merge branch 'hit_selection' of github.com:laurencekuhl/crisprseq int…
LaurenceKuhl Aug 27, 2024
f66fa7b
Fix hitselection
LaurenceKuhl Aug 28, 2024
7e7c52a
Polish the plot
LaurenceKuhl Aug 29, 2024
f79f7b7
hit selection documentation is added.
Aug 29, 2024
049fb2b
Add hitselection to mageck mle
LaurenceKuhl Aug 30, 2024
17c9558
Test
Aug 30, 2024
73bbca4
Add usage
LaurenceKuhl Sep 2, 2024
af36f3a
Update the nextflow config
LaurenceKuhl Sep 2, 2024
5a44897
Update docs/output/screening.md
LaurenceKuhl Sep 2, 2024
9a78d06
Update changelog
LaurenceKuhl Sep 2, 2024
e010fa8
Merge branch 'hit_selection' of github.com:laurencekuhl/crisprseq int…
LaurenceKuhl Sep 2, 2024
933d819
very early version of nf-gpt implementation
Sep 2, 2024
ada64fd
fixed some linting test errors
Sep 2, 2024
fd2367c
Add hitselection on bagel2 and if statements around the hitselection …
LaurenceKuhl Sep 2, 2024
1cd1007
fixed more linting test errors
Sep 2, 2024
0b995cc
Update docs/usage/screening.md
LaurenceKuhl Sep 2, 2024
6ac92a0
Add the nf-gpt version in the nextflow config
LaurenceKuhl Sep 2, 2024
7399348
Add the specific folder
LaurenceKuhl Sep 3, 2024
97c827d
Update docs/usage/screening.md
LaurenceKuhl Sep 4, 2024
19744a9
Update docs/usage/screening.md
LaurenceKuhl Sep 4, 2024
98d683e
Add a iteration number in the rra tests
LaurenceKuhl Sep 4, 2024
5f241de
Merge pull request #191 from LaurenceKuhl/hit_selection
LaurenceKuhl Sep 4, 2024
90ffcc0
Merge branch 'dev' of https://github.com/nf-core/crisprseq into dev
LaurenceKuhl Sep 5, 2024
699c49c
Run prettier
LaurenceKuhl Sep 5, 2024
bc038b0
Remove the gpt input
LaurenceKuhl Sep 5, 2024
b5775de
update CHANGELOG
LaurenceKuhl Sep 5, 2024
e5467c9
Drop down the hitselection interation
LaurenceKuhl Sep 5, 2024
516ff4c
Fix the input so that all processes run
LaurenceKuhl Sep 6, 2024
f5d2248
Delete templates/collect_gene_ids.py
LaurenceKuhl Sep 6, 2024
2c4b173
Update workflows/crisprseq_screening.nf
LaurenceKuhl Sep 6, 2024
6936128
Update docs/usage/screening.md
LaurenceKuhl Sep 6, 2024
ff84f28
Update docs/usage/screening.md
LaurenceKuhl Sep 6, 2024
42e494c
Update workflows/crisprseq_screening.nf
LaurenceKuhl Sep 6, 2024
bda7258
Update docs/usage/screening.md
LaurenceKuhl Sep 6, 2024
b8ca422
Update docs/usage/screening.md
LaurenceKuhl Sep 6, 2024
7ba7fee
Update nextflow_schema.json
LaurenceKuhl Sep 6, 2024
16a4a57
Update nextflow_schema.json
LaurenceKuhl Sep 6, 2024
f46857f
Update workflows/crisprseq_screening.nf
LaurenceKuhl Sep 6, 2024
8d5a3d4
Change docs
LaurenceKuhl Sep 6, 2024
59fa3d6
Update docs/usage/screening.md
LaurenceKuhl Sep 6, 2024
e9e9d4e
Merge pull request #194 from LaurenceKuhl/params
LaurenceKuhl Sep 6, 2024
35aa587
Changes
LaurenceKuhl Sep 16, 2024
9519db6
Fix merge
LaurenceKuhl Sep 16, 2024
f39b281
Add julias comments
LaurenceKuhl Sep 16, 2024
a27912a
Fix the decreasing rank
LaurenceKuhl Sep 16, 2024
1b8d03d
Merge branch 'dev' into params
LaurenceKuhl Sep 16, 2024
3d167c4
Fix name in the venn diagram
LaurenceKuhl Sep 16, 2024
0301862
Merge branch 'params' of github.com:laurencekuhl/crisprseq into params
LaurenceKuhl Sep 16, 2024
a2fb54b
Update docs/usage/screening.md
LaurenceKuhl Sep 16, 2024
b97469f
Update docs/usage/screening.md
LaurenceKuhl Sep 16, 2024
8b3813b
Update docs/usage/screening.md
LaurenceKuhl Sep 16, 2024
f8e11ea
Ran pre-commit
LaurenceKuhl Sep 17, 2024
72134bb
Merge pull request #196 from LaurenceKuhl/params
LaurenceKuhl Sep 17, 2024
94b4b8d
version bump
LaurenceKuhl Sep 17, 2024
20ab38d
Merge branch 'dev' into release
LaurenceKuhl Sep 17, 2024
ef4a459
Remove deprecated from the changelog
LaurenceKuhl Sep 17, 2024
b9e0241
Merge branch 'release' of github.com:laurencekuhl/crisprseq into release
LaurenceKuhl Sep 17, 2024
c589ccb
Merge pull request #197 from LaurenceKuhl/release
LaurenceKuhl Sep 17, 2024
0f595d3
Remove 2 comments in BAGEl2
LaurenceKuhl Sep 18, 2024
aa17b91
Merge pull request #199 from LaurenceKuhl/release
LaurenceKuhl Sep 18, 2024
ab14e2b
Update tubemap
LaurenceKuhl Sep 19, 2024
1a6156f
Merge branch 'dev' into release
LaurenceKuhl Sep 19, 2024
a9b3b78
Merge pull request #200 from LaurenceKuhl/release
LaurenceKuhl Sep 19, 2024
dd7156e
Fix typos
LaurenceKuhl Sep 23, 2024
12bd577
Merge branch 'release' of github.com:laurencekuhl/crisprseq into release
LaurenceKuhl Sep 23, 2024
55ab581
add docs for clonality classifier module
mirpedrol Sep 30, 2024
bb0aa63
Template update for nf-core/tools version 2.14.2.dev0
mirpedrol Oct 2, 2024
a038eb5
Merge pull request #203 from mirpedrol/clonality-docs
mirpedrol Oct 2, 2024
3f7bb34
Merge pull request #202 from LaurenceKuhl/release
LaurenceKuhl Oct 7, 2024
9a0849d
Template update for nf-core/tools version 3.0.0
nf-core-bot Oct 8, 2024
8273def
Template update for nf-core/tools version 3.0.1
nf-core-bot Oct 9, 2024
a84c3df
Template update for nf-core/tools version 3.0.2
nf-core-bot Oct 11, 2024
2aee95a
Modify date
LaurenceKuhl Oct 16, 2024
5b1b490
Merge branch 'dev' into release
LaurenceKuhl Oct 16, 2024
584c989
Merge pull request #209 from LaurenceKuhl/release
mirpedrol Oct 16, 2024
7bcb14d
Merge remote-tracking branch 'origin/dev' into nf-core-template-merge…
mashehu Oct 16, 2024
0fa8505
Update .github/workflows/awsfulltest.yml
LaurenceKuhl Oct 17, 2024
59e7775
Add the mod in screening aws test
LaurenceKuhl Oct 17, 2024
ab4669d
Merge branch 'manual-template-update-3_0_2' of https://github.com/nf-…
LaurenceKuhl Oct 17, 2024
b8c17f0
Update .github/workflows/ci.yml
LaurenceKuhl Oct 17, 2024
e927d7d
Add the function
LaurenceKuhl Oct 17, 2024
d9e4390
Add 2 empty channels in multiqc
LaurenceKuhl Oct 17, 2024
5f12dcf
Try again
LaurenceKuhl Oct 17, 2024
5c741bb
add empty channels to both workflows
mashehu Oct 17, 2024
4e3cef0
Merge branch 'manual-template-update-3_0_2' of https://github.com/nf-…
mashehu Oct 17, 2024
9fb3903
fix resourceLimits
mashehu Oct 17, 2024
1966d85
Merge pull request #210 from nf-core/manual-template-update-3_0_2
LaurenceKuhl Oct 18, 2024
be3bdc4
Merge branch 'dev' into add-more-tests
mirpedrol Oct 22, 2024
51b28fb
fix bug in ci.yml
mirpedrol Oct 22, 2024
8e8653e
fix default medaka_model in json schema
mirpedrol Oct 22, 2024
36ba081
Update templates/template_fluteMLE.R
LaurenceKuhl Oct 28, 2024
c3f7184
Merge pull request #185 from mirpedrol/add-more-tests
LaurenceKuhl Oct 28, 2024
9157248
Fix conda issues
LaurenceKuhl Oct 29, 2024
a9774f1
Merge branch 'dev' into dev
LaurenceKuhl Oct 29, 2024
cfb11a7
uncomment including pipeline config
mirpedrol Oct 29, 2024
28d66c8
Merge pull request #212 from LaurenceKuhl/dev
LaurenceKuhl Oct 29, 2024
d95cefc
Merge branch 'dev' into fix-linting
LaurenceKuhl Oct 29, 2024
eacc5e7
Merge pull request #213 from mirpedrol/fix-linting
mirpedrol Oct 29, 2024
886228c
Try to fix RRA
LaurenceKuhl Oct 29, 2024
b345455
Merge branch 'dev' into dev
LaurenceKuhl Oct 29, 2024
45eb416
Add singularity to the CI
LaurenceKuhl Oct 29, 2024
a0b30ee
Merge branch 'dev' of github.com:laurencekuhl/crisprseq into dev
LaurenceKuhl Oct 29, 2024
1ea9a93
Add temp location
LaurenceKuhl Oct 29, 2024
c6f5c19
try defining BIOMART_CACHE for mageck flute
mirpedrol Oct 29, 2024
9f0de4f
Modified to fix template_fluteMLE.R for Apptainer/Singularity
medmaca Nov 1, 2024
84a919a
Modified graphrra.nf for Apptainer/Singularity.
medmaca Nov 7, 2024
39f5f54
pre commit
LaurenceKuhl Nov 8, 2024
a7bc829
Update changelog
LaurenceKuhl Nov 8, 2024
5316dc9
Add Matthew as a conntributer
LaurenceKuhl Nov 8, 2024
957b5f2
Update .github/workflows/ci.yml
LaurenceKuhl Nov 8, 2024
be6922e
Merge pull request #214 from LaurenceKuhl/dev
LaurenceKuhl Nov 8, 2024
f91f470
put in the right date for the the release (finally) (template update …
LaurenceKuhl Nov 8, 2024
9ea138e
Merge branch 'nf-core:dev' into dev
LaurenceKuhl Nov 8, 2024
72afaab
Merge pull request #215 from LaurenceKuhl/dev
LaurenceKuhl Nov 8, 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
3 changes: 3 additions & 0 deletions .editorconfig
Original file line number Diff line number Diff line change
Expand Up @@ -31,3 +31,6 @@ indent_size = unset
# ignore python and markdown
[*.{py,md}]
indent_style = unset

[/assets/hgnc_complete_set.txt]
trim_trailing_whitespace = unset
12 changes: 6 additions & 6 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/crisprseq, the standard workflow is
1. Check that there isn't already an issue about your idea in the [nf-core/crisprseq issues](https://github.com/nf-core/crisprseq/issues) to avoid duplicating work. 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/crisprseq repository](https://github.com/nf-core/crisprseq) 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 pipelines 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 All @@ -40,7 +40,7 @@ There are typically two types of tests that run:
### Lint tests

`nf-core` has a [set of guidelines](https://nf-co.re/developers/guidelines) which all pipelines must adhere to.
To enforce these and ensure that all pipelines stay in sync, we have developed a helper tool which runs checks on the pipeline code. This is in the [nf-core/tools repository](https://github.com/nf-core/tools) and once installed can be run locally with the `nf-core lint <pipeline-directory>` command.
To enforce these and ensure that all pipelines stay in sync, we have developed a helper tool which runs checks on the pipeline code. This is in the [nf-core/tools repository](https://github.com/nf-core/tools) and once installed can be run locally with the `nf-core pipelines lint <pipeline-directory>` command.

If any failures or warnings are encountered, please follow the listed URL for more documentation.

Expand Down Expand Up @@ -75,7 +75,7 @@ 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 parameters to `nextflow.config` with a default (see below).
5. Add any new parameters to `nextflow_schema.json` with help text (via the `nf-core schema build` tool).
5. Add any new parameters to `nextflow_schema.json` with help text (via the `nf-core pipelines schema build` tool).
6. Add sanity checks and validation for all relevant parameters.
7. Perform local tests to validate that the new code works as expected.
8. If applicable, add a new test command in `.github/workflow/ci.yml`.
Expand All @@ -86,11 +86,11 @@ 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 pipelines schema build` to add to `nextflow_schema.json`.

### Default processes resource requirements

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.
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/main/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.cpus}` and `${task.memory}` variables in the `script:` block.

Expand All @@ -103,7 +103,7 @@ Please use the following naming schemes, to make it easy to understand what is g

### Nextflow version bumping

If you are using a new feature from core Nextflow, you may bump the minimum required version of nextflow in the pipeline with: `nf-core bump-version --nextflow . [min-nf-version]`
If you are using a new feature from core Nextflow, you may bump the minimum required version of nextflow in the pipeline with: `nf-core pipelines bump-version --nextflow . [min-nf-version]`

### Images and figures

Expand Down
2 changes: 1 addition & 1 deletion .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ Learn more about contributing: [CONTRIBUTING.md](https://github.com/nf-core/cris
- [ ] 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/crisprseq/tree/master/.github/CONTRIBUTING.md)
- [ ] If necessary, also make a PR on the nf-core/crisprseq _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 pipelines 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.
Expand Down
25 changes: 21 additions & 4 deletions .github/workflows/awsfulltest.yml
Original file line number Diff line number Diff line change
@@ -1,18 +1,35 @@
name: nf-core AWS full size tests targeted
# This workflow is triggered on published releases.
# This workflow is triggered on PRs opened against the master branch.
# It can be additionally triggered manually with GitHub actions workflow dispatch button.
# It runs the -profile 'test_full' on AWS batch

on:
release:
types: [published]
pull_request:
branches:
- master
workflow_dispatch:
pull_request_review:
types: [submitted]

jobs:
run-platform:
name: Run AWS full tests
if: github.repository == 'nf-core/crisprseq'
# run only if the PR is approved by at least 2 reviewers and against the master branch or manually triggered
if: github.repository == 'nf-core/crisprseq' && github.event.review.state == 'approved' && github.event.pull_request.base.ref == 'master' || github.event_name == 'workflow_dispatch'
runs-on: ubuntu-latest
steps:
- uses: octokit/[email protected]
id: check_approvals
with:
route: GET /repos/${{ github.repository }}/pulls/${{ github.event.pull_request.number }}/reviews
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- id: test_variables
if: github.event_name != 'workflow_dispatch'
run: |
JSON_RESPONSE='${{ steps.check_approvals.outputs.data }}'
CURRENT_APPROVALS_COUNT=$(echo $JSON_RESPONSE | jq -c '[.[] | select(.state | contains("APPROVED")) ] | length')
test $CURRENT_APPROVALS_COUNT -ge 2 || exit 1 # At least 2 approvals are required
- name: Launch workflow via Seqera Platform
uses: seqeralabs/action-tower-launch@v2

Expand Down
22 changes: 17 additions & 5 deletions .github/workflows/awsfulltest_screening.yml
Original file line number Diff line number Diff line change
@@ -1,18 +1,30 @@
name: nf-core AWS full size tests screening
# This workflow is triggered on published releases.
# This workflow is triggered on PRs opened against the master branch.
# It can be additionally triggered manually with GitHub actions workflow dispatch button.
# It runs the -profile 'test_full' on AWS batch

on:
release:
types: [published]
workflow_dispatch:
pull_request:
branches:
- master
jobs:
run-platform:
name: Run AWS full tests
if: github.repository == 'nf-core/crisprseq'
if: github.repository == 'nf-core/crisprseq' && github.event.review.state == 'approved' && github.event.pull_request.base.ref == 'master' || github.event_name == 'workflow_dispatch'
runs-on: ubuntu-latest
steps:
- uses: octokit/[email protected]
id: check_approvals
with:
route: GET /repos/${{ github.repository }}/pulls/${{ github.event.pull_request.number }}/reviews
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- id: test_variables
if: github.event_name != 'workflow_dispatch'
run: |
JSON_RESPONSE='${{ steps.check_approvals.outputs.data }}'
CURRENT_APPROVALS_COUNT=$(echo $JSON_RESPONSE | jq -c '[.[] | select(.state | contains("APPROVED")) ] | length')
test $CURRENT_APPROVALS_COUNT -ge 2 || exit 1 # At least 2 approvals are required
- name: Launch workflow via Seqera Platform
uses: seqeralabs/action-tower-launch@v2
with:
Expand Down
55 changes: 48 additions & 7 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,45 +7,86 @@ on:
pull_request:
release:
types: [published]
workflow_dispatch:

env:
NXF_ANSI_LOG: false
NXF_SINGULARITY_CACHEDIR: ${{ github.workspace }}/.singularity
NXF_SINGULARITY_LIBRARYDIR: ${{ github.workspace }}/.singularity

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 with test data (${{ matrix.NXF_VER }} | ${{ matrix.test_name }} | ${{ matrix.profile }})"
# 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/crisprseq') }}"
runs-on: ubuntu-latest
strategy:
matrix:
NXF_VER:
- "23.04.0"
- "24.04.2"
- "latest-everything"
ANALYSIS:
profile:
- "conda"
- "docker"
- "singularity"
test_name:
- "test_screening"
- "test_screening_paired"
- "test_screening_rra"
- "test_targeted"
- "test_umis"
- "test_screening_count_table"
isMaster:
- ${{ github.base_ref == 'master' }}
# Exclude conda and singularity on dev
exclude:
- isMaster: false
profile: "conda"
- isMaster: false
profile: "singularity"
fail-fast: false # run all tests even if one fails

steps:
- name: Check out pipeline code
uses: actions/checkout@0ad4b8fadaa221de15dcec353f45205ec38ea70b # v4

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

- name: Disk space cleanup
- name: Set up Apptainer
if: matrix.profile == 'singularity'
uses: eWaterCycle/setup-apptainer@main

- name: Set up Singularity
if: matrix.profile == 'singularity'
run: |
mkdir -p $NXF_SINGULARITY_CACHEDIR
mkdir -p $NXF_SINGULARITY_LIBRARYDIR

- name: Set up Miniconda
if: matrix.profile == 'conda'
uses: conda-incubator/setup-miniconda@a4260408e20b96e80095f42ff7f1a15b27dd94ca # v3
with:
miniconda-version: "latest"
auto-update-conda: true
conda-solver: libmamba
channels: conda-forge,bioconda

- name: Set up Conda
if: matrix.profile == 'conda'
run: |
echo $(realpath $CONDA)/condabin >> $GITHUB_PATH
echo $(realpath python) >> $GITHUB_PATH

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

- name: Run pipeline with test data (${{ matrix.ANALYSIS }})
- name: "Run pipeline with test data ${{ matrix.NXF_VER }} | ${{ matrix.test_name }} | ${{ matrix.profile }}"
run: |
nextflow run ${GITHUB_WORKSPACE} -profile ${{ matrix.ANALYSIS }},docker --outdir ./results_${{ matrix.ANALYSIS }}
nextflow run ${GITHUB_WORKSPACE} -profile ${{ matrix.test_name }},${{ matrix.profile }} --outdir ./results_${{ matrix.test_name }}_${{ matrix.profile }}
53 changes: 43 additions & 10 deletions .github/workflows/download_pipeline.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: Test successful pipeline download with 'nf-core download'
name: Test successful pipeline download with 'nf-core pipelines download'

# Run the workflow when:
# - dispatched manually
Expand All @@ -8,7 +8,7 @@ on:
workflow_dispatch:
inputs:
testbranch:
description: "The specific branch you wish to utilize for the test execution of nf-core download."
description: "The specific branch you wish to utilize for the test execution of nf-core pipelines download."
required: true
default: "dev"
pull_request:
Expand Down Expand Up @@ -39,9 +39,11 @@ jobs:
with:
python-version: "3.12"
architecture: "x64"
- uses: eWaterCycle/setup-singularity@931d4e31109e875b13309ae1d07c70ca8fbc8537 # v7

- name: Setup Apptainer
uses: eWaterCycle/setup-apptainer@4bb22c52d4f63406c49e94c804632975787312b3 # v2.0.0
with:
singularity-version: 3.8.3
apptainer-version: 1.3.4

- name: Install dependencies
run: |
Expand All @@ -54,33 +56,64 @@ jobs:
echo "REPOTITLE_LOWERCASE=$(basename ${GITHUB_REPOSITORY,,})" >> ${GITHUB_ENV}
echo "REPO_BRANCH=${{ github.event.inputs.testbranch || 'dev' }}" >> ${GITHUB_ENV}

- name: Make a cache directory for the container images
run: |
mkdir -p ./singularity_container_images

- name: Download the pipeline
env:
NXF_SINGULARITY_CACHEDIR: ./
NXF_SINGULARITY_CACHEDIR: ./singularity_container_images
run: |
nf-core download ${{ env.REPO_LOWERCASE }} \
nf-core pipelines 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-library "quay.io" -l "docker.io" -l "community.wave.seqera.io" \
--container-cache-utilisation 'amend' \
--download-configuration
--download-configuration 'yes'

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

- name: Count the downloaded number of container images
id: count_initial
run: |
image_count=$(ls -1 ./singularity_container_images | wc -l | xargs)
echo "Initial container image count: $image_count"
echo "IMAGE_COUNT_INITIAL=$image_count" >> ${GITHUB_ENV}

- name: Run the downloaded pipeline (stub)
id: stub_run_pipeline
continue-on-error: true
env:
NXF_SINGULARITY_CACHEDIR: ./
NXF_SINGULARITY_CACHEDIR: ./singularity_container_images
NXF_SINGULARITY_HOME_MOUNT: true
run: nextflow run ./${{ env.REPOTITLE_LOWERCASE }}/$( sed 's/\W/_/g' <<< ${{ env.REPO_BRANCH }}) -stub -profile test,singularity --outdir ./results
- name: Run the downloaded pipeline (stub run not supported)
id: run_pipeline
if: ${{ job.steps.stub_run_pipeline.status == failure() }}
env:
NXF_SINGULARITY_CACHEDIR: ./
NXF_SINGULARITY_CACHEDIR: ./singularity_container_images
NXF_SINGULARITY_HOME_MOUNT: true
run: nextflow run ./${{ env.REPOTITLE_LOWERCASE }}/$( sed 's/\W/_/g' <<< ${{ env.REPO_BRANCH }}) -profile test,singularity --outdir ./results

- name: Count the downloaded number of container images
id: count_afterwards
run: |
image_count=$(ls -1 ./singularity_container_images | wc -l | xargs)
echo "Post-pipeline run container image count: $image_count"
echo "IMAGE_COUNT_AFTER=$image_count" >> ${GITHUB_ENV}

- name: Compare container image counts
run: |
if [ "${{ env.IMAGE_COUNT_INITIAL }}" -ne "${{ env.IMAGE_COUNT_AFTER }}" ]; then
initial_count=${{ env.IMAGE_COUNT_INITIAL }}
final_count=${{ env.IMAGE_COUNT_AFTER }}
difference=$((final_count - initial_count))
echo "$difference additional container images were \n downloaded at runtime . The pipeline has no support for offline runs!"
tree ./singularity_container_images
exit 1
else
echo "The pipeline can be downloaded successfully!"
fi
23 changes: 19 additions & 4 deletions .github/workflows/linting.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
name: nf-core linting
# This workflow is triggered on pushes and PRs to the repository.
# It runs the `nf-core lint` and markdown lint tests to ensure
# It runs the `nf-core pipelines lint` and markdown lint tests to ensure
# that the code meets the nf-core guidelines.
on:
push:
Expand Down Expand Up @@ -41,17 +41,32 @@ jobs:
python-version: "3.12"
architecture: "x64"

- name: read .nf-core.yml
uses: pietrobolcato/[email protected]
id: read_yml
with:
config: ${{ github.workspace }}/.nf-core.yml

- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install nf-core
pip install nf-core==${{ steps.read_yml.outputs['nf_core_version'] }}

- name: Run nf-core pipelines lint
if: ${{ github.base_ref != 'master' }}
env:
GITHUB_COMMENTS_URL: ${{ github.event.pull_request.comments_url }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
GITHUB_PR_COMMIT: ${{ github.event.pull_request.head.sha }}
run: nf-core -l lint_log.txt pipelines lint --dir ${GITHUB_WORKSPACE} --markdown lint_results.md

- name: Run nf-core lint
- name: Run nf-core pipelines lint --release
if: ${{ github.base_ref == 'master' }}
env:
GITHUB_COMMENTS_URL: ${{ github.event.pull_request.comments_url }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
GITHUB_PR_COMMIT: ${{ github.event.pull_request.head.sha }}
run: nf-core -l lint_log.txt lint --dir ${GITHUB_WORKSPACE} --markdown lint_results.md
run: nf-core -l lint_log.txt pipelines lint --release --dir ${GITHUB_WORKSPACE} --markdown lint_results.md

- name: Save PR number
if: ${{ always() }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/linting_comment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Download lint results
uses: dawidd6/action-download-artifact@09f2f74827fd3a8607589e5ad7f9398816f540fe # v3
uses: dawidd6/action-download-artifact@bf251b5aa9c2f7eeb574a96ee720e24f801b7c11 # v6
with:
workflow: linting.yml
workflow_conclusion: completed
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release-announcements.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
- name: get topics and convert to hashtags
id: get_topics
run: |
echo "topics=$(curl -s https://nf-co.re/pipelines.json | jq -r '.remote_workflows[] | select(.full_name == "${{ github.repository }}") | .topics[]' | awk '{print "#"$0}' | tr '\n' ' ')" >> $GITHUB_OUTPUT
echo "topics=$(curl -s https://nf-co.re/pipelines.json | jq -r '.remote_workflows[] | select(.full_name == "${{ github.repository }}") | .topics[]' | awk '{print "#"$0}' | tr '\n' ' ')" | sed 's/-//g' >> $GITHUB_OUTPUT
- uses: rzr/fediverse-action@master
with:
Expand Down
Loading
Loading