Skip to content

Add vwr troughplate as trough#1003

Closed
har1eyk wants to merge 10 commits into
PyLabRobot:mainfrom
har1eyk:add-vwr-troughplate-as-trough
Closed

Add vwr troughplate as trough#1003
har1eyk wants to merge 10 commits into
PyLabRobot:mainfrom
har1eyk:add-vwr-troughplate-as-trough

Conversation

@har1eyk

@har1eyk har1eyk commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

The VWR_1_troughplate_195000uL_Ub was defined as a troughplate. This PR adds it as a trough.

@rickwierenga

Copy link
Copy Markdown
Member

thanks for the PR!

what is the benefit of having it as a trough? we have traditionally modeled "1 well plates" as Plates with a single well and just named "troughplate". happy to be convinced for a new design

@har1eyk

har1eyk commented Apr 22, 2026

Copy link
Copy Markdown
Contributor Author

I want to complete more testing on this branch.

@har1eyk har1eyk closed this Apr 22, 2026
@har1eyk

har1eyk commented Apr 24, 2026

Copy link
Copy Markdown
Contributor Author

thanks for the PR!

what is the benefit of having it as a trough? we have traditionally modeled "1 well plates" as Plates with a single well and just named "troughplate". happy to be convinced for a new design

I need to aspirate from a trough. My legacy firmware (2009) will not accept a troughplate.

This produces error:

  await lh.aspirate(
            hiOrange_trough["A1"]*8,
            vols=[500]*8, 
            use_channels=ALL_CHANNELS,
            spread="tight",
            liquid_height=[4]*8,           
            )

ValueError: Resource is too small to space channels.

@har1eyk

har1eyk commented Apr 24, 2026

Copy link
Copy Markdown
Contributor Author

This PR was re-evaluated and found to be needed for legacy-firmware Starlets to aspirate from troughs.

@har1eyk

har1eyk commented Apr 24, 2026

Copy link
Copy Markdown
Contributor Author

Reopening is disabled because this branch was force-pushed/recreated. I opened a replacement PR here: #1008

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.

2 participants