Remove unused ParameterEventDescriptors.msg#90
Conversation
Signed-off-by: Stephen Brawner <brawner@gmail.com>
|
@tfoote This was message I was talking about. We can also leave it in if that's preferable. |
tfoote
left a comment
There was a problem hiding this comment.
This message was part of the design to provide a way for UIs to subscribe to a lighter weight version of the ParameterEvents topic. In particular they would be expected to subscribe to this and then only call getParameter for the ones that they care about.
For most parameters this isn't a big deal, but for large parameters with lots of data this will significantly reduce the volume. As well as for tools that may want to support reconfiguring anything, but might only have a few parameters actively in the configuration gui this lets them be responsive, but not hog bandwidth. We ran out of time to implement that and didn't have the demand for it yet. but I think we should instead plan to implement that rather than removing this message.
|
Ok, good points. I'll add information about it to the documentation PR, and maybe a one-liner that the feature has not yet been implemented. I have no issue with it sticking around if you still think it will be part of future functionality. |
This message file was added several years ago and is not currently in use. Instead of adding documentation for an unused feature, I figured it would be better to just remove the message for now.