Skip to content

Conversation

arjo129
Copy link
Contributor

@arjo129 arjo129 commented Sep 19, 2025

🎉 New feature

Summary

A UI for the free-space-explorer system so that users can trivially export occupancy grids.

occupancy_export_ui_reduced.mp4

Currently you have to trigger "run" to get the scan to work.

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

arjo129 and others added 11 commits June 25, 2025 09:00
This PR adds a system which exports occupancy grids. Currently its a
work in progress. The BFS algorithm seems to not identify when there
cannot be any progress made.

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
This commit pushes an initial version of the code.

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
…m/gz-sim into arjo/feat/export_occupancy_grid
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
@arjo129 arjo129 changed the base branch from gz-sim10 to main September 20, 2025 10:51
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
};

ExportOccupancyUi::ExportOccupancyUi() :
dataPtr(std::make_unique<ExportOccupancyUiPrivate>())
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

include <memory>


ExportOccupancyUi::~ExportOccupancyUi()
{

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change

EntityComponentManager &_ecm)
{
auto world = World(worldEntity(_ecm));
if (!world.Valid(_ecm)) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if (!world.Valid(_ecm)) {
if (!world.Valid(_ecm))
{

@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development Sep 22, 2025
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Makes it trivial to export occupancy grids from gazebo worlds.

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
@arjo129 arjo129 marked this pull request as ready for review October 6, 2025 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪵 jetty Gazebo Jetty
Projects
Status: In review
Development

Successfully merging this pull request may close these issues.

3 participants