Closed snozawa closed 1 month ago
@snozawa what are the mujin repos tied to this pull request? thanks
@rdiankov complete description is available at https://tiny.mujin.co.jp/fkge8
By the way, it should be time to ask about the company policy to pasting link (cc @ntohge), though I used shlink. @kanbouchou asked me not to paste direct links inside intranet, but I have not checked that with the CTO yet.
repos and branch names would have been enough, no need for links. thanks
thanks~
Summary
chuckingDirection
ingripperInfo
to deprecate gripper settings inManipulatorInfo
. Now it's recommended to useGripperInfo
forchuckingDirection
andgripperJointNames
.Manipulator
has caches forchuckingDirection
andgripperJointNames
independent from those inManipulatorInfo
. Those inManipulator
are determined both byGripperInfo
andManipulatorInfo
.ManipulatorInfo
for backward compatibility purpose.Note for backward compatibility
GripperInfo
should havechuckingDirection
/gripperJointNames
.ManipulatorInfo
.AddManipulator
to define temporary manipulator, and it requires setting inManipulatorInfo
.ManipulatorInfo
. Instead,Manipulator
class has its own caches. That way, if the openrave loads the cleanly-migrated scene (e.g. setting is only inGripperInfo
) and saves it, it keeps clean (e.g.ManipulatorInfo
does not have settings).Manipulator
about chuckingSetClosingDirection
andSetChuckingDirection
, since these seem unused.