Skip to content

Add vwr troughplate as trough#1003

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

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

Conversation

@har1eyk
Copy link
Copy Markdown
Contributor

@har1eyk har1eyk commented Apr 20, 2026

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
Copy link
Copy Markdown
Contributor Author

har1eyk commented Apr 22, 2026

I want to complete more testing on this branch.

@har1eyk har1eyk closed this Apr 22, 2026
@har1eyk
Copy link
Copy Markdown
Contributor Author

har1eyk commented Apr 24, 2026

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
Copy link
Copy Markdown
Contributor Author

har1eyk commented Apr 24, 2026

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

@har1eyk
Copy link
Copy Markdown
Contributor Author

har1eyk commented Apr 24, 2026

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