Skip to content

Feature/improve id encoding and curated path#20

Merged
kfang4 merged 11 commits into
mainfrom
feature/improve-id-encoding-and-curated-path
Apr 24, 2026
Merged

Feature/improve id encoding and curated path#20
kfang4 merged 11 commits into
mainfrom
feature/improve-id-encoding-and-curated-path

Conversation

@kfang4

@kfang4 kfang4 commented Mar 10, 2026

Copy link
Copy Markdown
Collaborator

Description

Use "ID" and "name" consistently in pipeline. Change curated output path to be stored under release version folders instead of workflow version folders.

Dependencies (Issues/PRs)

https://app.clickup.com/t/9014209604/BIOS-2080
https://app.clickup.com/t/9014209604/BIOS-2024

Type of Change

  • Bug fix
  • New feature
  • Documentation

Task Checklist

  • documentation updated
  • miniwdl check passed
  • womtool validate passed

Testing

Workflow Engine Tested On

  • HealthOmics, Amazon Web Services
  • Google Cloud Platform, Cromwell
  • Google Cloud Platform
  • Microsoft Azure, Cromwell
  • GA4GH Workflow Execution Service, On Premises and Multi Cloud
  • Other

Successful Workflow IDs

N/A

@kfang4 kfang4 requested a review from Alexander-Morin April 22, 2026 16:02
@kfang4 kfang4 marked this pull request as ready for review April 22, 2026 16:02

@Alexander-Morin Alexander-Morin left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nice and clean doc 🧹

String workflow_name = "spatial_geomx"
String workflow_version = "v1.0.0"
String workflow_release = "https://github.com/ASAP-CRN/spatial-transcriptomics-wf/releases/tag/spatial_geomx_analysis-~{workflow_version}"
String crn_release_version = "v4.0.0"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same question as for the scRNA-seq pipeline, is this supposed to be hard-coded (and just updated per release?)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yes, it's helpful to keep the release version and workflow version history in the GitHub repo. It's rare that the pipeline won't need an update when running new datasets, so usually the workflow_version and crn_release_version will both need to be bumped. Also, this is now the new way we'll be storing curated outputs-- in a crn_release_version folder instead of workflow_version folder.

Comment thread README.md Outdated
@kfang4 kfang4 merged commit 9ae4a21 into main Apr 24, 2026
1 check passed
@kfang4 kfang4 deleted the feature/improve-id-encoding-and-curated-path branch April 24, 2026 13:38
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