orocos / rtt_ros_integration

Orocos-ROS integration libraries and tools
BSD 3-Clause "New" or "Revised" License
85 stars 56 forks source link

rtt_rosparam: add general support for sequence property serialization and deserialization #162

Closed Hugal31 closed 3 years ago

Hugal31 commented 3 years ago

For get() (ROS -> Orocos), add a special case to deconstruct array XmlRpcValue into property bags, fill it with "ElementX", as Orocos does itself. For set() (Orocos -> ROS), detect if the property is a sequence by looking at a write-only "size" int property, deconstruct it in a PropertyBag, and read its "ElementX" member properties, as RTT::types::propertyDecomposition does.

Add the corresponding unit tests.

Closes #155.

Hugal31 commented 3 years ago

I have some troubles making this work on the pipeline. This is probably because I am using ROS noetic, and not ROS kinetic as the pipeline. I'll try if I can reproduce a kinetic environment.

Hugal31 commented 3 years ago

Is it possible to merge this? It is ready now.

meyerj commented 3 years ago

Is it possible to merge this? It is ready now.

Yes, sorry for the delay!