Skip to content

Config string refactor#160

Merged
Zolkin1 merged 6 commits into
mainfrom
config-string-refactor
May 8, 2026
Merged

Config string refactor#160
Zolkin1 merged 6 commits into
mainfrom
config-string-refactor

Conversation

@Zolkin1
Copy link
Copy Markdown
Collaborator

@Zolkin1 Zolkin1 commented May 8, 2026

A few big refactors:

  • Removed config strings entirely. We now just dump the yaml into the ros string so it can be parsed cleaner with available yaml files.
  • Removed the "device" argument in the launch and in the config yamls
  • Made the depth sensor/lidar config in the config yaml rather than a seperate yaml
  • Made the config yamls composable to more easily merge them together.
  • Bag defaults to false.

Zolkin1 added 6 commits May 6, 2026 16:43
…ests. removed dead code and removed pixi and LEAP stuff. foxglove also deprecated as it was dropped from humble
…eightScan interfaces now accept a YAML::Node directly. Adds dummy_cpp_scan.yaml example and unit tests for the interfaces.
@Zolkin1 Zolkin1 marked this pull request as ready for review May 8, 2026 20:31
@Zolkin1 Zolkin1 merged commit df59cdb into main May 8, 2026
@Zolkin1 Zolkin1 deleted the config-string-refactor branch May 8, 2026 20:31
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.

1 participant