Skip to content

feat(monorepo): extend mainifold compatibility to accommodate direct wellplate#21639

Merged
ncdiehl11 merged 11 commits into
edgefrom
pd_extend-manifold-lw
Jun 12, 2026
Merged

feat(monorepo): extend mainifold compatibility to accommodate direct wellplate#21639
ncdiehl11 merged 11 commits into
edgefrom
pd_extend-manifold-lw

Conversation

@ncdiehl11

@ncdiehl11 ncdiehl11 commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Overview

This PR extends the logic to accommodate the full range of possible labware and adapter combinations on both the main manifold area and dock.

Also, I add 5 new labware definitions for filter plates to be used in the system. NOTE: these definitions are defined to physical spec, but their display names, loadnames, and volumes are subject to change.

Closes EXEC-2404

Test Plan and Hands on Testing

vacuum_labware_test.py

PD starting deck — valid labware matrix

Area Currently loaded Allowed to add
Main module Empty Collar (short/tall), Spacer (short/tall), Collection plate
Main module Spacer Collar, Collection plate
Main module Spacer + Collection plate Collar, Filter plate
Main module Spacer + Collar Collection plate, Filter plate
Main module Spacer + Collar + Collection plate Filter plate
Main module Collar Collection plate, Filter plate
Main module Collar + Collection plate Filter plate
Main module Collection plate Filter plate
Dock Empty Collar (short/tall)
Dock Collar Filter plate
Dock Collar + Filter plate (full — nothing)
  • custom SVGs for collars/spacers align correctly with the module and children labware
  • collars/spacers always render despite possibly not being the top-most element in a stack on a module
  • exported Python load statements for adapters on the vacuum module main area and dock match API spec
  • new filter plate logic is implemented such that they can be moved to arbitrary exposed labware (non-tiprack, non-lid)

Changelog

List changes introduced by this PR considering future developers and the end user. Give careful thought and clear documentation to breaking changes.

Review requests

  • smoke test according to the above labware compatibility matrix in PD
  • export a protocol that loads some labware on the vacuum module and verify that analysis passes successfully

Risk assessment

medium. touches a lot of core PD deck setup files, but generally branches on vacuum module special logic so as to not disrupt existing logic for non-vacuum labware loads

ncdiehl11 added 2 commits June 2, 2026 10:59
…e direct wellplate

This PR extends the manifold to accommodate a collection plate directly (valid combination, bottom
up: collectin plate, filter plate, collar)
@codecov

codecov Bot commented Jun 2, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 13.04348% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 21.62%. Comparing base (c22bcad) to head (7189c7f).

Files with missing lines Patch % Lines
.../src/pages/Designer/DeckSetup/DeckSetupToolbox.tsx 0.00% 10 Missing ⚠️
.../components/organisms/SelectLabwareModal/index.tsx 42.85% 4 Missing ⚠️
...col-designer/src/pages/Designer/DeckSetup/utils.ts 0.00% 4 Missing ⚠️
...rc/pages/Designer/DeckSetup/DeckSetupContainer.tsx 0.00% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             edge   #21639   +/-   ##
=======================================
  Coverage   21.62%   21.62%           
=======================================
  Files        3907     3907           
  Lines      320262   320276   +14     
  Branches    57770    57779    +9     
=======================================
+ Hits        69246    69272   +26     
+ Misses     248046   248034   -12     
  Partials     2970     2970           
Flag Coverage Δ
protocol-designer 39.52% <13.04%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...designer/src/pages/Designer/DeckSetup/constants.ts 10.00% <ø> (ø)
...l-designer/src/utils/labwareModuleCompatibility.ts 11.19% <ø> (ø)
...rc/pages/Designer/DeckSetup/DeckSetupContainer.tsx 17.03% <0.00%> (ø)
.../components/organisms/SelectLabwareModal/index.tsx 31.81% <42.85%> (+6.96%) ⬆️
...col-designer/src/pages/Designer/DeckSetup/utils.ts 16.52% <0.00%> (ø)
.../src/pages/Designer/DeckSetup/DeckSetupToolbox.tsx 29.31% <0.00%> (-0.66%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ncdiehl11 ncdiehl11 self-assigned this Jun 11, 2026
@ncdiehl11 ncdiehl11 requested a review from jerader June 11, 2026 19:13
… are transparent in the middle; update vacuum hardware sim slightly
@ncdiehl11 ncdiehl11 requested a review from vegano1 June 11, 2026 19:31
@ncdiehl11 ncdiehl11 changed the title feat(protocol-designer): extend mainifold compatibility to accommodate direct wellplate feat(monorepo): extend mainifold compatibility to accommodate direct wellplate Jun 11, 2026
@ncdiehl11 ncdiehl11 marked this pull request as ready for review June 11, 2026 19:33
@ncdiehl11 ncdiehl11 requested review from a team as code owners June 11, 2026 19:33
@ncdiehl11 ncdiehl11 removed request for a team June 11, 2026 19:34
Comment on lines +54 to +58
(movingLabwareIsFilterPlate &&
!adapterIsLid &&
!adapterIsTiprack &&
!adapterIsFilterPlate) ||
(adapterIsVacuumSpacer && movingLabwareIsCollar) ||

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.

oof lol, i guess not much we can do to make it more readable though

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

yeah it's so gross but these are the rules for movement

@jerader jerader 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.

nice! left a few comments, all mild though

@vegano1 vegano1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Perfect, thank you!

@ncdiehl11 ncdiehl11 merged commit 50c2f19 into edge Jun 12, 2026
127 checks passed
@ncdiehl11 ncdiehl11 deleted the pd_extend-manifold-lw branch June 12, 2026 14:12
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