-
Notifications
You must be signed in to change notification settings - Fork 341
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add type masquerading demos #482
Conversation
b4f3875
to
ef45cb5
Compare
Signed-off-by: Audrow Nash <[email protected]>
Signed-off-by: Audrow Nash <[email protected]>
Signed-off-by: Audrow Nash <[email protected]>
Signed-off-by: William Woodall <[email protected]>
Signed-off-by: William Woodall <[email protected]>
Signed-off-by: William Woodall <[email protected]>
c370326
to
07debb3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, will wait for @audrow to review and run CI.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like we'll have to investigate the opencv related failure. |
Signed-off-by: Audrow Nash <[email protected]>
I opened ros2/rclcpp#1674 to address the GCC issue. @audrow can you run CI? I know you did something odd to run it before, IIRC. |
Signed-off-by: William Woodall <[email protected]>
Signed-off-by: Audrow Nash <[email protected]>
* Revert "Revert "Add type masquerading demos (#482)" (#520)" This reverts commit 61aeb28. * revert out-of-date changes to intra process demo Signed-off-by: William Woodall <[email protected]>
This PR shows how type masquerading could be for images and with an intraprocess pub/sub.
Currently, type masquerading isn't implemented.
This is part of the Type Masquerading project.