Skip to content

[moveit_configs_utils] Switch warehouse defaults to SQLite - #3805

Open
dlanov wants to merge 1 commit into
moveit:mainfrom
dlanov:fix-1296-sqlite-warehouse
Open

[moveit_configs_utils] Switch warehouse defaults to SQLite#3805
dlanov wants to merge 1 commit into
moveit:mainfrom
dlanov:fix-1296-sqlite-warehouse

Conversation

@dlanov

@dlanov dlanov commented Jul 31, 2026

Copy link
Copy Markdown

Description

Switch the generated MoveIt warehouse launch configuration from MongoDB to SQLite.

This change:

  • Configures warehouse_ros_sqlite::DatabaseConnection instead of the MongoDB plugin.
  • Uses moveit_warehouse_database_path as SQLite's warehouse_host database-file path.
  • Stores the default database under $ROS_HOME or ~/.ros, using a config-package-specific filename.
  • Removes the MongoDB wrapper process and MongoDB-specific host and port launch arguments.
  • Scopes warehouse parameters to the generated move_group and RViz launches so they do not affect unrelated nodes.
  • Preserves database reset support through moveit_init_demo_warehouse.
  • Adds tests for the SQLite defaults, custom database paths, reset behavior, parameter scoping, and launch ordering.
  • Documents the launch-argument migration in MIGRATION.md.

This PR changes the generated warehouse defaults and launch wiring only. It does not address the separate SQLite planning-scene schema issue reported in #3607.

Fixes #1296.

Testing

pre-commit run --files \
  MIGRATION.md \
  moveit_configs_utils/moveit_configs_utils/launches.py \
  moveit_configs_utils/test/test_warehouse_launch.py

colcon build \
  --packages-select moveit_configs_utils \
  --symlink-install

colcon test \
  --packages-select moveit_configs_utils

colcon test-result --verbose

Result:

7 tests, 0 errors, 0 failures, 0 skipped

Checklist

  • Required by CI: Code is auto formatted using the repository pre-commit configuration.
  • Extend the tutorials / documentation — not required for this generated-launch change.
  • Document API changes relevant to the user in MIGRATION.md.
  • Create tests which fail without this PR.
  • Include a screenshot if changing a GUI — not applicable; no GUI changes.
  • While waiting for review, help review another open pull request.

Signed-off-by: Dennis Lanov <dennis.lanov@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[moveit_configs_utils] Switch warehouse defaults to SQLite

1 participant