-
Notifications
You must be signed in to change notification settings - Fork 5
Tidy up device creation for full grid scan #661
Conversation
Codecov Report
@@ Coverage Diff @@
## main #661 +/- ##
==========================================
+ Coverage 89.04% 89.75% +0.71%
==========================================
Files 34 34
Lines 1488 1494 +6
==========================================
+ Hits 1325 1341 +16
+ Misses 163 153 -10
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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 for tidying this up! I just added a little sneaky change (making the get beamline params a bit more general, in prep for adding beamlines) and a couple more tests, so I'll approve this and leave for you to look at
I like moving to the dictionary for |
Done, now this depends on DiamondLightSource/dodal#66 |
Fixes #660
To test: