unified name

[daichengxin]

Pull Request

Description

Checklist

• Module follows nf-core standards
• main.nf includes process definition
• meta.yml includes complete documentation
• environment.yml specifies dependencies
• Tests are included
• Code is formatted (prettier)
• CI checks pass

Module Type

• New module
• Module update
• Bug fix
• Documentation

Related Issues

Closes #

Summary by CodeRabbit

• Refactor
  • Updated parameter references in algorithm configuration for improved consistency; default values remain unchanged.

[qodo-code-review]

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c1547e2f-348d-4748-b388-f89b98b71a2d

📥 Commits

Reviewing files that changed from the base of the PR and between a49a187 and 75c37ee.

📒 Files selected for processing (1)

• modules/bigbio/onsite/main.nf

---

📝 Walkthrough

Walkthrough

In modules/bigbio/onsite/main.nf, the LucXor algorithm command construction is updated to source --max-charge-state and --max-peptide-length flags from params.max_precursor_charge and params.max_peptide_length instead of the previously used params.onsite_max_charge_state and params.onsite_max_peptide_length. Default values remain '5' and '40'.

Changes

LucXor Parameter Rename

Layer / File(s)	Summary
LucXor command parameter sources modules/bigbio/onsite/main.nf	max_charge and max_peptide_len defaults now reference params.max_precursor_charge and params.max_peptide_length instead of the onsite-specific parameter names.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A bunny swapped two names with care,
onsite_max was shed right there.
Now max_precursor_charge leads the way,
And max_peptide_length joins the fray.
Tidy params, hip-hop hooray! 🐇

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codacy-production]

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

_{NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.}