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_pcl_ros_utils/src] add onInitPostProcess for StaticPolygonArrayPublisher, PolygonArrayTransformer #2126

Merged
merged 1 commit into from
Jun 22, 2017

Conversation

kochigami
Copy link
Contributor

When I launch
StaticPolygonArrayPublisher (node name is set to [/people_detection/footstep_polygon_publisher)
and PolygonArrayTransformer (node name is set to [/people_detection/polygon_array_transformer)
of jsk_pcl_ros_utils, the warning below occurred.

[ WARN] [1498035213.941695751]: [/people_detection/footstep_polygon_publisher] onInitPostProcess is not yet called.
[ WARN] [1498035213.942671862]: '/people_detection/footstep_polygon_publisher' subscribes topics only with child subscribers.
[ WARN] [1498035214.353049047]: [/people_detection/polygon_array_transformer] onInitPostProcess is not yet called.
[ WARN] [1498035214.353923307]: '/people_detection/polygon_array_transformer' subscribes topics only with child subscribers.

If there is any comments, please let me know.

…lisher_nodelet.cpp, polygon_array_transformer_nodelet.cpp
@kochigami kochigami changed the title [jsk_pcl_ros_utils/src] add onInitPostProcess to static_polygon_array_publisher_nodelet.cpp, polygon_array_transformer_nodelet.cpp [jsk_pcl_ros_utils/src] add onInitPostProcess for StaticPolygonArrayPublisher, PolygonArrayTransformer Jun 21, 2017
@k-okada
Copy link
Member

k-okada commented Jun 21, 2017

I'll merge after travis passed, btw did jsk-ros-pkg/jsk_common#1513 (comment) help you to find this problem?

@kochigami
Copy link
Contributor Author

btw did jsk-ros-pkg/jsk_common#1513 (comment) help you to find this problem?

Yes.

I was debugging my launch because it is not working, and I looked into the terminal log.
Then I found it.

@k-okada k-okada merged commit 2c7f0a3 into jsk-ros-pkg:master Jun 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants