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

Dev -> Master 2.5.0 #347

Merged
merged 49 commits into from
Oct 18, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
49 commits
Select commit Hold shift + click to select a range
86d73c7
bump version 2.5.0dev
sateeshperi Jun 12, 2023
13cb0fe
Merge pull request #326 from sateeshperi/dev
sateeshperi Jun 12, 2023
5ece807
Template update for nf-core/tools version 2.9
nf-core-bot Jun 30, 2023
9e2d80f
Update modules.config
FelixKrueger Jul 25, 2023
107f736
Update CHANGELOG.md
FelixKrueger Jul 25, 2023
e048710
Apply easy module updates
ewels Jul 31, 2023
ace1a39
Changelog
ewels Jul 31, 2023
e576ec8
Merge pull request #331 from FelixKrueger/publish_coverage_files
ewels Aug 22, 2023
6afd472
Merge branch 'dev' into nf-core-template-merge-2.9
edmundmiller Aug 22, 2023
b565c21
Merge pull request #329 from nf-core/nf-core-template-merge-2.9
edmundmiller Aug 24, 2023
b58eb2a
Merge branch 'dev' into master
edmundmiller Aug 24, 2023
76ef5de
Merge pull request #333 from ewels/master
edmundmiller Aug 24, 2023
2caf09e
Update nextflow_schema.json
wassimsalam01 Aug 17, 2023
591cc41
Update nextflow_schema.json
wassimsalam01 Aug 17, 2023
16007cf
Merge pull request #335 from update_kit_schema
edmundmiller Aug 25, 2023
7910cc1
refactor: Use fromSamplesheet instead of input_check
edmundmiller Aug 25, 2023
47d631c
fix: Add some magic from rnaseq
edmundmiller Aug 25, 2023
b6bb69f
chore: Update CHANGELOG
edmundmiller Aug 25, 2023
675f73c
chore: Update snapshots
edmundmiller Aug 25, 2023
0cd09ec
update tests, snapshots and ci
sateeshperi Aug 28, 2023
89b0aa4
Merge pull request #341 from nf-core/fromSamplesheet
edmundmiller Aug 28, 2023
1737eda
Update nextflow.config
maxulysse Sep 21, 2023
c5bbf8e
Merge pull request #343 from nf-core/maxulysse-patch-1
sateeshperi Sep 21, 2023
8e2a6f3
Template update for nf-core/tools version 2.10
nf-core-bot Sep 25, 2023
ee70918
docs: Update Maintainers
edmundmiller Oct 16, 2023
766a26b
chore: Add CODEOWNERS file
edmundmiller Oct 16, 2023
92312de
chore: Update CHANGELOG
edmundmiller Oct 16, 2023
0fe9fbd
Merge pull request #345 from nf-core/maintainers
edmundmiller Oct 16, 2023
4ae729b
Merge remote-tracking branch 'origin/dev' into nf-core-template-merge…
edmundmiller Oct 16, 2023
8e1411a
Merge pull request #344 from nf-core/nf-core-template-merge-2.10
edmundmiller Oct 16, 2023
aa24e65
feat: Add exists to schema
edmundmiller Aug 28, 2023
f795c9a
feat: Add patterns to schema
edmundmiller Aug 28, 2023
93cd17c
fix(schema): Update formats and exists
edmundmiller Oct 17, 2023
3d63bca
chore: Update CHANGELOG
edmundmiller Aug 28, 2023
b5727f7
Merge pull request #342 from nf-core/schema-exists
edmundmiller Oct 17, 2023
4a747e0
chore: Remove a TODO that was done
edmundmiller Oct 17, 2023
e457cde
chore(#336): Add most of tools to citations bibtex
edmundmiller Oct 17, 2023
d1adc92
chore(#336): Add tools citations
edmundmiller Oct 17, 2023
d2d652f
feat(#336): Try adding a fancy join
edmundmiller Oct 17, 2023
a8bb3b4
chore(#336): Update reference text
edmundmiller Oct 17, 2023
bd1a33f
chore: Add a note to update TrimGalore Citation
edmundmiller Oct 17, 2023
157d807
style: Run bib through citations
edmundmiller Oct 17, 2023
d8a759e
Merge pull request #346 from nf-core/citationtools
edmundmiller Oct 17, 2023
e6db8a8
chore: Add issues that were missed and add date
edmundmiller Oct 18, 2023
ee11838
chore: Bump version
edmundmiller Oct 18, 2023
45c4468
add emojis to changelog v2.5.0
sateeshperi Oct 18, 2023
1814838
bump snapshots for 2.5.0 release
sateeshperi Oct 18, 2023
f4fe5fd
style: Ignore MultiQC config
edmundmiller Oct 18, 2023
e02fd8c
Merge pull request #348 from nf-core/release-prep
edmundmiller Oct 18, 2023
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 .devcontainer/devcontainer.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
"name": "nfcore",
"image": "nfcore/gitpod:latest",
"remoteUser": "gitpod",
"runArgs": ["--privileged"],

// Configure tool-specific properties.
"customizations": {
Expand Down
1 change: 1 addition & 0 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
* @nf-core/methylseq
5 changes: 3 additions & 2 deletions .github/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,9 @@ 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 ;)

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

## Contribution workflow

Expand Down Expand Up @@ -116,4 +118,3 @@ To get started:
Devcontainer specs:

- [DevContainer config](.devcontainer/devcontainer.json)
- [Dockerfile](.devcontainer/Dockerfile)
2 changes: 1 addition & 1 deletion .github/ISSUE_TEMPLATE/bug_report.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ body:
attributes:
label: System information
description: |
* Nextflow version _(eg. 22.10.1)_
* 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, Charliecloud, or Apptainer)_
Expand Down
13 changes: 9 additions & 4 deletions .github/workflows/awsfulltest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,19 +17,24 @@ jobs:
aligner: ["bismark", "bismark_hisat", "bwameth"]
steps:
- name: Launch workflow via tower
uses: seqeralabs/action-tower-launch@v1
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/methylseq/work-${{ github.sha }}
parameters: |
{
"aligner": "${{ matrix.aligner }}",
"hook_url": "${{ secrets.MEGATESTS_ALERTS_SLACK_HOOK_URL }}",
"outdir": "s3://${{ secrets.AWS_S3_BUCKET }}/methylseq/results-${{ github.sha }}/${{matrix.aligner}}"
"aligner": "${{ matrix.aligner }}",
}
profiles: test_full,aws_tower
profiles: test_full

- uses: actions/upload-artifact@v3
with:
name: Tower debug log file
path: tower_action_*.log
path: |
tower_action_*.log
tower_action_*.json
10 changes: 7 additions & 3 deletions .github/workflows/awstest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,19 +15,23 @@ jobs:
steps:
# Launch workflow using Tower CLI tool action
- name: Launch workflow via tower
uses: seqeralabs/action-tower-launch@v1
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/methylseq/work-${{ github.sha }}
parameters: |
{
"aligner": "${{ matrix.aligner }}",
"outdir": "s3://${{ secrets.AWS_S3_BUCKET }}/methylseq/results-test-${{ github.sha }}/${{matrix.aligner}}"
}
profiles: test,aws_tower
profiles: test

- uses: actions/upload-artifact@v3
with:
name: Tower debug log file
path: tower_action_*.log
path: |
tower_action_*.log
tower_action_*.json
47 changes: 25 additions & 22 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,58 +21,59 @@ concurrency:
cancel-in-progress: true

jobs:
define_nxf_versions:
name: Choose nextflow versions to test against depending on target branch
runs-on: ubuntu-latest
outputs:
matrix: ${{ steps.nxf_versions.outputs.matrix }}
steps:
- id: nxf_versions
run: |
if [[ "${{ github.event_name }}" == "pull_request" && "${{ github.base_ref }}" == "dev" && "${{ matrix.NXF_VER }}" != "latest-everything" ]]; then
echo matrix='["latest-everything"]' | tee -a $GITHUB_OUTPUT
else
echo matrix='["latest-everything", "23.04.0"]' | tee -a $GITHUB_OUTPUT
fi

test:
name: Run pipeline with test data
needs: define_nxf_versions
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
NXF_VER:
- "22.10.1"
- "latest-everything"
aligner:
NXF_VER: ${{ fromJson(needs.define_nxf_versions.outputs.matrix) }}
test_tags:
- "bismark"
- "bismark_hisat"
- "bwameth"
profile:
- "docker"
steps:
# Skip if it's a pull_request to dev and NXF_VER is '22.10.1'
- name: Skip condition
id: condition
run: |
if [[ "${{ github.event_name }}" == "pull_request" && "${{ github.base_ref }}" == "dev" && "${{ matrix.NXF_VER }}" != "latest-everything" ]]; then
echo "skip=true" >> $GITHUB_OUTPUT
fi

steps:
- name: Check out pipeline code
if: steps.condition.outputs.skip != 'true'
uses: actions/checkout@v3

- name: Hash Github Workspace
if: steps.condition.outputs.skip != 'true'
id: hash_workspace
run: |
echo "digest=$(echo methylseq2_${{ github.workspace }} | md5sum | cut -c 1-25)" >> $GITHUB_OUTPUT

- name: Cache test data
if: steps.condition.outputs.skip != 'true'
id: cache-testdata
uses: actions/cache@v3
with:
path: test-datasets/
key: ${{ steps.hash_workspace.outputs.digest }}

- name: Check out test data
if: steps.cache-testdata.outputs.cache-hit != 'true' && steps.condition.outputs.skip != 'true'
uses: actions/checkout@v3
with:
repository: nf-core/test-datasets
ref: methylseq
path: test-datasets/

- name: Replace remote paths in samplesheets
if: steps.condition.outputs.skip != 'true'
run: |
for f in ./test-datasets/samplesheet/*csv; do
sed -i 's=https://github.com/nf-core/test-datasets/raw/methylseq/=./test-datasets/=g' $f
Expand All @@ -83,25 +84,27 @@ jobs:
done;

- name: Install Nextflow
if: steps.condition.outputs.skip != 'true'
uses: nf-core/setup-nextflow@v1
with:
version: "${{ matrix.NXF_VER }}"

- name: Install nf-test
if: steps.condition.outputs.skip != 'true'
run: |
wget -qO- https://code.askimed.com/install/nf-test | bash -s $NFTEST_VER
sudo mv nf-test /usr/local/bin/

- name: Run nf-test
if: steps.condition.outputs.skip != 'true'
run: |
nf-test test tests/pipeline/${{ matrix.aligner }}/ --profile "test,${{ matrix.profile }}" --tap=test.tap
nf-test test --tag ${{ matrix.test_tags }} --profile "test,${{ matrix.profile }}" --junitxml=test.xml

# If the test fails, output the software_versions.yml using the 'batcat' utility
- name: Output log on failure
if: failure()
run: |
sudo apt install bat > /dev/null
batcat --decorations=always --color=always ${{ github.workspace }}/.nf-test/tests/*/output/pipeline_info/software_versions.yml

- name: Publish Test Report
uses: mikepenz/action-junit-report@v3
if: always() # always run even if the previous step fails
with:
report_paths: test.xml
2 changes: 1 addition & 1 deletion .github/workflows/linting.yml
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ jobs:

- uses: actions/setup-python@v4
with:
python-version: "3.8"
python-version: "3.11"
architecture: "x64"

- name: Install dependencies
Expand Down
68 changes: 68 additions & 0 deletions .github/workflows/release-announcments.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,68 @@
name: release-announcements
# Automatic release toot and tweet anouncements
on:
release:
types: [published]
workflow_dispatch:

jobs:
toot:
runs-on: ubuntu-latest
steps:
- uses: rzr/fediverse-action@master
with:
access-token: ${{ secrets.MASTODON_ACCESS_TOKEN }}
host: "mstdn.science" # custom host if not "mastodon.social" (default)
# GitHub event payload
# https://docs.github.com/en/developers/webhooks-and-events/webhooks/webhook-events-and-payloads#release
message: |
Pipeline release! ${{ github.repository }} v${{ github.event.release.tag_name }} - ${{ github.event.release.name }}!

Please see the changelog: ${{ github.event.release.html_url }}

send-tweet:
runs-on: ubuntu-latest

steps:
- uses: actions/setup-python@v4
with:
python-version: "3.10"
- name: Install dependencies
run: pip install tweepy==4.14.0
- name: Send tweet
shell: python
run: |
import os
import tweepy

client = tweepy.Client(
access_token=os.getenv("TWITTER_ACCESS_TOKEN"),
access_token_secret=os.getenv("TWITTER_ACCESS_TOKEN_SECRET"),
consumer_key=os.getenv("TWITTER_CONSUMER_KEY"),
consumer_secret=os.getenv("TWITTER_CONSUMER_SECRET"),
)
tweet = os.getenv("TWEET")
client.create_tweet(text=tweet)
env:
TWEET: |
Pipeline release! ${{ github.repository }} v${{ github.event.release.tag_name }} - ${{ github.event.release.name }}!

Please see the changelog: ${{ github.event.release.html_url }}
TWITTER_CONSUMER_KEY: ${{ secrets.TWITTER_CONSUMER_KEY }}
TWITTER_CONSUMER_SECRET: ${{ secrets.TWITTER_CONSUMER_SECRET }}
TWITTER_ACCESS_TOKEN: ${{ secrets.TWITTER_ACCESS_TOKEN }}
TWITTER_ACCESS_TOKEN_SECRET: ${{ secrets.TWITTER_ACCESS_TOKEN_SECRET }}

bsky-post:
runs-on: ubuntu-latest
steps:
- uses: zentered/[email protected]
with:
post: |
Pipeline release! ${{ github.repository }} v${{ github.event.release.tag_name }} - ${{ github.event.release.name }}!

Please see the changelog: ${{ github.event.release.html_url }}
env:
BSKY_IDENTIFIER: ${{ secrets.BSKY_IDENTIFIER }}
BSKY_PASSWORD: ${{ secrets.BSKY_PASSWORD }}
#
5 changes: 5 additions & 0 deletions .gitpod.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,9 @@
image: nfcore/gitpod:latest
tasks:
- name: Update Nextflow and setup pre-commit
command: |
pre-commit install --install-hooks
nextflow self-update

vscode:
extensions: # based on nf-core.nf-core-extensionpack
Expand Down
2 changes: 1 addition & 1 deletion .nf-core.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
# FIXME - CI currently runs for dev and dsl2 branches, remove the latter before merging to dev
repository_type: pipeline
lint:
files_unchanged:
- .gitattributes
files_exist:
- bin/markdown_to_html.py
actions_ci: False
multiqc_config: False
19 changes: 19 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,24 @@
# nf-core/methylseq

## [v2.5.0](https://github.com/nf-core/methylseq/releases/tag/2.5.0) - 2023-10-18

### Pipeline Updates

- 🔄 Updated template to nf-core/tools v2.9
- 🔄 Updated template to nf-core/tools v2.10
- 🔧 Updated nf-core modules for FastQC, samtools sort, samtools flagstat
- ❌ Removes problematic `-m` memory assignment for samtools sort [#81](https://github.com/nf-core/methylseq/issues/81)
- 🧾 Use `fromSamplesheet` from nf-validation [#341](https://github.com/nf-core/methylseq/pull/341)
- 🚀 Update Maintainers and add CODEOWNERS [#345](https://github.com/nf-core/methylseq/pull/345)
- ⚙️ Update schema to utilize exists and add more patterns [#342](https://github.com/nf-core/methylseq/pull/342)
- 📁 Support pipeline-specific configs [#343](https://github.com/nf-core/methylseq/pull/343)

### Bug fixes & refactoring

- 🛠️ Added publishing of coverage (`*cov.gz`) files for NOMe-seq filtered reads for `coverage2cytosine`
- 🛠️ Wrong display values for "zymo" and "em_seq" presets on help page [#335](https://github.com/nf-core/methylseq/pull/335)
- 📚 Use new Citation tools functions [#336](https://github.com/nf-core/methylseq/issues/336)

## [v2.4.0](https://github.com/nf-core/methylseq/releases/tag/2.4.0) - 2023-06-02

### Pipeline Updates
Expand Down
16 changes: 11 additions & 5 deletions CITATIONS.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,12 +16,19 @@

- [FastQC](https://www.bioinformatics.babraham.ac.uk/projects/fastqc/)

> <https://www.bioinformatics.babraham.ac.uk/projects/fastqc/>
> Andrews, S. (2010). FastQC: A Quality Control Tool for High Throughput Sequence Data [Online].

- [MultiQC](https://pubmed.ncbi.nlm.nih.gov/27312411/)

> Ewels P, Magnusson M, Lundin S, Käller M. MultiQC: summarize analysis results for multiple tools and samples in a single report. Bioinformatics. 2016 Oct 1;32(19):3047-8. doi: 10.1093/bioinformatics/btw354. Epub 2016 Jun 16. PubMed PMID: 27312411; PubMed Central PMCID: PMC5039924.

- [Trim Galore!](https://www.bioinformatics.babraham.ac.uk/projects/trim_galore/)

> <https://www.bioinformatics.babraham.ac.uk/projects/trim_galore/>

<!-- TODO https://github.com/FelixKrueger/TrimGalore/issues/25 -->
<!-- https://zenodo.org/records/7598955 -->

- [Bismark](https://doi.org/10.1093/bioinformatics/btr167)

> Felix Krueger, Simon R. Andrews, Bismark: a flexible aligner and methylation caller for Bisulfite-Seq applications, Bioinformatics, Volume 27, Issue 11, 1 June 2011, Pages 1571–1572, doi: [10.1093/bioinformatics/btr167](https://doi.org/10.1093/bioinformatics/btr167)
Expand All @@ -42,10 +49,6 @@

> Daley, T., Smith, A. Predicting the molecular complexity of sequencing libraries. Nat Methods 10, 325–327 (2013). doi: [10.1038/nmeth.2375](https://doi.org/10.1038/nmeth.2375)

- [MultiQC](http://dx.doi.org/10.1093/bioinformatics/btw354/)

> Ewels P, Magnusson M, Lundin S, Käller M. MultiQC: summarize analysis results for multiple tools and samples in a single report. Bioinformatics. 2016 Oct 1;32(19):3047-8. doi: [10.1093/bioinformatics/btw354](http://dx.doi.org/10.1093/bioinformatics/btw354/). Epub 2016 Jun 16. PubMed PMID: 27312411; PubMed Central PMCID: PMC5039924.

## Software packaging/containerisation tools

- [Anaconda](https://anaconda.com)
Expand All @@ -62,5 +65,8 @@

- [Docker](https://dl.acm.org/doi/10.5555/2600239.2600241)

> Merkel, D. (2014). Docker: lightweight linux containers for consistent development and deployment. Linux Journal, 2014(239), 2. doi: 10.5555/2600239.2600241.

- [Singularity](https://pubmed.ncbi.nlm.nih.gov/28494014/)

> Kurtzer GM, Sochat V, Bauer MW. Singularity: Scientific containers for mobility of compute. PLoS One. 2017 May 11;12(5):e0177459. doi: 10.1371/journal.pone.0177459. eCollection 2017. PubMed PMID: 28494014; PubMed Central PMCID: PMC5426675.
Loading
Loading