This is cool, but the use case I'm envisioning will be inefficient with this. Because I want to get the list of the nodes (e.g., all python nodes) automatically, I have to create the LaunchConfig beforehand. And then here, we read the launch file again. Maybe it would make sense to allow passing a LaunchConfig to this method. What do you think?
This is cool, but the use case I'm envisioning will be inefficient with this. Because I want to get the list of the nodes (e.g., all python nodes) automatically, I have to create the
LaunchConfigbeforehand. And then here, we read the launch file again. Maybe it would make sense to allow passing aLaunchConfigto this method. What do you think?_Originally posted by @afsafzal in https://github.com/ChrisTimperley/roswire/pull/370#discussion_r459166023_