Fixing bug issues with SNR; quantms-utils -> quantms-rescoring#544
Fixing bug issues with SNR; quantms-utils -> quantms-rescoring#544ypriverol merged 3 commits intobigbio:devfrom
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Pull Request Overview
This PR addresses bugs in the SNR processing by switching from quantms‑utils to quantms‑rescoring and updating related command parameters and container images.
- Updated container image references and command invocation in the spectrum2features module.
- Introduced a new configuration flag in the test DDA configuration file.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| modules/local/spectrum2features/main.nf | Updated container image and command parameters from quantms‑utils to quantms‑rescoring. |
| conf/test_dda_id.config | Added a new flag to enable SNR feature percolator testing. |
Comments suppressed due to low confidence (1)
modules/local/spectrum2features/main.nf:25
- Ensure the updated command parameters for 'rescoring' (using '--mzml' instead of '--ms_path') match the rescoring CLI documentation to avoid unintended behavior.
rescoring spectrum2feature --mzml "${ms_file}" --idxml "${id_file}" --output "${id_file.baseName}_snr.idXML" 2>&1 | tee add_snr_feature.log
| id_only = true | ||
| enable_pmultiqc = true | ||
| ms2rescore = true | ||
| add_snr_feature_percolator = true |
There was a problem hiding this comment.
[nitpick] Consider updating the usage or configuration documentation to explain the purpose of the new 'add_snr_feature_percolator' flag.
PR checklist
nf-core pipelines lint).nextflow run . -profile test,docker --outdir <OUTDIR>).nextflow run . -profile debug,test,docker --outdir <OUTDIR>).docs/usage.mdis updated.docs/output.mdis updated.CHANGELOG.mdis updated.README.mdis updated (including new tool citations and authors/contributors).