Skip to content

fix: Molecular Devices SoftMax Pro - handle reduced data blocks with empty rows correctly - #1110

Merged
nathan-stender merged 1 commit into
mainfrom
nstender-smp-reduced-empty-rows
Oct 28, 2025
Merged

fix: Molecular Devices SoftMax Pro - handle reduced data blocks with empty rows correctly#1110
nathan-stender merged 1 commit into
mainfrom
nstender-smp-reduced-empty-rows

Conversation

@nathan-stender

@nathan-stender nathan-stender commented Oct 27, 2025

Copy link
Copy Markdown
Collaborator

Raw data blocks are read using plate dimensions, but reduced data blocks were being parsed with pop_csv_block_as_df which ends at an empty line.

This does not always work for reduced blocks either, which may have empty rows (row a row of wells with no measurements). So use plate dimensions for reduced blocks as well.

The function to read using plate dimensions is factored out to allow for code reuse.

@nathan-stender
nathan-stender requested review from a team and slopez-b as code owners October 27, 2025 21:40
@nathan-stender nathan-stender changed the title fix: Molecular Devices SoftMax Pro: handle reduced data blocks with empty rows correctly fix: Molecular Devices SoftMax Pro - handle reduced data blocks with empty rows correctly Oct 27, 2025
@nathan-stender
nathan-stender merged commit 18bc216 into main Oct 28, 2025
22 checks passed
@nathan-stender
nathan-stender deleted the nstender-smp-reduced-empty-rows branch October 28, 2025 19:08
nathan-stender added a commit that referenced this pull request Oct 28, 2025
### Added

- Thermo Fisher Scientific VISIONlite - add get unread data (#1097)
- Flowjo - Added new example (#1108)
- Beckman Coulter Biomek - clear unread data warnings (#1092)
- Cytiva Biacore T200 - Refactor to use JsonData (#1095)
- CTL ImmunoSpot - use proper units in data processing document (#1098)
- Thermo Fisher Scientific SkanIt - add unread data (#1104)

### Fixed

- Molecular Devices SoftMax Pro - handle reduced data blocks with empty
rows correctly (#1110)
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