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

Detect if filter has diverged #25

Open
wants to merge 3 commits into
base: hydro-devel
Choose a base branch
from

Conversation

trainman419
Copy link
Contributor

My particular configuration was diverging frequently due to poor configuration, so I added a check for it and some debugging output about the last set of messages the the filter received before it diverged.

This is generally quite useful for spotting bad input data or bad covariances.

}
}

ROS_ERROR_STREAM(errmsg.str());
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does it make sense to use a throttled message here? once you have diverged, this will spam the console quite badly I would think

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does spam the console rather a lot. I'll switch it to a throttled message or a once-only message.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest throttle over once-only so that if you remotely open rqt_console or similar after the filter has diverged, you will still see the message.

@mikeferguson
Copy link
Member

Should this also be added to the other filters? (velocity & attitude)

@trainman419
Copy link
Contributor Author

I updated the error message to only print every 5 seconds.

I'll look into adding this to the other filters as well.

@trainman419
Copy link
Contributor Author

I've added divergence check of the velocity and attitude filters. Let me know if you'd like me to squash these commits down before they're merged.

@trainman419 trainman419 mentioned this pull request Jun 10, 2015
@mikeferguson
Copy link
Member

Sorry for the long delay -- this looks good, if you want to squash it, we can merge it

@moriarty moriarty mentioned this pull request Feb 5, 2019
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