The ROSSession class was made for specifically to interact with ROS. Currently, the Script class has a method queryScriptStatus in which it instanciates a ROS node handle, and make service call etc.
It would be better if the ROSSession implemented to logic to do this, and exposed a queryScriptStatus() function that Script would simply call.
Anything that needs to make ROS service calls should use a pointer to this ROSSession and use its exposed functions.
The
ROSSessionclass was made for specifically to interact with ROS. Currently, theScriptclass has a methodqueryScriptStatusin which it instanciates a ROS node handle, and make service call etc. It would be better if theROSSessionimplemented to logic to do this, and exposed aqueryScriptStatus()function thatScriptwould simply call.Anything that needs to make ROS service calls should use a pointer to this
ROSSessionand use its exposed functions.