Skip to content

feat: Cytiva Biacore Insight - initial implementation#1066

Merged
nathan-stender merged 26 commits into
mainfrom
slopez/cytiva-biacore-insight
Sep 12, 2025
Merged

feat: Cytiva Biacore Insight - initial implementation#1066
nathan-stender merged 26 commits into
mainfrom
slopez/cytiva-biacore-insight

Conversation

@slopez-b

Copy link
Copy Markdown
Collaborator

No description provided.

@slopez-b slopez-b changed the title Slopez/cytiva biacore insight feat: Cytiva Biacore Insight Aug 30, 2025
@slopez-b slopez-b changed the title feat: Cytiva Biacore Insight feat: Cytiva Biacore Insight - initial implementation Aug 30, 2025
@slopez-b
slopez-b marked this pull request as ready for review September 2, 2025 16:14
@slopez-b
slopez-b requested a review from a team as a code owner September 2, 2025 16:14
	# Please enter a commit message to explain why this merge is necessary,
	# especially if it merges an updated upstream into a topic branch.
	#
	# Lines starting with '#' will be ignored, and an empty message aborts
	# the commit.
james-leinas
james-leinas previously approved these changes Sep 8, 2025

@james-leinas james-leinas 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.

ASM Approval 🎉

@nathan-stender nathan-stender 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.

Code LGTM

One change in requirements though after discussing with @james-leinas - requesting that we use UUID for report point identifiers to ensure uniqueness.

After that change - should be good to merge!

@nathan-stender
nathan-stender merged commit eb16cee into main Sep 12, 2025
7 checks passed
@nathan-stender
nathan-stender deleted the slopez/cytiva-biacore-insight branch September 12, 2025 19:18
nathan-stender added a commit that referenced this pull request Sep 30, 2025
### Added

- AppBio QuantStudio Design & Analysis - remove duplicates and hardcoded
values from measurement custom info doc (#1088)
- Tecan Magellan - remove unread keys warning (#1086)
- Beckman Coulter Biomek - Add handle for unified transfer and pipetting
samples (#1085)
- Agilent OpenLab CDS - add peak area percentage and peak height (#1083)
- Roche Cedex Bioht - use get_unread (#1084)
- Chemometec NC View - Migrated to use get unread (#1082)
- Migrate beckman_pharmspec to use SeriesData.get_unread (#1081)
- Beckman Coulter Biomek Liquid Handler - add support for log headerless
files (#1078)
- Migrate ctl_immunospot to use SeriesData.get_unread (#1076)
- Cytiva Biacore Insight/Cytiva Biacore T200 Control - Update Slope
calculated data's reported units to RU/s (#1077)
- Molecular Devices SoftMax Pro - Add unread data (#1071)
- Perkin elmer envision get unread (#1073)
- BMG mars - Migrate to use get_unread info (#1074)
- Unchained Labs Lunnatic & Stunner - refactor measurements to use data
cubes for wavelength spectrums (#1062)
- Cytiva Biacore Insight - initial implementation (#1066)
- NovaBio Flex2 - use SeriesData.get_unread (#1070)
- Migrate revvity_kaleido to use SeriesData.get_unread (#1060)

### Fixed

- Binding Affinity Analyzer - add support for special keys in processed
data document (#1087)
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.

3 participants