Skip to content

Move to nf-core sbwf for GLIMPSE2#240

Merged
LouisLeNezet merged 8 commits intonf-core:devfrom
LouisLeNezet:sbwf_glimpse2
Dec 16, 2025
Merged

Move to nf-core sbwf for GLIMPSE2#240
LouisLeNezet merged 8 commits intonf-core:devfrom
LouisLeNezet:sbwf_glimpse2

Conversation

@LouisLeNezet
Copy link
Collaborator

PR checklist

Replace #220
Add support for genetic map for GLIMPSE2 imputation.

  • 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/phaseimpute branch on the nf-core/test-datasets repository.
  • Make sure your code lints (nf-core pipelines lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,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).

@LouisLeNezet LouisLeNezet changed the base branch from master to dev December 12, 2025 15:28
@LouisLeNezet LouisLeNezet self-assigned this Dec 13, 2025
@LouisLeNezet LouisLeNezet mentioned this pull request Dec 13, 2025
28 tasks
Comment on lines +25 to +26
ch_chunks
.filter { _meta, regionin, regionout -> regionin.size() == 0 && regionout.size() == 0 }
Copy link
Collaborator

Choose a reason for hiding this comment

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

Current code errors only if all entries are predefined, mixed input (some predefined, some empty) can slip through.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

True, I was thinking of adding a more thorough check during the initialisation process to verify that each channel is properly defined.
This will be done in another PR.

…nfig

Co-authored-by: Anabella Trigila <18577080+atrigila@users.noreply.github.com>
@LouisLeNezet LouisLeNezet added the enhancement New feature or request label Dec 15, 2025
This was linked to issues Dec 15, 2025
@LouisLeNezet LouisLeNezet merged commit 28bef6a into nf-core:dev Dec 16, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Map is not supported yet Contribute back to nf-core

2 participants