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_interactive_marker] Explicitly resolve dependency for jsk_recognition_utils in package.xml #699

Merged
merged 1 commit into from
Aug 6, 2018

Conversation

YutoUchimi
Copy link
Contributor

jsk_recognition_utils package is installed in dependent pacakges such as jsk_rviz_plugins, but jsk_recognition_utils is imported or included in some files in jsk_interactive_marker package (e.g. jsk_interactive_marker/scripts/transformable_markers_client.py).

So I think it desirable to explicitly install that package from jsk_interactive_marker.

@k-okada
Copy link
Member

k-okada commented Jul 18, 2018

can you visualize current dependency graph using rqt_dep ?

@k-okada k-okada requested a review from iory July 23, 2018 03:21
@YutoUchimi
Copy link
Contributor Author

OK. This is the image of dependency graph.

rospackgraph_jsk_interactive_marker resized

@YutoUchimi
Copy link
Contributor Author

... and here is the dependency graph in DOT format.

https://drive.google.com/open?id=1Vcg-pshcnJQ1iPYFkmFvUCXuRUCmkboY

@k-okada
Copy link
Member

k-okada commented Jul 24, 2018 via email

@YutoUchimi
Copy link
Contributor Author

These are the packages which jsk_interactive_marker depends on.

  • dependency level = 1
    rospackgraph_jsk_interactive_marker_level1

  • dependency level = 2
    rospackgraph_jsk_interactive_marker_level2

  • dependency level = 2 (part of graph)
    rospackgraph_jsk_interactive_marker_level2_part

As shown in images above, only jsk_rviz_plugins directly depnds on jsk_recognition_utils before this PR.

@k-okada
Copy link
Member

k-okada commented Aug 3, 2018 via email

@YutoUchimi
Copy link
Contributor Author

Visualizing with ros-visualization/rqt_dep#6

  • dependency level = 1
    rospackgraph_jsk_interactive_marker_l1_stack

  • dependency level = 2
    rospackgraph_jsk_interactive_marker_l2_stack

  • dependency level = 2 (part of graph)
    rospackgraph_jsk_interactive_marker_l2_part_stack

@k-okada
Copy link
Member

k-okada commented Aug 6, 2018

ok #666 (comment) says

jsk_rviz_plugins depends on jsk_recognition_msgs and jsk_recognition_utils at build time.
jsk_pcl_ros depends on jsk_rviz_plugins at run time.

and now jsk_interactive_marker also depends on jsk_recognition_utils at build time
note that for example some package in jsk_recognition depends on jsk_visualization
but we may not explicitly add dependencies

jsk_recognition$ grep -ri jsk_interac *
jsk_pcl_ros/CHANGELOG.rst:  this nodelet is supposed to be used with jsk_interactive_marker/bounding_box_marker
jsk_pcl_ros/launch/pcl_roi_remote.launch:    <node pkg="jsk_interactive_marker"
jsk_pcl_ros/launch/octomap_object_model_generation.launch:  <node pkg="jsk_interactive_marker"
jsk_pcl_ros/launch/organized_multi_plane_segmentation.launch:  <node pkg="jsk_interactive_marker"
jsk_pcl_ros/launch/pr2_tilt_laser_multi_plane_segmentation.launch:  <node pkg="jsk_interactive_marker"
jsk_pcl_ros/launch/snapit.launch:  <node pkg="jsk_interactive_marker" type="marker_6dof" name="marker">
jsk_pcl_ros/launch/tabletop_tracking.launch:  <node pkg="jsk_interactive_marker"
jsk_pcl_ros/launch/valve_detection.launch:    <node pkg="jsk_interactive_marker" type="polygon_marker" name="polygon_marker">
jsk_pcl_ros/sample/sample_boundingbox_occlusion_rejector.launch:  <node pkg="jsk_interactive_marker" type="marker_6dof" name="sensor_frame">
jsk_pcl_ros/sample/sample_boundingbox_occlusion_rejector.launch:  <node pkg="jsk_interactive_marker" type="marker_6dof" name="target_frame">
jsk_pcl_ros/sample/sample_boundingbox_occlusion_rejector.launch:  <node pkg="jsk_interactive_marker" type="dummy_camera.py" name="dummy_camera">
jsk_pcl_ros/sample/sample_plane_supported_cuboid_estimator.launch:  <node pkg="jsk_interactive_marker" type="marker_6dof" name="sensor_frame">
jsk_pcl_ros/sample/sample_plane_supported_cuboid_estimator.launch:  <node pkg="jsk_interactive_marker" type="dummy_camera.py" name="dummy_camera">
jsk_pcl_ros/sample/tabletop_object_detector.launch:  <node name="bounding_box_marker" pkg="jsk_interactive_marker" type="bounding_box_marker"
jsk_perception/launch/color_histogram_matcher_openni.launch:  <node pkg="jsk_interactive_marker"
jsk_recognition_utils/CHANGELOG.rst:  * Stop depending on jsk_interactive_marker

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants