Skip to content

Conversation

@dariarom94
Copy link
Contributor

@dariarom94 dariarom94 commented Dec 12, 2025

Describe your changes

Checklist before requesting a review

  • I have performed a self-review of my code

  • Check the correct box. Does this PR contain:

    • Breaking changes
    • New functionality
    • Major changes
    • Minor changes
    • Bug fixes
  • Proposed changes are described in the CHANGELOG.md

  • CI Tests succeed and look good!

@dariarom94 dariarom94 requested a review from LouisK92 December 12, 2025 09:56
Copy link
Contributor

@LouisK92 LouisK92 left a comment

Choose a reason for hiding this comment

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

Added a few comments in the code. But component looks almost perfect now.

To include the method also into the pipeline there are a few files that need small adjustments:

1. workflow configuration

task_ist_preprocessing/src/workflows/run_benchmark/config.vsh.yaml

  • line 101: add singler to default: "ssam:tacco:moscot:mapmycells:tangram"
  • line 172: add singler to dependencies

2. workflow file

task_ist_preprocessing/src/workflows/run_benchmark/main.nf

  • line 379: add singler to ct annotation methods

3. script to run the benchmark

task_ist_preprocessing/scripts/run_benchmark/*.sh
In each script add singler to the celltype_annotation_methods (comment it out if most other methods are commented out)

@LouisK92 LouisK92 merged commit 142e0f5 into main Dec 13, 2025
2 checks passed
@LouisK92 LouisK92 deleted the add-singler branch December 13, 2025 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants