Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(api): disengage Z motors during 96-channel pipettes bracket detach/attach flows #14999

Conversation

ahiuchingau
Copy link
Contributor

@ahiuchingau ahiuchingau commented Apr 24, 2024

Overview

This PR makes sure we are engaging ebrakes when we're attaching/detaching the 96-channel pipette and mount bracket.
On top of that, we can now use retract instead of using a movement Point just to move the z up.
Fixes RQA-2631.

@ahiuchingau ahiuchingau requested a review from a team as a code owner April 24, 2024 19:24
Copy link
Member

@sfoster1 sfoster1 left a comment

Choose a reason for hiding this comment

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

Nice!

@ahiuchingau ahiuchingau merged commit 0eb749f into edge Apr 29, 2024
20 checks passed
@ahiuchingau ahiuchingau deleted the RQA-2631-error-2003-stall-or-collision-detected-96-channel-detach-step-1 branch April 29, 2024 21:47
Carlos-fernandez pushed a commit that referenced this pull request May 20, 2024
…h/attach flows (#14999)

<!--
Thanks for taking the time to open a pull request! Please make sure
you've read the "Opening Pull Requests" section of our Contributing
Guide:


https://github.com/Opentrons/opentrons/blob/edge/CONTRIBUTING.md#opening-pull-requests

To ensure your code is reviewed quickly and thoroughly, please fill out
the sections below to the best of your ability!
-->

# Overview
This PR makes sure we are engaging ebrakes when we're
attaching/detaching the 96-channel pipette and mount bracket.
On top of that, we can now use retract instead of using a movement Point
just to move the z up.
Fixes [RQA-2631](https://opentrons.atlassian.net/browse/RQA-2631).


[RQA-2631]:
https://opentrons.atlassian.net/browse/RQA-2631?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
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