-
Notifications
You must be signed in to change notification settings - Fork 2
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
Use ophyd_async OAV #594
Use ophyd_async OAV #594
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #594 +/- ##
==========================================
+ Coverage 78.65% 78.67% +0.02%
==========================================
Files 97 97
Lines 6844 6846 +2
==========================================
+ Hits 5383 5386 +3
+ Misses 1461 1460 -1
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thank you!
assert oav.parameters.micronsPerXPixel is not None | ||
tip_offset_px = int(tip_offset_microns / oav.parameters.micronsPerXPixel) | ||
microns_per_pixel_x = yield from bps.rd(oav.microns_per_pixel_x) | ||
assert microns_per_pixel_x is not None |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: I don't think it can be None
so I would remove this check
Needs Dodal#857