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

Increase max stddev, add pose visualization, fix error messages #192

Merged
merged 4 commits into from
Dec 20, 2023

Conversation

adivardi
Copy link
Contributor

@adivardi adivardi commented Dec 18, 2023

Added a few modifications for easier usage and debugging:

  • Add a debug publisher of the current orientation as a Pose. This allows visualizing the estimated orientation in rviz as an arrow
  • Change error streams in checkTopicsTimerCallback to show correct topics names even if the topics are remapped
  • Increase maximum orientation_stddev to allow higher stddev
  • add timeout to lookupTransform - to avoid spamming warnings of lookup would require extrapolation into the future

@adivardi adivardi force-pushed the av/remove_launchers branch from 460055e to 25f310d Compare December 19, 2023 09:03
@mintar
Copy link
Collaborator

mintar commented Dec 20, 2023

Thanks for the PR, merging!

@mintar mintar merged commit 13ff13a into CCNYRoboticsLab:noetic Dec 20, 2023
1 check passed
@adivardi adivardi deleted the av/remove_launchers branch January 8, 2024 13:38
@adivardi
Copy link
Contributor Author

@mintar any plans for a release soon?

@mintar
Copy link
Collaborator

mintar commented Jan 12, 2024

Thanks for the reminder. Done: ros/rosdistro#39528

mintar added a commit that referenced this pull request Apr 26, 2024
This PR, similar to #192, changes the topic names in the warning message to remapped ones in order to make the message more understandable.

---------

Signed-off-by: Tamaki Nishino <otamachan@gmail.com>
Co-authored-by: Martin Günther <martin.guenther@dfki.de>
sharminramli added a commit to enwaytech/imu_tools that referenced this pull request Jul 9, 2024
sharminramli added a commit to enwaytech/imu_tools that referenced this pull request Jul 9, 2024
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