-
Notifications
You must be signed in to change notification settings - Fork 5
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1084 +/- ##
=======================================
Coverage 93.35% 93.35%
=======================================
Files 62 62
Lines 2919 2921 +2
=======================================
+ Hits 2725 2727 +2
Misses 194 194 ☔ 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.
Great thanks, just one comment
yield from bps.stage( | ||
fgs_composite.zocalo | ||
) # connect to zocalo and make sure the queue is clear |
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: I'm a bit worried that we're not explicitly waiting on the queue being cleared. If we run this without specifying wait
I think the behaviour looks to be that it will just not wait so we end up assuming the 2s queue clear is less than the first zocalo message we actually want. Probably a reasonable assumption but I would prefer if we put this in a group that we do wait on at some point, not sure when we would though?
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, thanks @dperl-dls!
…Source/dodal_298 add unstage for zocalo
Required for DiamondLightSource/dodal#300
To test: