You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The text was updated successfully, but these errors were encountered:
dcconner
changed the title
foxy version: image_rotate does not initialize the target and source _vectors in constructor
ros2 version: image_rotate does not initialize the target and source _vectors in constructor
Apr 4, 2022
I verified compilation for image_rotate, but I went in another direction and developed a new node for ROS 2 I called image_flip that I present in PR #743 for review . I'm not set up to fully test this PR
This is the first component/node with a cleanup pass to be fully
implemented:
* Fix#740 by initializing vectors. Do this by declaring parameters
AFTER we define the callback
* Implement lazy subscribers (I missed this in the earlier PRs)
* Add image_transport parameter so we can specify that desired transport
of our subscriptions
* Update how we test for connectivity (and update the debug message -
has nothing to do with whether we are remapped, it's really about
whether we are connected)
Need to add the following after onInit() call to update data; target_vector and source_vector default to zero
`
target_vector_.vector.x = config_.target_x;
target_vector_.vector.y = config_.target_y;
target_vector_.vector.z = config_.target_z;
source_vector_.vector.x = config_.source_x;
source_vector_.vector.y = config_.source_y;
source_vector_.vector.z = config_.source_z;
`
The text was updated successfully, but these errors were encountered: