search

ros2 / demos

Apache License 2.0
491 stars 329 forks source link

Update README.md with image pipeline all seperately executable name error #689

Closed trushant05 closed 3 months ago

trushant05 commented 3 months ago

During the testing of issue Intra-process executables #1328, I noticed a discrepancy in the node name specified for the Image Pipeline All Separately. The documentation states the executable name as watermarked_node, which should be corrected to watermark_node for consistency and accuracy.

Brief: Executable name: watermarked_node -> watermark_node

samuel-ongzx commented 3 months ago

Would like to add that the node name needs to be updated to watermark_node (instead of watermarked_node), Line 75 in the README.md. Node name here.

trushant05 commented 3 months ago

@ahcorde will create a new PR and closing this one. Thank you for the input.