-
Notifications
You must be signed in to change notification settings - Fork 5
Fix issues in robot load found in testing #1051
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1051 +/- ##
==========================================
- Coverage 93.75% 93.72% -0.03%
==========================================
Files 58 58
Lines 2801 2807 +6
==========================================
+ Hits 2626 2631 +5
- Misses 175 176 +1 ☔ 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, works for me. Not thrilled about having to make the parameter types Any
, can we make an issue for this?
@@ -185,7 +185,7 @@ def put(self, plan_name: str, action: Actions): | |||
) | |||
if plan is None: | |||
raise PlanNotFound( | |||
f"Experiment plan '{plan_name}' has no 'run' method." | |||
f"Experiment plan '{plan_name}' not found in context. Context has {self.context.plan_functions.keys()}" |
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.
❤️
Managed to fix it by just removing the |
Test is transient, see #999. So I will merge this |
Fixes #1049
To test:
In another terminal run an xray centre then robot load plan:
curl -X PUT http://127.0.0.1:5005/wait_for_robot_load_then_centre/start --data-binary "@tests/test_data/parameter_json_files/good_test_wait_for_robot_load_params.json" -H "Content-Type: application/json"
. Confirm this gives a success