Skip to content

fix: Tecan Magellan - support compact Magellan Pro 7.5 exports - #1256

Closed
tamargrey wants to merge 2 commits into
mainfrom
tamar-fix-magellan-pro
Closed

fix: Tecan Magellan - support compact Magellan Pro 7.5 exports#1256
tamargrey wants to merge 2 commits into
mainfrom
tamar-fix-magellan-pro

Conversation

@tamargrey

Copy link
Copy Markdown
Collaborator

Summary

Magellan Pro v7.5 can produce a compact measurement metadata block that omits the Unit:, Number of flashes, and trailing Meas. temperature fields expected by the Tecan Magellan parser. The parser consequently consumed the remaining metadata and raised Expected non-null value.

This change:

  • Supports the compact v7.5 metadata layout without changing the existing parsing path.
  • Infers Absorbance mode from the measurement section.
  • Maps the measurement label and wavelength normally.
  • Recovers measurement temperature from the table’s pre-data temperature row.
  • Uses the bracketed plate description as the plate identifier.
  • Omits number of averages when it is unavailable.
  • Preserves plate description, range, and action as custom information.
  • Adds a sanitized regression workbook and expected ASM output.

Testing

  • Confirmed the original customer workbook converts all 96 wells successfully.
  • Ran hatch run test tests/parsers/tecan_magellan/to_allotrope_test.py.
  • Ran hatch run lint.

@tamargrey
tamargrey requested review from a team and slopez-b as code owners August 25, 2026 14:45
@tamargrey

Copy link
Copy Markdown
Collaborator Author

Closing in favor of PR with verified commits #1258

@tamargrey tamargrey closed this Aug 25, 2026
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