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

Release 2.6.0 #325

Merged
merged 154 commits into from
Jun 17, 2024
Merged

Release 2.6.0 #325

merged 154 commits into from
Jun 17, 2024

Conversation

jonasscheid
Copy link
Collaborator

@jonasscheid jonasscheid commented May 30, 2024

Highlights

Added

  • Added MS²Rescore module with the underlying python CLI #293 🤖
  • Added support for handling various archive formats: d|d.tar.gz|d.tar|d.zip|mzML.gz|raw|RAW|mzML #323
    --> Provide PRIDE ftp link of raw data in samplesheet to directly stage + process projects 🥳

Fixed

  • Create only one decoy database #299 🚀
  • Use groupKey to streamline group-wise processing #310 🚀
  • Replaced local modules with nf-core modules #310 🍏

Comments

  • --use_deeplc and --use_ms2pip were removed and can now be specified via --feature_generators deeplc,ms2pip
  • --ms2pip_model_name was renamed to --ms2pip_model

PR checklist

  • 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
  • If necessary, also make a PR on the nf-core/mhcquant branch on the 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>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

Copy link

github-actions bot commented May 30, 2024

nf-core lint overall result: Passed ✅ ⚠️

Posted for pipeline commit 722343b

+| ✅ 234 tests passed       |+
#| ❔   1 tests were ignored |#
!| ❗   1 tests had warnings |!

❗ Test warnings:

❔ Tests ignored:

  • files_unchanged - File ignored due to lint config: .github/CONTRIBUTING.md

✅ Tests passed:

Run details

  • nf-core/tools version 2.14.1
  • Run at 2024-06-17 11:40:28

Copy link
Collaborator

@marissaDubbelaar marissaDubbelaar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A lot of effort went into this PR!
On a small note, I would change the version to 2.0 since many changes were made, and the prediction of classes and rt is retired at the moment.
Maybe someone from the core team or contributors can give a second review.

docs/usage.md Outdated Show resolved Hide resolved
docs/usage.md Outdated Show resolved Hide resolved
workflows/mhcquant.nf Show resolved Hide resolved
workflows/mhcquant.nf Show resolved Hide resolved
Copy link
Member

@mirpedrol mirpedrol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
I only commented a couple of small details.
It's great that you could change some of the local modules for nf-core modules :) But I am wondering if the remaining local modules such as all the openms tools could also be nf-core modules? note that you can use nf-core modules patch if you need to make small modifications to the module. Anyway, I don't want this to be a blocker for this PR, just something to consider after release.

.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
assets/multiqc_config.yml Outdated Show resolved Hide resolved
docs/output.md Outdated Show resolved Hide resolved
modules/local/ms2rescore.nf Show resolved Hide resolved
modules/local/ms2rescore.nf Show resolved Hide resolved
@jonasscheid jonasscheid merged commit 1b30692 into master Jun 17, 2024
18 of 19 checks passed
@jonasscheid
Copy link
Collaborator Author

Thanks for the reviews!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants