pangtao22 / planning_through_contact

MIT License
41 stars 7 forks source link

Tree growth visualization #34

Closed pangtao22 closed 2 years ago

pangtao22 commented 2 years ago

Features added to visualizer:

image

Some changes are made to IrsRrt.calc_metric_batch to support computing distances using different metrics while building the tree and after the tree is built. As a result, IrsRrtGlobal is removed as its only purpose is to support a different distance metric. Per f2f discussion with @hjsuh94, distance metric is now a param in IrsRrtParams and all different metrics are handled by IrsRrt.calc_metric_batch. @lujieyang This will require some changes on your Allegro hand branch, but it should accommodate the new quaternion distances you just wrote?


This change is Reviewable

lujieyang commented 2 years ago

I feel we have two versions of implementing different modes for cal_metric_batch: 1) using if statement 2) inherit for subclass. I think quaternion mode can fit in both.