Specifically, moving
<rosparam if="$(eval image_topic != '')" param="/$(arg param_prefix)/subscription_topic" subst_value="true">$(arg image_topic)</rosparam> to the global scope caused it to override the subscription topic of the video streamer node, while it should have only affected the encoder node.
And I think that's not the only issue.
Opened https://github.com/aws-robotics/aws-robomaker-sample-application-persondetection/pull/30 to try and address some of the problems found.
I suspect the changes at https://github.com/aws-robotics/aws-robomaker-sample-application-persondetection/pull/4 were only tested on RoboMaker and not on a physical robot.
Specifically, moving
<rosparam if="$(eval image_topic != '')" param="/$(arg param_prefix)/subscription_topic" subst_value="true">$(arg image_topic)</rosparam>
to the global scope caused it to override the subscription topic of the video streamer node, while it should have only affected the encoder node. And I think that's not the only issue. Opened https://github.com/aws-robotics/aws-robomaker-sample-application-persondetection/pull/30 to try and address some of the problems found.