ros-visualization / rqt_reconfigure

http://wiki.ros.org/rqt_reconfigure
Other
23 stars 50 forks source link

Add support for array types #108

Closed shrijitsingh99 closed 2 years ago

shrijitsingh99 commented 2 years ago

Closes #53 and #56

MatthijsBurgh commented 2 years ago

Could you create a PR for ROS1 too? As it is also missing array support.

shrijitsingh99 commented 2 years ago

Could you create a PR for ROS1 too? As it is also missing array support.

I will try to look into it, though I might not have the cycles available for it and I haven't looked into ROS1 parameters in a while.

Flova commented 2 years ago

Any updates on this?

shrijitsingh99 commented 2 years ago

I don't have the cycles to create a PR for ROS1 at the moment but I think they can be merged in if there are no blockers.

clalancette commented 2 years ago

CI:

clalancette commented 2 years ago

I rebased this and fixed the small style issues pointed out by CI. Here's another CI:

mergify[bot] commented 6 months ago

backport humble

❌ Command disallowed due to command restrictions in the Mergify configuration.

- [ ] `sender-permission>=write`