ros2_documentation
ros2_documentation copied to clipboard
Building a custom rviz display
I hate RST.
Review please?
@clalancette ? @audrow ? Any movement?
Sorry I never looked back at this.
I think this would be a good tutorial to have, but I also think it belongs in a series with other RViz changes. I'm going to hold off on looking at this one until we land #4040 , at which point this will have a natural place to slot into.
OK, we've now landed #4040. So I'm going to suggest that this gets moved into a tutorial under the new Tutorials/Intermediate/RViz directory.
Rebased, and moved to the new RViz
folder, and I even changed things up to follow the same directory structure as the existing tutorial in that folder.
I will update the code style if you can provide a .clang-format
file that is to be used canonically for tutorials.
I find this process frustrating, to have a PR sit untouched for a year, then get held up in the end for the placement of curly braces. This seems like perfect is the enemy of the good. If you want to do something to increase the number of community generated tutorials, you need to lower the bar a little.
I will update the code style if you can provide a
.clang-format
file that is to be used canonically for tutorials.
Unfortunately, that is impossible; clang-format
can't be configured to follow our coding style currently.
Unfortunately, that is impossible;
clang-format
can't be configured to follow our coding style currently.
Sorry, that was a little terse. We can't do clang-format
, but the right solution here is to enable uncrustify
on the end repository, and then fix all of the uncrustify problems. That will then conform to the style that we want. I'm willing to do that work if the rviz_plugin_tutorial
repository is available somewhere (that's a blocking issue anyway).
Sorry, that was a little terse. We can't do
clang-format
, but the right solution here is to enableuncrustify
on the end repository, and then fix all of the uncrustify problems. That will then conform to the style that we want. I'm willing to do that work if therviz_plugin_tutorial
repository is available somewhere (that's a blocking issue anyway).
All right, see https://github.com/MetroRobots/rviz_plugin_tutorial/pull/1
@Mergifyio backport humble iron
backport humble iron
✅ Backports have been created
-
#4285 Building a custom rviz display (backport #3384) has been created for branch
humble
-
#4286 Building a custom rviz display (backport #3384) has been created for branch
iron