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

Pre-v1 merge #28

Merged
merged 102 commits into from
Jul 18, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
102 commits
Select commit Hold shift + click to select a range
2989f16
Update generate_cram_csv.nf
scorreard Oct 27, 2023
9514488
Merge pull request #15 from scorreard/dev
DLBPointon Oct 30, 2023
cadd40c
Updating everything
DLBPointon Feb 6, 2024
389caeb
Updating testing
DLBPointon Feb 6, 2024
bad5076
Forgotten module
DLBPointon Feb 6, 2024
73cbd14
Channel assignment
DLBPointon Feb 6, 2024
27e1a3f
Messed up workflow
DLBPointon Feb 6, 2024
98d36ef
Messed up workflow
DLBPointon Feb 6, 2024
644b0b1
Messed up workflow
DLBPointon Feb 6, 2024
c59909c
Messed up workflow
DLBPointon Feb 6, 2024
d2144be
Getting closer to parity with TreeVal
DLBPointon Feb 7, 2024
c254312
Resource corrections
DLBPointon Feb 7, 2024
8b7e2be
Resource corrections
DLBPointon Feb 7, 2024
8994796
Missing files and incorrect arg
DLBPointon Feb 7, 2024
a3c792e
Linting and removed docker.useremulation
DLBPointon Feb 7, 2024
e089acd
Ci fix
DLBPointon Feb 7, 2024
9ea01a4
Ci fix
DLBPointon Feb 7, 2024
fd4f5bd
Ci fix
DLBPointon Feb 7, 2024
4da9378
Updating all modules
DLBPointon Feb 8, 2024
9b09fa7
Linting
DLBPointon Feb 8, 2024
8f17dbf
Variable misspell
DLBPointon Feb 8, 2024
0360796
Updating the pretext modules to use local pretext software
DLBPointon Feb 8, 2024
9e0eca6
Merge pull request #18 from sanger-tol/dp24_modules_update
DLBPointon Feb 8, 2024
1c23e8b
updating documents also caught bug that was stopping params file from…
DLBPointon Feb 9, 2024
7ecef09
Merge branch 'dp24_treeval_parity' into dp24_docs_update
DLBPointon Feb 9, 2024
2045e35
caught a rogue pacbio!
DLBPointon Feb 9, 2024
b5814ae
prettier
DLBPointon Feb 9, 2024
226b4dd
subworkflow updates
DLBPointon Feb 9, 2024
5c4c467
subworkflow updates
DLBPointon Feb 9, 2024
7212f0d
subworkflow updates
DLBPointon Feb 9, 2024
0342959
Removing pretext hires snapshot
DLBPointon Feb 9, 2024
a86834c
Merge pull request #19 from sanger-tol/dp24_docs_update
DLBPointon Feb 12, 2024
8359993
updated docs
DLBPointon Feb 13, 2024
885bb87
prettier
DLBPointon Feb 13, 2024
4494480
Merge branch 'dp24_treeval_parity' into dp24_docs_update
DLBPointon Feb 16, 2024
563bef6
Merge pull request #20 from sanger-tol/dp24_docs_update
DLBPointon Feb 16, 2024
f9b8f22
Update CHANGELOG.md
DLBPointon Feb 22, 2024
60ac7f0
Parity
DLBPointon Mar 6, 2024
894cb9b
Parity
DLBPointon Mar 6, 2024
1b96ce7
Parity
DLBPointon Mar 6, 2024
627fe56
Parity
DLBPointon Mar 6, 2024
3719d36
Parity
DLBPointon Mar 6, 2024
728ed9b
Parity
DLBPointon Mar 6, 2024
9d7ae0b
updating test file
DLBPointon Apr 12, 2024
d3c7d6e
Updating the test case
DLBPointon Apr 12, 2024
da6c17d
Updating documentation
DLBPointon Apr 12, 2024
3efbbda
Updating documentation
DLBPointon Apr 12, 2024
ddfd30a
clarification of channels and logic
DLBPointon Apr 12, 2024
7c1ba40
Updating the version information
DLBPointon Apr 12, 2024
c044be9
minor update to ci
DLBPointon Apr 12, 2024
f8fb698
Update to modules to output the proper files
DLBPointon Apr 12, 2024
4456f4d
Updating the input channel setup, niche case caused an error highligh…
DLBPointon Apr 12, 2024
1aa4857
Fix for testing
DLBPointon Apr 12, 2024
5dc3fed
Updating the input channel setup, niche case caused an error highligh…
DLBPointon Apr 12, 2024
145db15
Fixed error where I had created a malformed channel inside a channel
DLBPointon May 24, 2024
ad1f4d6
Update for linting
DLBPointon May 24, 2024
c69ea4c
Erroneous '''
DLBPointon May 24, 2024
43f7b1e
Erroneous '''
DLBPointon May 24, 2024
b00bb0f
Update to Lisence for Arima Script
DLBPointon May 24, 2024
d65efb8
Updating test data and temp dir in modules
DLBPointon May 24, 2024
35e87e8
Fixed the RIGHT files for the test data
DLBPointon May 24, 2024
ac9e5f4
Fixed the pattern for input
DLBPointon May 24, 2024
a3fe905
Comment out two lines which check file location, doesn't work for s3 …
DLBPointon May 24, 2024
d36ecb5
Comment out two lines which check file location, doesn't work for s3 …
DLBPointon May 24, 2024
ff56482
Reverting change to tests and param checks
DLBPointon May 24, 2024
c521a87
Update minimap2 align algorithm
DLBPointon May 28, 2024
f44a52b
Logo for the pipeline
DLBPointon Jul 4, 2024
9b11919
Adding the new logo for the pipeline, only in light version
DLBPointon Jul 4, 2024
5e22cd9
2nd Parity update to 1.1.1 treeval
DLBPointon Jul 4, 2024
2e26ea7
Update to linting
DLBPointon Jul 4, 2024
2f0fd0d
linting
DLBPointon Jul 4, 2024
419ca9b
Update to download containers before running pipeline
DLBPointon Jul 5, 2024
a07757e
Update to download containers before running pipeline
DLBPointon Jul 5, 2024
01c23d1
Update version in command
DLBPointon Jul 5, 2024
fbea6a4
DARK MODE!
DLBPointon Jul 5, 2024
9d9f779
Update for Dark mode
DLBPointon Jul 5, 2024
c0650ec
Merge pull request #17 from sanger-tol/dp24_treeval_parity
DLBPointon Jul 8, 2024
0638e64
Merge remote-tracking branch 'origin/main' into dp24_treeval_parity
DLBPointon Jul 12, 2024
29cd810
Merge branch 'dev' into dp24_treeval_parity
DLBPointon Jul 12, 2024
73490f8
Merge pull request #27 from sanger-tol/dp24_treeval_parity
DLBPointon Jul 12, 2024
b9d2630
Added missing when: clause
muffato Jul 15, 2024
501572c
nf-core lint doesn't like double quotes for labels
muffato Jul 15, 2024
f71aeb2
"process_tiny" doesn't exist
muffato Jul 15, 2024
01a1b26
Added notifications that those modules don't support Conda
muffato Jul 15, 2024
68b369b
nf-core doesn't like the whitespace
muffato Jul 15, 2024
d57f61a
Adding version output to reformat_intersect
DLBPointon Jul 16, 2024
d9a5cec
Updated Version Int
DLBPointon Jul 16, 2024
c037750
Adding citation file and updating version
DLBPointon Jul 16, 2024
c733f15
Linting fix
DLBPointon Jul 17, 2024
8b72624
Changes to CITATIONS and testing for the tupled env
DLBPointon Jul 17, 2024
716ae67
We don't run tests on AWS
muffato Jul 17, 2024
4b3fa6b
Removed badges that are not relevant to us
muffato Jul 17, 2024
43c7a6e
We run prettier, not EditorConfig on .cff files
muffato Jul 17, 2024
fe29d8a
Empty block
muffato Jul 17, 2024
8875267
CITATIONS.md already exists
muffato Jul 17, 2024
353325b
bugfix: the value is in the second element (the first one is meta)
muffato Jul 17, 2024
6c3cf57
Merge pull request #29 from sanger-tol/release_lint
DLBPointon Jul 17, 2024
fe03305
conf/test_full.config
DLBPointon Jul 18, 2024
46f54e7
adding the LSF test profile to the config
DLBPointon Jul 18, 2024
59ea628
Missing quote
DLBPointon Jul 18, 2024
c51a3ac
Changing to the Tiny Test Data
DLBPointon Jul 18, 2024
b72aaac
Merge pull request #30 from sanger-tol/dp24_spelling
DLBPointon Jul 18, 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
34 changes: 0 additions & 34 deletions .github/workflows/awsfulltest.yml

