Skip to content

Skip extra qPCR files instead of failing them on reprocess - #219

Merged
wasimxyz merged 1 commit into
stagingfrom
cursor/skip-qpcr-sidecar-files
Sep 1, 2026
Merged

Skip extra qPCR files instead of failing them on reprocess#219
wasimxyz merged 1 commit into
stagingfrom
cursor/skip-qpcr-sidecar-files

Conversation

@wasimxyz

@wasimxyz wasimxyz commented Sep 1, 2026

Copy link
Copy Markdown
Member

Summary

  • Azure Cielo writes extra CSVs and a report PDF next to the melting-curve and Cq Values files we actually parse.
  • Reprocessing a run sent every raw file to Lambda. Files the processor did not claim were marked failed, so the run looked failed even when melting-curve processing worked.
  • Lambda now accepts every .csv and .pdf on qPCR. It still parses melting curves and Cq Values. Extra CSVs and the report PDF are marked completed without parsing, so they cannot fail the run.

Test plan

  • uv run pytest lambda/tests/azure_cielo_qpcr/test_process_file.py lambda/tests/test_processors.py
  • After deploy, reprocess a qPCR run that has extra CSVs and a report PDF
  • Amplification Values, Dye calibration, Post Processed Amp Values, and the report PDF show completed
  • Melting-curve processing still writes the derivatives CSV and plate JSON
  • Cq Values still writes dye-channel metadata
  • A broken Cq Values file still fails

Made with Cursor

…rocess.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
data-hub Ready Ready Preview Sep 1, 2026 11:09pm UTC

Request Review

@wasimxyz wasimxyz self-assigned this Sep 1, 2026
@wasimxyz
wasimxyz merged commit cbfab4c into staging Sep 1, 2026
6 checks passed
@wasimxyz
wasimxyz deleted the cursor/skip-qpcr-sidecar-files branch September 1, 2026 23:14
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.

1 participant