Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(shared-data, api): add uiMaxFlowRate key to pipette definitions #14859

Merged
merged 16 commits into from
Apr 17, 2024
+207 −43
Merged

feat(shared-data, api): add uiMaxFlowRate key to pipette definitions #14859

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
8555c2e
feat(shared-data): create getMaxFlowRateByVolume util
jerader Apr 10, 2024
5cf7c57
update the pipette definitions with max flow rate numbers
jerader Apr 10, 2024
d83baf8
maxFlowRate volumes to match pipette tip volumes
jerader Apr 11, 2024
717cd14
add missing maxFlowRates for a few volumes in p1000 3.5 single channe…
jerader Apr 11, 2024
0cd9f8b
add maxFlowRate to the liquid properties schema
jerader Apr 11, 2024
4a99416
update defs to only have 1 maxFlowRate per supported tip and start wr…
jerader Apr 12, 2024
8fe83e6
fix test_max_flow_rates_per_volume
jerader Apr 12, 2024
a526a63
fix js test, make maxFlowRate optional
jerader Apr 15, 2024
9693f27
reformat file
jerader Apr 15, 2024
0a69042
rename maxFlowRate to uiMaxFlowRate, adjust to only have 2% safety
jerader Apr 15, 2024
e2506bd
fix pipettes js test
jerader Apr 15, 2024
b3753e6
Merge branch 'edge' into shared-data_create-max-flowrate-util
jerader Apr 15, 2024
c360d34
add lowVolumeDefault comments to pipette definition and liquid schema
jerader Apr 16, 2024
d01741e
use only shared-data ul_per_mm util in hardware_control
jerader Apr 16, 2024
992bb59
revert wrong edge change to 3_6 def
jerader Apr 16, 2024
3e19949
fix js type typo
jerader Apr 16, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
145 changes: 145 additions & 0 deletions shared-data/js/__tests__/pipettes.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import {
getPipetteSpecsV2,
getPipetteNameSpecs,
getPipetteModelSpecs,
getMaxFlowRateByVolume,
} from '../pipettes'
import type { PipetteV2LiquidSpecs, PipetteV2Specs } from '../types'

Expand Down Expand Up @@ -158,6 +159,53 @@ describe('pipette data accessors', () => {
minVolume: 5,
supportedTips: {
t50: {
maxFlowRate: {
'6': 48.7,
'7': 49.3,
'8': 49.7,
'9': 50,
'10': 50.3,
'11': 50.5,
'12': 50.7,
'13': 50.8,
'14': 50.9,
'15': 51,
'16': 51.1,
'17': 51.1,
'18': 51.2,
'19': 51.2,
'20': 51.3,
'21': 51.3,
'22': 51.4,
'23': 51.4,
'24': 51.5,
'25': 51.6,
'26': 51.6,
'27': 51.7,
'28': 51.7,
'29': 51.7,
'30': 51.7,
'31': 51.7,
'32': 51.7,
'33': 51.8,
'34': 51.8,
'35': 51.8,
'36': 51.8,
'37': 51.8,
'38': 51.8,
'39': 51.8,
'40': 51.9,
'41': 51.9,
'42': 51.9,
'43': 51.9,
'44': 51.9,
'45': 51.9,
'46': 51.9,
'47': 52,
'48': 52,
'49': 52,
'50': 52,
},
aspirate: {
default: {
1: expect.anything(),
Expand Down Expand Up @@ -205,6 +253,38 @@ describe('pipette data accessors', () => {
minVolume: 1,
supportedTips: {
t50: {
maxFlowRate: {
'1': 32.8,
'2': 41.5,
'3': 42.6,
'4': 45.5,
'5': 48,
'6': 48.7,
'7': 49.3,
'8': 49.7,
'9': 50,
'10': 50.3,
'11': 50.5,
'12': 50.7,
'13': 50.8,
'14': 50.9,
'15': 51,
'16': 51.1,
'17': 51.1,
'18': 51.2,
'19': 51.2,
'20': 51.3,
'21': 51.3,
'22': 51.4,
'23': 51.4,
'24': 51.5,
'25': 51.6,
'26': 51.6,
'27': 51.7,
'28': 51.7,
'29': 51.7,
'30': 51.7,
},
aspirate: {
default: {
1: expect.anything(),
Expand Down Expand Up @@ -248,4 +328,69 @@ describe('pipette data accessors', () => {
mockLiquids
)
})

describe('getMaxFlowRateByVolume', () => {
const mockMaxFlowRate: Record<string, number> = {
jerader marked this conversation as resolved.
Show resolved Hide resolved
'6': 48.4,
'7': 48.9,
'8': 49.3,
'9': 49.6,
'10': 49.9,
'11': 50.1,
'12': 50.2,
'13': 50.4,
'14': 50.5,
'15': 50.7,
'16': 50.8,
'17': 51,
'18': 51,
'19': 51.1,
'20': 51.2,
'21': 51.3,
'22': 51.3,
'23': 51.4,
'24': 51.4,
'25': 51.5,
'26': 51.5,
'27': 51.6,
'28': 51.6,
'29': 51.7,
'30': 51.7,
'31': 51.8,
'32': 51.8,
'33': 51.8,
'34': 52,
'35': 52,
'36': 52,
'37': 52.1,
'38': 52.1,
'39': 52.1,
'40': 52.1,
'41': 52.2,
'42': 52.2,
'43': 52.1,
'44': 52.1,
'45': 52.1,
'46': 52.1,
'47': 52.2,
'48': 52.2,
'49': 52.2,
'50': 52.2,
}

for (let i = 1; i <= 50; i++) {
it(`renders a max flow rate for ${i}uL for p50_single_v3.3 aspirate`, () => {
const volume = i.toString()
expect(getMaxFlowRateByVolume(mockMaxFlowRate, i)).toEqual(
mockMaxFlowRate[volume]
)
})
}
it('returns 0 if max flow rate is undefined', () => {
expect(getMaxFlowRateByVolume(undefined, 5)).toEqual(0)
})
it('returns 0 if volume is 0', () => {
expect(getMaxFlowRateByVolume({} as any, 0)).toEqual(0)
})
})
})
17 changes: 17 additions & 0 deletions shared-data/js/pipettes.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -235,3 +235,20 @@ export const getPipetteSpecsV2 = (

return pipetteV2Specs
}

/* returns the maximum flow rate from a specific volume,
and the maxFlowRate key
**/
export const getMaxFlowRateByVolume = (
maxFlowRate?: Record<string, number>,
volume?: number
): number => {
if (
volume == null ||
(volume != null && volume === 0) ||
maxFlowRate == null
) {
return 0
}
return maxFlowRate[volume]
}
1 change: 1 addition & 0 deletions shared-data/js/types.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -486,6 +486,7 @@ export interface SupportedTip {
}
defaultReturnTipHeight?: number
defaultFlowAcceleration?: number
maxFlowRate?: Record<string, number>
}

export interface SupportedTips {
Expand Down
Loading
Loading