Remove `prefix_is_for_params` from PID_Ros

[bmagyar]

#129 (review)

[christophfroehlich]

@bmagyar why to remove that, and what should be the default behavior?

[github-actions]

This issue is being labeled as stale because it has been open 45 days with no activity. It will be automatically closed after another 45 days without follow-ups.

[github-actions]

This issue was closed because it has been stalled for 45 days with no activity.

[christophfroehlich]

quoting @bmagyar

yes, I think the idea was to move away from that parameter and maybe even entirely disable that functionality or have one default and not allow different ones

[christophfroehlich]

proposal together with @saikishor

• new two constructors with arguments
  • param_prefix -> no topic publisher per default
  • param_prefix, topic_namespace_prefix -> topic publisher is on per default
• no string manipulation for both prefixes -> user has to be explicit
• topic uses the namespace of the node (i.e, "~/" + topic_namespace_prefix)
• ROS Parameter for opt-in/opt-out the publisher with default from above

[saikishor]

Thank you for summarising

[destogl]

I agree on "no string manipulation" as this is the issue here. We were removing "/" from parameters and then the interface names of the following controllers that habe prefix with "/" in it were invalidated.

If we use the local topic names, it means with the node prefix "~/", do we require parameter prefix? I am not sure if we need this for embedding PID to other controllers