Skip to content

SLAM: add localization-only launch and params#5

Open
harryyangzy wants to merge 1 commit into
WE-Autopilot:mainfrom
harryyangzy:feature/slam-localization-only
Open

SLAM: add localization-only launch and params#5
harryyangzy wants to merge 1 commit into
WE-Autopilot:mainfrom
harryyangzy:feature/slam-localization-only

Conversation

@harryyangzy

Copy link
Copy Markdown

Merge Request / Pull Request

Summary of Changes

Added a launch file and params YAML for running Kitware LiDAR SLAM in localization-only mode with a prebuilt map.
The launch exposes a map_file argument and disables mapping so it should only estimate pose.
This addresses the localization-only SLAM task and depends on SLAM-001 for the actual SLAM install.

Type of Change

  • Bug fix
  • New feature / task
  • Refactor
  • Documentation

Checklist (to be completed before review)

  • Code follows team standards (as best as I could)
  • Tested locally
  • Tests added / updated if relevant
  • Documentation updated if relevant
  • CI / checks pass

Related Issue / Task

  • Localization-only SLAM launch (depends on SLAM-001)

Notes / Additional Context

I’m not 100% sure it runs yet since SLAM-001 isn’t installed. I also tried to follow the documentation to the best of my ability and used like a good amount of ai... let me know if it works!!

@lblommesteyn

Copy link
Copy Markdown
Contributor

Triage note from AP1 bringup: this adds a separate localization-only launch/config. It may overlap with #4 and should be reviewed together with the Kitware localization plan. Not required for the immediate AP1 bag smoke test, but useful once the team is ready to run localization against a saved map.

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.

2 participants