Skip to content

Fix typo in openms_peak_picker processOption value#689

Open
Copilot wants to merge 3 commits intodevfrom
copilot/fix-typo-in-openms-peak-picker
Open

Fix typo in openms_peak_picker processOption value#689
Copilot wants to merge 3 commits intodevfrom
copilot/fix-typo-in-openms-peak-picker

Conversation

Copy link
Contributor

Copilot AI commented Mar 25, 2026

PeakPickerHiRes -processOption was passed inmermory instead of the valid value inmemory, causing the option to be silently ignored or rejected.

  • modules/local/openms/openms_peak_picker/main.nf: corrected "inmermory""inmemory"
# Before
in_mem = params.peakpicking_inmemory ? "inmermory" : "lowmemory"

# After
in_mem = params.peakpicking_inmemory ? "inmemory" : "lowmemory"
Original prompt

This section details on the original issue you should resolve

<issue_title>Typo in openms_peak_picker</issue_title>
<issue_description>### Description of the bug

inmermory -> inmemory

reference

in_mem = params.peakpicking_inmemory ? "inmermory" : "lowmemory"

Command used and terminal output

Relevant files

No response

System information

No response</issue_description>

<agent_instructions>make the one-line change.</agent_instructions>

Comments on the Issue (you are @copilot in this section)


📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

Copilot AI changed the title [WIP] Fix typo in openms_peak_picker documentation Fix typo in openms_peak_picker processOption value Mar 25, 2026
Copilot AI requested a review from enryH March 25, 2026 20:43
@github-actions
Copy link

github-actions bot commented Mar 26, 2026

nf-core pipelines lint overall result: Passed ✅ ⚠️

Posted for pipeline commit aaa0b91

+| ✅ 110 tests passed       |+
#| ❔  19 tests were ignored |#
!| ❗   3 tests had warnings |!
Details

❗ Test warnings:

  • pipeline_todos - TODO string in nextflow.config: Specify any additional parameters here
  • pipeline_if_empty_null - ifEmpty(null) found in /home/runner/work/quantms/quantms/subworkflows/local/id/main.nf: _ ch_software_versions = ch_software_versions.mix(PHOSPHO_SCORING.out.versions.ifEmpty(null))
    _
  • pipeline_if_empty_null - ifEmpty(null) found in /home/runner/work/quantms/quantms/subworkflows/local/dda_id/main.nf: _ ch_software_versions = ch_software_versions.mix(PHOSPHO_SCORING.out.versions.ifEmpty(null))
    _

❔ Tests ignored:

✅ Tests passed:

Run details

  • nf-core/tools version 3.5.2
  • Run at 2026-03-26 07:34:16

@enryH enryH linked an issue Mar 26, 2026 that may be closed by this pull request
@jpfeuffer jpfeuffer marked this pull request as ready for review March 26, 2026 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Typo in openms_peak_picker

4 participants