Skip to content
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

[jsk_perception][people_pose_estimation_2d] publish image when only subscribed #2164

Merged
merged 1 commit into from Jul 14, 2017

Conversation

Projects
None yet
4 participants
@furushchev
Copy link
Contributor

commented Jul 14, 2017

No description provided.

@furushchev furushchev self-assigned this Jul 14, 2017

@furushchev furushchev requested a review from iory Jul 14, 2017

@@ -189,7 +191,8 @@ def _cb_with_depth(self, img_msg, depth_msg, camera_info_msg):

self.pose_2d_pub.publish(people_pose_2d_msg)
self.pose_pub.publish(people_pose_msg)
self.pub.publish(pose_estimated_msg)
if self.visualize:

This comment has been minimized.

Copy link
@mmurooka

mmurooka Jul 14, 2017

Member

self.visualize()

This comment has been minimized.

Copy link
@wkentaro

wkentaro Jul 14, 2017

Member

It's property, so it looks good.

This comment has been minimized.

Copy link
@mmurooka

mmurooka Jul 14, 2017

Member

Oh, sorry.

@@ -189,7 +191,8 @@ def _cb_with_depth(self, img_msg, depth_msg, camera_info_msg):

self.pose_2d_pub.publish(people_pose_2d_msg)
self.pose_pub.publish(people_pose_msg)
self.pub.publish(pose_estimated_msg)
if self.visualize:

This comment has been minimized.

Copy link
@wkentaro

wkentaro Jul 14, 2017

Member

It's property, so it looks good.

@wkentaro wkentaro added this to the 1.1.4 milestone Jul 14, 2017

@k-okada k-okada merged commit 3a96b8b into jsk-ros-pkg:master Jul 14, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@furushchev furushchev deleted the furushchev:people-vis branch Jul 14, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.