robotology / gz-sim-yarp-plugins

YARP plugins for Modern Gazebo (gz-sim).
BSD 3-Clause "New" or "Revised" License
9 stars 2 forks source link

Add support for loading vector from configuration files with parenthesis #141

Closed xela-95 closed 7 months ago

xela-95 commented 7 months ago

Closes #138

As discussed in https://github.com/robotology/yarp/discussions/3092, it is recommended for future software that uses YARP to use vectors in configuration files by indicating them with parenthesis, i.e. to refer to them with:

nameOfList (elem1 elem2 elem3)

instead of:

nameOfList elem1 elem2 elem3

After this PR, now plugins support both the with-parentheses and without-parenthesis style, printing a warning if the without-parenthesis style is used.

traversaro commented 7 months ago

If you can require gazebo-yarp-plugins 4.11.0, consider switching your configuration files to the vector with parenthesis style.

Copy and paste error?

xela-95 commented 7 months ago

If you can require gazebo-yarp-plugins 4.11.0, consider switching your configuration files to the vector with parenthesis style.

Copy and paste error?

Of course 🤣

xela-95 commented 7 months ago

Merging 🚀