This file was deleted.

29 changes: 0 additions & 29 deletions .github/workflows/awstest.yml

This file was deleted.

43 changes: 35 additions & 8 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@ on:

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 }}"
Expand All @@ -19,14 +21,20 @@ jobs:
test:
name: Run pipeline with test data
# 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 == 'sanger-tol/curationpretextt') }}"
if: "${{ github.event_name != 'push' || (github.event_name == 'push' && github.repository == 'sanger-tol/curationpretext') }}"
runs-on: ubuntu-latest
strategy:
matrix:
NXF_VER:
- "22.10.1"
- "latest-everything"
steps:
- name: Get branch names
# Pulls the names of current branches in repo
# steps.branch-names.outputs.current_branch is used later and returns the name of the branch the PR is made FROM not to
id: branch-names
uses: tj-actions/branch-names@v8

- name: Check out pipeline code
uses: actions/checkout@v3

Expand All @@ -35,17 +43,36 @@ jobs:
with:
version: "${{ matrix.NXF_VER }}"

- name: Setup apptainer
uses: eWaterCycle/setup-apptainer@main

- name: Set up Singularity
run: |
mkdir -p $NXF_SINGULARITY_CACHEDIR
mkdir -p $NXF_SINGULARITY_LIBRARYDIR

- name: Install Python
uses: actions/setup-python@v5
with:
python-version: "3.10"

