-
Notifications
You must be signed in to change notification settings - Fork 5
Conversation
Tested on beamline today. Broadly works but need to:
|
Handed over to @dperl-dls . |
I'll pick this up now so that @dperl-dls can focus on rotation scan stuff |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #938 +/- ##
==========================================
+ Coverage 93.35% 93.80% +0.44%
==========================================
Files 62 58 -4
Lines 2921 2759 -162
==========================================
- Hits 2727 2588 -139
+ Misses 194 171 -23 ☔ View full report in Codecov by Sentry. |
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 for tidying this up
|
||
LOGGER.info("Pin tip not found, waiting a second and trying again") | ||
|
||
if found_tip == ophyd_pin_tip_detection.INVALID_POSITION: |
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.
Do I understand correctly that this will be adressed in #1069?
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.
Yep
23a5dca
to
aadbfa5
Compare
…ource/allow_using_new_pin_tip_centring Ophyd pin-tip centring
Requires DiamondLightSource/dodal#289
Provide support for running both Ophyd + AD MXSC pin tip centring workflows in hyperion.
To test: