Skip to content

Creation of a workflow for Post-curation assembly processing#1164

Open
Delphine-L wants to merge 10 commits intogalaxyproject:mainfrom
Delphine-L:postcur
Open

Creation of a workflow for Post-curation assembly processing#1164
Delphine-L wants to merge 10 commits intogalaxyproject:mainfrom
Delphine-L:postcur

Conversation

@Delphine-L
Copy link
Copy Markdown
Contributor

@Delphine-L Delphine-L commented Mar 20, 2026

FOR CONTRIBUTOR:

  • I have read the Adding workflows guidelines
  • License permits unrestricted use (educational + commercial)
  • Please also take note of the reviewer guidelines below to facilitate a smooth review process.

FOR REVIEWERS:

  • .dockstore.yml: file is present and aligned with creator metadata in workflow. ORCID identifiers are strongly encouraged in creator metadata. The .dockstore.yml file is required to run tests
  • Workflow is sufficiently generic to be used with lab data and does not hardcode sample names, reference data and can be run without reading an accompanying tutorial.
  • In workflow: annotation field contains short description of what the workflow does. Should start with This workflow does/runs/performs … xyz … to generate/analyze/etc …
  • In workflow: workflow inputs and outputs have human readable names (spaces are fine, no underscore, dash only where spelling dictates it), no abbreviation unless it is generally understood. Altering input or output labels requires adjusting these labels in the the workflow-tests.yml file as well
  • In workflow: name field should be human readable (spaces are fine, no underscore, dash only where spelling dictates it), no abbreviation unless generally understood
  • Workflow folder: prefer dash (-) over underscore (_), prefer all lowercase. Folder becomes repository in iwc-workflows organization and is included in TRS id
  • Readme explains what workflow does, what are valid inputs and what outputs users can expect. If a tutorial or other resources exist they can be linked. If a similar workflow exists in IWC readme should explain differences with existing workflow and when one might prefer one workflow over another
  • Changelog contains appropriate entries
  • Large files (> 100 KB) are uploaded to zenodo and location urls are used in test file

@Delphine-L Delphine-L changed the title Postcur Creation of a workflow for Post-curation assembly processing Mar 20, 2026
@mvdbeek mvdbeek requested a review from Copilot March 23, 2026 11:51
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new Galaxy workflow package for VGP post-curation assembly processing, including documentation, Dockstore registration, and a Zenodo-backed test definition.

Changes:

  • Added workflow README documenting inputs/outputs for post-curation processing and QC.
  • Added Dockstore descriptor and a workflow test parameter file using Zenodo-hosted fixtures.
  • Added initial changelog entry for v0.1.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 11 comments.

File Description
workflows/VGP-assembly-v2/post-curation-processing/README.md Documents workflow purpose plus detailed input/output expectations
workflows/VGP-assembly-v2/post-curation-processing/Post_Curation-tests.yml Defines a synthetic-data test job and output assertions
workflows/VGP-assembly-v2/post-curation-processing/CHANGELOG.md Introduces initial versioned changelog for the workflow
workflows/VGP-assembly-v2/post-curation-processing/.dockstore.yml Registers the workflow and points Dockstore to the test parameter file

Comment thread workflows/VGP-assembly-v2/post-curation-processing/README.md Outdated
Comment thread workflows/VGP-assembly-v2/post-curation-processing/README.md Outdated
Comment thread workflows/VGP-assembly-v2/post-curation-processing/README.md Outdated
Comment thread workflows/VGP-assembly-v2/post-curation-processing/README.md Outdated
Comment thread workflows/VGP-assembly-v2/post-curation-processing/README.md Outdated
Comment thread workflows/VGP-assembly-v2/post-curation-processing/README.md Outdated
Comment thread workflows/VGP-assembly-v2/post-curation-processing/README.md Outdated
Comment thread workflows/VGP-assembly-v2/post-curation-processing/Post_Curation-tests.yml Outdated
Comment thread workflows/VGP-assembly-v2/post-curation-processing/Post_Curation-tests.yml Outdated
Comment thread workflows/VGP-assembly-v2/post-curation-processing/Post_Curation-tests.yml Outdated
@Delphine-L
Copy link
Copy Markdown
Contributor Author

@github-copilot review

@Delphine-L Delphine-L requested a review from mvdbeek March 26, 2026 14:40
@mvdbeek mvdbeek requested a review from Copilot March 26, 2026 14:57
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 5 changed files in this pull request and generated 6 comments.

Comment thread workflows/VGP-assembly-v2/post-curation-processing/.dockstore.yml
Comment thread workflows/VGP-assembly-v2/post-curation-processing/README.md Outdated
Comment thread workflows/VGP-assembly-v2/post-curation-processing/README.md Outdated
Comment thread workflows/VGP-assembly-v2/post-curation-processing/Post_Curation-tests.yml Outdated
Comment thread workflows/VGP-assembly-v2/post-curation-processing/Post_Curation-tests.yml Outdated
Comment thread workflows/VGP-assembly-v2/post-curation-processing/Post_Curation-tests.yml Outdated
Copy link
Copy Markdown
Member

@mvdbeek mvdbeek left a comment

Choose a reason for hiding this comment

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

Very cool!

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