search

start-jsk / jsk_apc

Other
36 stars 35 forks source link

Use weight_candidates_refiner as simple detector of weight change #2659

Closed pazeshun closed 6 years ago

knorth55 commented 6 years ago

what is this for? with not_refine, it seems the node publish [] for the picked candidates and placed candidates

pazeshun commented 6 years ago

不特定の物体を扱うためlabel_names.yamlやobject_weight.yamlがない状況で、電子天秤を物体を持ち上げたかどうかの判定のみに使いたくて(changed_from_resetトピックの利用)、candidates_publisherがいなくても動くようにしました。 空リストのpublishについては、こうするとarc-interfaceの:get-scale-candidatesがそのまま使えるのでこうしました。

ですが、weight_candidates_refinerなのにrefineしないのは変なので、diffの少なさをとるより新しいノードを作るべきだと思えてきました。 その方向でいいですかね？

knorth55 commented 6 years ago

refinerがnot_refineなのはよくわからないなというのが感想です。

wkentaro commented 6 years ago

今の方向性でいい気がしますが、 もう少し上手くかける気がします。

ちょっとやってみます。

ghost commented 6 years ago

ごめん、僕も別ノードの方がいい気がしてきました。

その方向でいいですかね？

その方向でお願いできますか? @pazeshun

pazeshun commented 6 years ago

その方向でお願いできますか? @pazeshun

まだテストしていませんが、 #2660 を作りました。お騒がせしました。