Skip to content
This repository has been archived by the owner on Sep 2, 2024. It is now read-only.

#1227 add edge logging #1238

Merged
merged 1 commit into from
Mar 8, 2024
Merged

#1227 add edge logging #1238

merged 1 commit into from
Mar 8, 2024

Conversation

d-perl
Copy link
Contributor

@d-perl d-perl commented Mar 8, 2024

Adds some logs to help us diagnose edge detection issues

@d-perl d-perl force-pushed the 1227_add_edge_logging branch from a0c4f78 to 9817425 Compare March 8, 2024 11:43
DominicOram
DominicOram approved these changes Mar 8, 2024
View reviewed changes
Copy link
Collaborator

@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.

Thanks, this does make the logs a bit dense. I wonder if we want to remove one of the sets of logs in the long run but this is good for now.

@DominicOram DominicOram merged commit 5cc4de0 into main Mar 8, 2024
6 checks passed
@DominicOram DominicOram deleted the 1227_add_edge_logging branch March 8, 2024 17:38
olliesilvester pushed a commit to olliesilvester/mx-bluesky that referenced this pull request Aug 23, 2024
@DominicOram
Merge pull request DiamondLightSource/hyperion#1238 from DiamondLight…
d41cb3b 
…Source/1227_add_edge_logging

DiamondLightSource/hyperion#1227 add edge logging
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@DominicOram DominicOram DominicOram approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@d-perl @DominicOram