Skip to content

New Module: Octopusv/correct#11540

Merged
manascripts merged 6 commits intonf-core:masterfrom
manascripts:octopusv-correct
May 8, 2026
Merged

New Module: Octopusv/correct#11540
manascripts merged 6 commits intonf-core:masterfrom
manascripts:octopusv-correct

Conversation

@manascripts
Copy link
Copy Markdown
Member

@manascripts manascripts commented May 6, 2026 •
edited
Loading

Description

Add octopusv/correct nf-core module to standardize SV VCFs into OctopuSV SVCF format.

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 module conventions in the contribution docs
  • If necessary, include test data in your PR.
  • Remove all TODO statements.
  • Broadcast software version numbers to topic: versions - See version_topics
  • Follow the naming conventions.
  • Follow the parameters requirements.
  • Follow the input/output options guidelines.
  • Add a resource label
  • Use BioConda and BioContainers if possible to fulfil software requirements.
  • Ensure that the test works with either Docker / Singularity. Conda CI tests can be quite flaky:
    • For modules:
      • nf-core modules test <MODULE> --profile docker
      • nf-core modules test <MODULE> --profile singularity

mashehu
mashehu reviewed May 6, 2026
View reviewed changes
Comment thread modules/nf-core/octopusv/correct/tests/main.nf.test Outdated
@manascripts
Copy link
Copy Markdown
Member Author

manascripts commented May 8, 2026

@nf-core-bot fix linting

mashehu
mashehu reviewed May 8, 2026
View reviewed changes
Comment thread modules/nf-core/octopusv/correct/main.nf Outdated
@manascripts
Copy link
Copy Markdown
Member Author

manascripts commented May 8, 2026

@mashehu All good now?

@manascripts manascripts enabled auto-merge May 8, 2026 11:59
@manascripts manascripts disabled auto-merge May 8, 2026 12:40
manascripts and others added 5 commits May 8, 2026 17:03
@manascripts
Add octopusv/merge module files
64f182a
@manascripts
Add tests and snap
9ffa9d2
@manascripts
add tests and snaps
d616f2d
@manascripts @mashehu
Update modules/nf-core/octopusv/correct/main.nf
6d2edf6 
minor update to line 16 (main.nf)

Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
@manascripts
fix linting issue
9d1817a
@manascripts manascripts enabled auto-merge May 8, 2026 15:05
@manascripts manascripts disabled auto-merge May 8, 2026 15:05
@manascripts manascripts added this pull request to the merge queue May 8, 2026
Merged via the queue into nf-core:master with commit 4ade92b May 8, 2026
23 checks passed
@manascripts manascripts deleted the octopusv-correct branch May 8, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mashehu mashehu mashehu approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@manascripts @mashehu