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] add weight_candidates_refiner node with no new message #2198

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

pazeshun
Copy link
Contributor

@pazeshun pazeshun commented Aug 19, 2017

Another version of #2193
Depends on jsk-ros-pkg/jsk_3rdparty#120

  • Add weight_candidates_refiner.py with sample, test and doc

Changes from #2193

  • Move Weight and WeightStamped msg to and_scale_ros in jsk_3rdparty

Please see discussion in jsk-ros-pkg/jsk_3rdparty#119 (comment)

@pazeshun pazeshun force-pushed the weight-candidates-refiner-no-msg branch from 0b7a562 to 663d824 Compare August 19, 2017 04:52
@k-okada
Copy link
Member

k-okada commented Sep 12, 2017

please look at #634 (comment), if you are confeden that you never feel like

jsk_recognition_msgsを追加。
  jsk_commonにあるべきパッケージだが、試行錯誤しやすいようにjsk_recognitionに配置

you can move jsk_recognition_msgs to jsk_common

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

Successfully merging this pull request may close these issues.

3 participants