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): refresh the current position in capacitive probe #11608

Merged
merged 1 commit into from
Oct 24, 2022

Conversation

Laura-Danielle
Copy link
Contributor

We need to refresh the current position to ensure that we have the correct current position before moving to safe z height in the capacitive probe.

We need to refresh the current position to ensure that we have the correct current position before
moving to safe z height in the capacitive probe.
@Laura-Danielle Laura-Danielle requested a review from a team as a code owner October 21, 2022 20:17
@codecov
Copy link

codecov bot commented Oct 21, 2022

Codecov Report

Merging #11608 (91132a6) into edge (8d6519b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             edge   #11608   +/-   ##
=======================================
  Coverage   75.01%   75.01%           
=======================================
  Files        2056     2056           
  Lines       57435    57435           
  Branches     5979     5979           
=======================================
  Hits        43082    43082           
  Misses      12949    12949           
  Partials     1404     1404           
Flag Coverage Δ
notify-server 88.26% <ø> (ø)

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

Impacted Files Coverage Δ
api/src/opentrons/hardware_control/ot3api.py 80.66% <ø> (ø)

@Laura-Danielle Laura-Danielle merged commit 6d39339 into edge Oct 24, 2022
@Laura-Danielle Laura-Danielle deleted the refresh_position_in_capacitive_probe branch October 24, 2022 21:07
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