feat(hardware): add SN support for the P1KP pipette

Opentrons · Nov 20, 2024 · 908df40 · 908df40
1 parent 54a4017
commit 908df40
Show file tree

Hide file tree

Showing 7 changed files with 14 additions and 3 deletions.
diff --git a/api/src/opentrons/hardware_control/backends/ot3controller.py b/api/src/opentrons/hardware_control/backends/ot3controller.py
@@ -1008,6 +1008,7 @@ def _lookup_serial_key(pipette_name: FirmwarePipetteName) -> str:
         lookup_name = {
             FirmwarePipetteName.p1000_single: "P1KS",
             FirmwarePipetteName.p1000_multi: "P1KM",
+            FirmwarePipetteName.p1000_multi_emulsify: "P1KP",
             FirmwarePipetteName.p50_single: "P50S",
             FirmwarePipetteName.p50_multi: "P50M",
             FirmwarePipetteName.p1000_96: "P1KH",

diff --git a/hardware/opentrons_hardware/firmware_bindings/constants.py b/hardware/opentrons_hardware/firmware_bindings/constants.py
@@ -359,6 +359,7 @@ class PipetteName(int, Enum):
     p1000_96 = 0x04
     p50_96 = 0x05
     p200_96 = 0x06
+    p1000_multi_emulsify = 0x07
     unknown = 0xFFFF
 
 

diff --git a/hardware/opentrons_hardware/instruments/pipettes/serials.py b/hardware/opentrons_hardware/instruments/pipettes/serials.py
@@ -27,6 +27,7 @@
 NAME_LOOKUP: Dict[str, PipetteName] = {
     "P1KS": PipetteName.p1000_single,
     "P1KM": PipetteName.p1000_multi,
+    "P1KP": PipetteName.p1000_multi_emulsify,
     "P50S": PipetteName.p50_single,
     "P50M": PipetteName.p50_multi,
     "P1KH": PipetteName.p1000_96,

diff --git a/hardware/tests/opentrons_hardware/instruments/test_serials.py b/hardware/tests/opentrons_hardware/instruments/test_serials.py
@@ -40,6 +40,12 @@
             1,
             b"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00",
         ),
+        (
+            "P1KPV30",
+            PipetteName.p1000_multi_emulsify,
+            30,
+            b"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00",
+        ),
     ],
 )
 def test_scan_valid_pipette_serials(

diff --git a/shared-data/js/constants.ts b/shared-data/js/constants.ts
@@ -144,7 +144,7 @@ export const OT3_PIPETTES = [
   'p20_single_flex',
   'p50_single_flex',
   'p50_multi_flex',
-  'p1000_multi_flex',
+  'p1000_multi_emulsify_flex',
   'p1000_96',
   'p200_96',
 ]

diff --git a/shared-data/python/opentrons_shared_data/pipette/dev_types.py b/shared-data/python/opentrons_shared_data/pipette/dev_types.py
@@ -31,6 +31,7 @@
     "p1000_single_gen2",
     "p1000_single_flex",
     "p1000_multi_flex",
+    "p1000_multi_emulsify_flex",
     "p1000_96",
     "p200_96",
 ]
@@ -57,6 +58,7 @@ class PipetteNameType(str, Enum):
     P1000_SINGLE_GEN2 = "p1000_single_gen2"
     P1000_SINGLE_FLEX = "p1000_single_flex"
     P1000_MULTI_FLEX = "p1000_multi_flex"
+    P1000_MULTI_EMULSIFY_FLEX = "p1000_multi_emulsify_flex"
     P1000_96 = "p1000_96"
     P200_96 = "p200_96"
 

diff --git a/shared-data/python/opentrons_shared_data/pipette/types.py b/shared-data/python/opentrons_shared_data/pipette/types.py
@@ -216,7 +216,7 @@ def dict_for_encode(self) -> bool:
     "p1000_single_gen2",
     "p1000_single_flex",
     "p1000_multi_flex",
-    "p1000_multi_emulsify",
+    "p1000_multi_emulsify_flex",
     "p1000_96",
     "p200_96",
 ]
@@ -243,7 +243,7 @@ class PipetteNameType(str, enum.Enum):
     P1000_SINGLE_GEN2 = "p1000_single_gen2"
     P1000_SINGLE_FLEX = "p1000_single_flex"
     P1000_MULTI_FLEX = "p1000_multi_flex"
-    P1000_MULTI_EMULSIFY = "p1000_multi_emulsify"
+    P1000_MULTI_EMULSIFY_FLEX = "p1000_multi_emulsify_flex"
     P1000_96 = "p1000_96"
     P200_96 = "p200_96"