-
Notifications
You must be signed in to change notification settings - Fork 208
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
Migrate Axes Display #429
Migrate Axes Display #429
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for taking so long to get to this.
LGTM 👍 After a rebase I'll trigger CI.
- Shapes are now unique pointers - When accessing them, return a reference only (no need for anything else
850f59e
to
1fa3907
Compare
Rebase done. I also took the liberty to tweak the readme: The "Oculus" display isn't part of this repository and the Interactive Marker display as well as message filters have been ported. On the other hand, stereo functionality is currently not enabled, so that should be put on the list. I can revert those changes if you want to, of course. |
I noticed that with the default value, |
Edit: re-triggered with dbe74f6 |
rviz_default_plugins/src/rviz_default_plugins/displays/axes/axes_display.cpp
Outdated
Show resolved
Hide resolved
To ensure we get a frame ID in the case the property is set to the fixed frame. Signed-off-by: Jacob Perron <jacob@openrobotics.org>
Thanks for catching this. I only retested with the visual test and some frame transformations. Looks good to me. |
Closes #93