Skip to content

Add genotyping-array manifest generator script - #47

Merged
joshuamschmidt merged 2 commits into
mainfrom
add-manifest-generator-script
Aug 17, 2026
Merged

Add genotyping-array manifest generator script#47
joshuamschmidt merged 2 commits into
mainfrom
add-manifest-generator-script

Conversation

@joshuamschmidt

Copy link
Copy Markdown
Contributor

Purpose

Add scripts/generate_genotyping_manifest.py. It builds the genotyping-array manifest CSV for a cohort, uploads it to gs://cpg-<project>/gtc_genotyping_array/manifests/, and registers it as a manifest analysis in Metamist. In production the transfer pipeline creates this manifest; this script does the same for hand-made cohorts, so any test cohort can be run through phase 1.

Proposed Changes

  • Query the cohort's sequencing groups and build one row per SG from its meta (sentrix_barcode_a, sentrix_position_a, sample_plate). Same column layout as the production manifests.
  • Check every derived GTC path exists in GCS before writing anything.
  • Fail on a missing cohort, a non-genotypingarray SG, missing meta, a duplicate sentrix well, or a missing GTC.
  • Refuse to overwrite an existing manifest without --force. --dry-run prints the CSV without uploading or registering.

Testing

  • Dry-run against COH13479 (ourdna-test) and compared with the production manifest: identical values for every field the pipeline parser reads. Fields production left blank are filled from SG meta.
  • Dry-run against COH13461 (ourdna-test): 95 rows, all GTC paths verified.

@Alexander-Stuckey Alexander-Stuckey 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.

Looks good to me

@joshuamschmidt
joshuamschmidt merged commit f45cb38 into main Aug 17, 2026
5 checks passed
@joshuamschmidt
joshuamschmidt deleted the add-manifest-generator-script branch August 17, 2026 21:40
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