spatio_temporal_voxel_layer icon indicating copy to clipboard operation
spatio_temporal_voxel_layer copied to clipboard

VISUALIZE_FRUSTUM gone from ROS2?

Open tonynajjar opened this issue 2 years ago • 4 comments

I noticed you removed the VISUALIZE_FRUSTUM "tool" in this commit. It is still in the README though

I still need to visualize my frustum to understand why voxels are disappearing so quick. Any chance in getting it back? Any existing alternatives?

tonynajjar avatar Jul 21 '22 11:07 tonynajjar

Also being worked on here: https://github.com/PickNikRobotics/rviz_visual_tools/issues/181

tonynajjar avatar Jul 21 '22 11:07 tonynajjar

I forget why I removed it in the port, but I do remember doing that intentionally due to some technical issue. I'm looking at the code now and have no idea why I thought that. If you were to submit a PR to add it back, I'd be happy to merge it (Unfortunately, I'm leaving for Italy in 48 hours for a few weeks, I would not be able to get to an implementation myself in that period of time).

SteveMacenski avatar Jul 21 '22 16:07 SteveMacenski

I found that Foxglove supports visualizing the Frustum from the camera_info topic image

The problem is still that STVL's frustum are not created based on the camera_info topic, but based on the parameters, which is redundant information. So I can't be sure that what i'm visualizing in foxglove is the same in STVL.

Feature request: STVL creates the Frustums from the camera_info if it exists, fallbacks to parameters.

On second thought, as for what VISUALIZE_FRUSTUM accomplishes, I think this should be an rviz plugin.

Sorry I won't have time to contribute here, my after-work time is currently full with the new tricycle controller

tonynajjar avatar Jul 21 '22 17:07 tonynajjar

Feature request: STVL creates the Frustums from the camera_info if it exists, fallbacks to parameters.

Let me know if you want me to create a separate feature request for that (if you agree with it)

tonynajjar avatar Jul 21 '22 18:07 tonynajjar