-
Notifications
You must be signed in to change notification settings - Fork 5
Correctly read pin tip location using ophyd #1143
Conversation
tip_x_y_px = yield from bps.rd(pin_tip) | ||
if isinstance(pin_tip, PinTipDetection): | ||
tip_x_y_px = yield from bps.rd(pin_tip) | ||
LOGGER.info("Pin tip not found, waiting a second and trying again") |
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.
Should: This logging message should be in the below if statement
@@ -266,6 +266,13 @@ def flux(): | |||
return i03.flux(fake_with_ophyd_sim=True) | |||
|
|||
|
|||
@pytest.fixture | |||
def ophyd_pin_tip_detection(): | |||
RunEngine() # A RE is needed to start the bluesky loop |
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.
Should: Use the RE from this conftest
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.
Looks good thanks. I'll just move that logging statement
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1143 +/- ##
==========================================
- Coverage 93.09% 92.83% -0.26%
==========================================
Files 69 65 -4
Lines 3419 3351 -68
==========================================
- Hits 3183 3111 -72
- Misses 236 240 +4 ☔ View full report in Codecov by Sentry. |
…Source/1125_ophyd_pin_tip_hotfixes Correctly read pin tip location using ophyd
Fixes #1125
Link to dodal PR (if required): DiamondLightSource/dodal#328
Note, will be tidied up in #1069
To test: