Skip to content

feat: Cytiva Biacore T200 - Refactor to use JsonData - #1095

Merged
nathan-stender merged 11 commits into
Benchling-Open-Source:mainfrom
felipenarv:felipenarv/cytiva_biacore_t200_refactor_to_JsonData
Oct 24, 2025
Merged

feat: Cytiva Biacore T200 - Refactor to use JsonData#1095
nathan-stender merged 11 commits into
Benchling-Open-Source:mainfrom
felipenarv:felipenarv/cytiva_biacore_t200_refactor_to_JsonData

Conversation

@felipenarv

Copy link
Copy Markdown
Contributor

Basically replaced DictType with JsonData, which keeps track of which keys have been called and create a warning when the object is deleted without having read all keys

@felipenarv
felipenarv requested review from a team and slopez-b as code owners October 2, 2025 16:51
@felipenarv
felipenarv requested a review from ajcariaga16 October 2, 2025 16:51
Comment thread src/allotropy/parsers/utils/tracked_dict.py Outdated
@felipenarv
felipenarv requested a review from slopez-b October 14, 2025 16:58

@joshua-benchling joshua-benchling left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good implementation for wrapping up nested dicts. Left optional comments

Comment thread src/allotropy/parsers/utils/dict_data.py Outdated
nathan-stender
nathan-stender previously approved these changes Oct 24, 2025
@nathan-stender
nathan-stender merged commit 24f3b11 into Benchling-Open-Source:main Oct 24, 2025
7 checks passed
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.

5 participants