Skip to content

implement annotation subtyping workflow - #23

Merged
dorien-er merged 9 commits into
mainfrom
subtyping
Jul 14, 2026
Merged

implement annotation subtyping workflow#23
dorien-er merged 9 commits into
mainfrom
subtyping

Conversation

@dorien-er

Copy link
Copy Markdown
Contributor

No description provided.

@dorien-er
dorien-er requested a review from jakubmajercik July 13, 2026 09:37
include { parallel_subtyping } from params.rootDir + "/target/nextflow/single_cell/parallel_subtyping/main.nf"
// add_id is an openpipeline dependency; it builds into the version-pinned
// dependency cache rather than target/nextflow (tag set in _viash.yaml).
include { add_id } from params.rootDir + "/target/dependencies/vsh/vsh/openpipeline/v4.1.1/nextflow/metadata/add_id/main.nf"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
include { add_id } from params.rootDir + "/target/dependencies/vsh/vsh/openpipeline/v4.1.1/nextflow/metadata/add_id/main.nf"
include { add_id } from params.rootDir + "/target/dependencies/vsh/vsh/openpipeline/v4.2.0/nextflow/metadata/add_id/main.nf"

Comment thread src/single_cell/parallel_subtyping/test.nf Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could you add another test case to cover --allow_missing_reference_cell_type?

@dorien-er
dorien-er requested a review from jakubmajercik July 14, 2026 08:42

@jakubmajercik jakubmajercik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One more thing

Comment thread src/single_cell/parallel_subtyping/config.vsh.yaml
Co-authored-by: Jakub Majercik <57993790+jakubmajercik@users.noreply.github.com>
@dorien-er
dorien-er merged commit 8e5fd9f into main Jul 14, 2026
2 checks passed
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.

2 participants