search

xqms / rosmon

ROS node launcher & monitoring daemon
Other
180 stars 47 forks source link

core/launch: support YAML merge keys inside arrays #153

Closed xqms closed 3 years ago

xqms commented 3 years ago

This should fix #150, where problems with the non-standard YAML merge keys << inside arrays were noticed:

defaults: &defaults
  A: 1
  B: 2
array:
  - <<: *defaults
    A: 23
    C: 99

Previously, merge keys where only handled on a per-parameter level. Since arrays are imported onto the ROS parameter server as a single XmlRpcValue, merge keys inside were not handled at all. We now do that inside yamlToXmlRpc(), which fixes this situation.

It also adds a rudimentary unit test.