- name: Install nf-core
run: |
pip install nf-core

- name: NF-Core Download - download singularity containers
# Forcibly download repo on active branch and download SINGULARITY containers into the CACHE dir if not found
# Must occur after singularity install or will crash trying to dl containers
# Zip up this fresh download and run the checked out version
run: |
nf-core download sanger-tol/curationpretext --revision ${{ steps.branch-names.outputs.current_branch }} --compress none -d --force --outdir sanger-curationpretext --container-cache-utilisation amend --container-system singularity

- name: Download test data
# Download A fungal test data set that is full enough to show some real output.
run: |
curl https://tolit.cog.sanger.ac.uk/test-data/resources/treeval/TreeValTinyData.tar.gz | tar xzf -

- name: Run MAPS_ONLY pipeline with test data
# Remember that you can parallelise this by using strategy.matrix
run: |
nextflow run ${GITHUB_WORKSPACE} -profile test,docker --outdir ./results -entry MAPS_ONLY

- name: Run ALL_FILES pipeline with test data
- name: Singularity - Run ALL_FILES pipeline with test data
# Remember that you can parallelise this by using strategy.matrix
run: |
nextflow run ${GITHUB_WORKSPACE} -profile test,docker --outdir ./results
nextflow run ./sanger-curationpretext/${{ steps.branch-names.outputs.current_branch }}/main.nf -profile test,singularity --outdir ./Sing-res
2 changes: 1 addition & 1 deletion .github/workflows/linting.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ jobs:
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')
run: editorconfig-checker -exclude README.md $(find .* -type f | grep -v '.git\|.py\|.md\|cff\|json\|yml\|yaml\|html\|css\|work\|.nextflow\|build\|nf_core.egg-info\|log.txt\|Makefile')

