Skip to content

Conversation

@fderop
Copy link
Contributor

@fderop fderop commented Nov 3, 2025

No description provided.

@fderop fderop requested a review from rickwierenga November 3, 2025 23:19
Comment on lines 80 to 81
"size_x": diameter, # from spec
"size_y": diameter, # from spec
Copy link
Member

Choose a reason for hiding this comment

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

which spec? please link it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I came to the conclusion that the old definition was simply wrong. There is no Ub of that type, only Vb, and the linked spec sheet, PLR dimensions, and part number match the Vb. don't think depreciation func is necessary for this correction.

return liquid_height


def calculate_liquid_height_container_1segment_round_vbottom(
Copy link
Contributor Author

Choose a reason for hiding this comment

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

hard to believe that there was no existing v-bottom well calculator?

@fderop fderop requested a review from rickwierenga November 9, 2025 21:46
@rickwierenga rickwierenga merged commit b5ee6e1 into PyLabRobot:main Nov 10, 2025
9 checks passed
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