Skip to content
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

Improve task input checking #302

Open
marip8 opened this issue Apr 6, 2023 · 1 comment
Open

Improve task input checking #302

marip8 opened this issue Apr 6, 2023 · 1 comment

Comments

@marip8
Copy link
Contributor

marip8 commented Apr 6, 2023

We should add an additional check on most of the task composer task inputs that there is at least one move instruction in the composite. Otherwise the task silently completes without actually doing anything and without populating the output data, making it a difficult error to track down

We are currently not doing this for the motion planning task or the min length task

We might also consider putting this check in the CheckInput task/profile if it makes more sense

@Levi-Armstrong
Copy link
Contributor

I would add this to the check input profile.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants