-
Notifications
You must be signed in to change notification settings - Fork 8
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
Make the PinTipDetection device also store top and bottom edges #358
Make the PinTipDetection device also store top and bottom edges #358
Conversation
6f0c355
to
c193fc1
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #358 +/- ##
==========================================
+ Coverage 93.28% 93.30% +0.01%
==========================================
Files 85 85
Lines 3290 3299 +9
==========================================
+ Hits 3069 3078 +9
Misses 221 221 ☔ 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, I think we need to update the tests here to confirm that the trigger and read is also giving back top/bottom arrays though.
async def _set_edges(self, values: tuple[NDArray, NDArray]): | ||
await self.triggered_top_edge._backend.put(values[0]) | ||
await self.triggered_bottom_edge._backend.put(values[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.
Could: Feels like we can just have one _set_triggered_values
function that takes a SampleLocation
rather than this split
….com:DiamondLightSource/dodal into hyperion_1068_1213_ophyd_oav_for_grid_detect
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!
Needed for hyperion DiamondLightSource/hyperion#1213 DiamondLightSource/hyperion#1068
Instructions to reviewer on how to test:
Checks for reviewer