Prettier:
runs-on: ubuntu-latest
Expand Down
3 changes: 3 additions & 0 deletions .nf-core.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ repository_type: pipeline
lint:
files_exist:
- assets/multiqc_config.yml
- assets/nf-core-curationpretext_logo_light.png
- docs/images/nf-core-curationpretext_logo_light.png
- docs/images/nf-core-curationpretext_logo_dark.png
files_unchanged:
- .github/workflows/linting.yml
- LICENSE
Expand Down
55 changes: 54 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,60 @@
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [[1.0.0](https://github.com/sanger-tol/curationpretext/releases/tag/1.0.0)] - UNSC Infinity - [2023-10-02]
## [[1.0.0](https://github.com/sanger-tol/curationpretext/releases/tag/1.0.0)] - UNSC Cradle - [2024-02-22]

### Added

- Subworkflows for both minimap2 and bwamem2 mapping.
- Subworkflow for Pretext accessory file ingestion.
- Considerations for other longread datatypes

### Paramters

| Old Version | New Versions |
| ----------- | --------------- |
| | --aligner |
| | --longread_type |
| --pacbio | --longread |

### Software Dependencies

Note, since the pipeline is using Nextflow DSL2, each process will be run with its own Biocontainer. This means that on occasion it is entirely possible for the pipeline to be using different versions of the same tool. However, the overall software dependency changes compared to the last release have been listed below for reference.

| Module | Old Version | New Versions |
| ------------------------------------------------------------------- | -------------- | -------------- |
| bamtobed_sort ( bedtools + samtools ) | - | 2.31.0 + 1.17 |
| bedtools ( genomecov, bamtobed, intersect, map, merge, makewindows) | 2.31.0 | 2.31.1 |
| bwamem2 index | - | 2.2.1 |
| cram_filter_align_bwamem2_fixmate_sort | - | |
| ^ ( samtools + bwamem2 ) ^ | 1.16.1 + 2.2.1 | 1.17 + 2.2.1 |
| cram_filter_minimap2_filter5end_fixmate_sort | - | |
| ^ ( samtools + minimap2 ) ^ | - | 1.17 + 2.24 |
| extract_cov_id ( coreutils ) | - | 9.1 |
| extract_repeat ( perl ) | - | 5.26.2 |
| extract_telo ( coreutils ) | - | 9.1 |
| find_telomere_regions ( gcc ) | - | 7.1.0 |
| find_telomere_windows ( java-jdk ) | - | 8.0.112 |
| gap_length ( coreutils ) | - | 9.1 |
| generate_cram_csv ( samtools ) | - | 1.17 |
| get_largest_scaff ( coreutils ) | - | 9.1 |
| gnu-sort | - | 8.25 |
| pretextmap + samtools | 0.1.9 + 1.17 | 0.1.9\* + 1.18 |
| pretextgraph | | 0.0.4 |
| pretextsnapshot + UCSC | 0.0.6 + 447 | 0.0.6b + 447 |
| seqtk | - | 1.4 |
| samtools (faidx,merge,sort,view) | 1.17 | 1.18 |
| tabix | - | 1.11 |
| ucsc | 377 | 445 |
| windowmasker (blast) | - | 2.14.0 |

- This version has been modified by @yumisims inorder to expose the texture buffer variable

### Dependencies

### Deprecated

## [[0.1.0](https://github.com/sanger-tol/curationpretext/releases/tag/0.1.0)] - UNSC Infinity - [2023-10-02]

Initial release of sanger-tol/curationpretext, created with the [sager-tol](https://nf-co.re/) template.

Expand Down
34 changes: 34 additions & 0 deletions CITATION.cff
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
# This CITATION.cff file was generated with cffinit.
# Visit https://bit.ly/cffinit to generate yours today!

cff-version: 1.2.0
title: sanger-tol/curationpretext v1.0.0
message: >-
If you use this software, please cite it using the
metadata from this file.
type: software
authors:
- given-names: Damon-Lee Bernard
family-names: Pointon
affiliation: Wellcome Sanger Institute
orcid: "https://orcid.org/0000-0003-2949-6719"
- given-names: Matthieu
family-names: Muffato
affiliation: Wellcome Sanger Institute
orcid: "https://orcid.org/0000-0002-7860-3560"
- given-names: Ying
family-names: Sims
affiliation: Wellcome Sanger Institute
orcid: "https://orcid.org/0000-0003-4765-4872"
- given-names: William
family-names: Eagles
affiliation: Wellcome Sanger Institute
orcid: "https://orcid.org/0009-0006-9956-0404"
identifiers:
- type: doi
value: 10.5281/zenodo.XXXXXXX
repository-code: "https://github.com/sanger-tol/curationpretext"
license: MIT
commit: TODO
version: 1.0.0
date-released: "2024-07-18"
5 changes: 5 additions & 0 deletions LICENSE
Original file line number Diff line number Diff line change
Expand Up @@ -19,3 +19,8 @@ AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
SOFTWARE.


The filter_five_end.ph script has been taken from the Arima Mapping Pipeline, has not been modified and is subject to the below license:

Copyright (c) 2017 Arima Genomics, Inc.
Loading
Loading