-
Notifications
You must be signed in to change notification settings - Fork 26
Description
Hi all,
First of all, thanks for making this library available!
I would like to propose a simplified way of defining the .params file. Essentially, the current .params file is a Python file, which makes the process of managing metaconfigurations and auto-generation complicated. I created the concept of a metaconfig file, which describes all arguments for the add function. I've also changed the add function to allow positional and keyword args.
After these modifications, the .params file is much simpler.
I've done some test on my forks and they work fine. Please feel free to test it using my repos, here:
https://github.com/lucascoelhof/rosparam_handler
https://github.com/lucascoelhof/rosparam_handler_tests
Do you think this modification is relevant? If yes, how could we make the transition smooth for everyone?
Best regards