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

throw error if the tilt joint was not found by name #6

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

annaborn
Copy link

@annaborn annaborn commented Mar 2, 2022

Add error handling if the name of sensor is wrong.
Otherwise, the task goes in exception state without any error message

@skasperski Sebastian, could you please take a look and merge it? Thank you in advance!

@annaborn
Copy link
Author

annaborn commented Mar 3, 2022

@jmachowinski Hello Janosch! :)
Could you please check the commits and merge it?

@jmachowinski
Copy link
Contributor

I'm confused by these commits, is the outcome not exactly the same, as not caching the exception ?
AFAIK The main orocos loop catches the exception and would transit the task state to exception.
Or is this just about getting a nice error message ?

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

Successfully merging this pull request may close these issues.

2 participants