Pending TODOs Ignition Garden

[ahcorde]

List of TODOs to review before Garden release

• /// \todo(anyone) Move to GuiRunner::Implementation when porting to v5
  • https://github.com/ignitionrobotics/gz-sim/blob/gz-sim7/src/gui/GuiRunner.hh#L74
• // \todo(anyone) Change this to uint64_t in Ignition H?
  • https://github.com/ignitionrobotics/ign-gazebo/blob/ign-gazebo6/src/systems/shader_param/ShaderParam.cc#L288
  • Use uint64_t with gazebo-entity user dataa #1451
• / \todo(anyone) instead of using ifdef to check for macOS, expose add an accessor function to get the GraphicsApi from rendering::RenderEngine
  • https://github.com/ignitionrobotics/ign-gazebo/blob/ign-gazebo6/src/systems/shader_param/ShaderParam.cc#L317-L320
• // \todo handle args for constants in ign-rendering
  • https://github.com/ignitionrobotics/ign-gazebo/blob/ign-gazebo6/src/systems/shader_param/ShaderParam.cc#L365

[chapulina]

We can also update our context menu:

https://github.com/ignitionrobotics/ign-gazebo/blob/fc9cadd680e1414b38be71b404958aa8e16ff70d/src/gui/plugins/modules/EntityContextMenu.qml#L52-L73

[ahcorde]

• Use the field is_light_off in light.proto from Garden on.
  • Used Light ign-msgs is_light_off #1449
• Use the field visualize_visual in light.proto from Garden on.
  • Added visualize_visual to light.proto gz-msgs#248
  • Use message field visualize_visual #1453

[jennuine]

/// \todo(anyone) Move to GuiRunner::Implementation when porting to v5

gz-sim/src/gui/GuiRunner.hh

Line 78 in c52e140

/// \todo(anyone) Move to GuiRunner::Implementation when porting to v5

The todo comment will be removed in #1678

[azeey]

Looks like the first item is done as @jennuine mentioned. I don't think we have time to handle the rest before the Garden release.

[iche033]

the 3rd todo should be addressed by #1713

[azeey]

I think only the last item is left. @iche033 Can we close this issue and create a separate issue for item 4?

[iche033]

I think only the last item is left. @iche033 Can we close this issue and create a separate issue for item 4?

yes that sounds good to me