Skip to content

Wait for OAV to conntect and read zoom level as string#246

Merged
DominicOram merged 3 commits intomainfrom
mx_i24-test-oav-fix
Nov 28, 2023
Merged

Wait for OAV to conntect and read zoom level as string#246
DominicOram merged 3 commits intomainfrom
mx_i24-test-oav-fix

Conversation

@noemifrisina
Copy link
Collaborator

@noemifrisina noemifrisina commented Nov 28, 2023

Wait for OAV to connect and subscribe to the zoom level once connected.

Instructions to reviewer on how to test:

  1. Run tests

Checks for reviewer

  • Would the PR title make sense to a scientist on a set of release notes

@DominicOram DominicOram added the needed_for_release Issues that must be complete before the next release label Nov 28, 2023
@noemifrisina noemifrisina marked this pull request as ready for review November 28, 2023 18:17
@codecov
Copy link

codecov bot commented Nov 28, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (93145a9) 88.57% compared to head (5a08f07) 88.58%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #246      +/-   ##
==========================================
+ Coverage   88.57%   88.58%   +0.01%     
==========================================
  Files          73       73              
  Lines        2669     2672       +3     
==========================================
+ Hits         2364     2367       +3     
  Misses        305      305              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@DominicOram DominicOram left a comment

Choose a reason for hiding this comment

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

Great, thank you!

@DominicOram DominicOram merged commit af16a04 into main Nov 28, 2023
@DominicOram DominicOram deleted the mx_i24-test-oav-fix branch November 28, 2023 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needed_for_release Issues that must be complete before the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants