Skip to content

fix: BMG Mars: wrap keyError when index out of range#1005

Merged
nathan-stender merged 4 commits into
mainfrom
jherna-check-error-bmg-mars-for-index-1
Jun 10, 2025
Merged

fix: BMG Mars: wrap keyError when index out of range#1005
nathan-stender merged 4 commits into
mainfrom
jherna-check-error-bmg-mars-for-index-1

Conversation

@joshua-benchling

Copy link
Copy Markdown
Collaborator

No description provided.

@joshua-benchling
joshua-benchling requested review from a team and slopez-b as code owners June 4, 2025 18:04
@joshua-benchling joshua-benchling changed the title fix: BMG Mars: wrape keyError when index out of range fix: BMG Mars: wrap keyError when index out of range Jun 4, 2025
Comment thread src/allotropy/parsers/bmg_mars/bmg_mars_reader.py Outdated
@nathan-stender
nathan-stender merged commit 37c7e8a into main Jun 10, 2025
7 checks passed
@nathan-stender
nathan-stender deleted the jherna-check-error-bmg-mars-for-index-1 branch June 10, 2025 17:37
nathan-stender added a commit that referenced this pull request Jun 10, 2025
### Fixed

- Beckman Coulter PharmSpec - add validation for Particle and approver
label, add unittests (#1008)
- BMG Mars: wrap keyError when index out of range (#1005)
- ChemoMetec Nucleoview - wrap key error when indexing dataframe (#1006)
- MSD Methodical Mind - support wells larger than 192 wells correctly
(#1009)
joshua-benchling added a commit that referenced this pull request Sep 15, 2025
Co-authored-by: Nathan Stender <nathan.stender@benchling.com>
joshua-benchling pushed a commit that referenced this pull request Sep 15, 2025
### Fixed

- Beckman Coulter PharmSpec - add validation for Particle and approver
label, add unittests (#1008)
- BMG Mars: wrap keyError when index out of range (#1005)
- ChemoMetec Nucleoview - wrap key error when indexing dataframe (#1006)
- MSD Methodical Mind - support wells larger than 192 wells correctly
(#1009)
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