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

Add failure or missed msg conditions #39

Open
fnivek opened this issue Jun 29, 2015 · 1 comment
Open

Add failure or missed msg conditions #39

fnivek opened this issue Jun 29, 2015 · 1 comment
Labels

Comments

@fnivek
Copy link
Contributor

fnivek commented Jun 29, 2015

All over our code nodes rely on other nodes, if something hangs up, such as odometry, it causes other nodes to hang. We need to identify these issues and resolve them. In example when AZI drive does not get an odometry message it just sits and waits until it does. Instead AZI drive should have some default behaviour such as continue with last known odometry until some allowable amount of time has passed after which it would send a kill.

This is a major task involving review of all of our code. We need to make a standard and plan ahead before starting to patch up all these issues.

@fnivek fnivek added the bug label Jun 29, 2015
@forrestv
Copy link
Contributor

It probably makes more sense to just have some kind of status monitor node
that monitors all the relevant topics and issues a kill if something is
wrong. IMO that's better than cluttering up all the nodes with checks and
kill broadcaster code.

On Mon, Jun 29, 2015 at 2:04 PM, fnivek notifications@github.com wrote:

All over our code nodes rely on other nodes, if something hangs up, such
as odometry, it causes other nodes to hang. We need to identify these
issues and resolve them. In example when AZI drive does not get an odometry
message it just sits and waits until it does. Instead AZI drive should have
some default behaviour such as continue with last known odometry until some
allowable amount of time has passed after which it would send a kill.

This is a major task involving review of all of our code. We need to make
a standard and plan ahead before starting to patch up all these issues.


Reply to this email directly or view it on GitHub
#39.

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

No branches or pull requests

2 participants