aikkala / O2MConverter

OpenSim to MuJoCo XML converter
63 stars 13 forks source link

Add forcerange approximation with max isometric force #6

Closed SashaMalysheva closed 2 years ago

SashaMalysheva commented 2 years ago

For OpenSim Rl usage add forcerange.

gpzhaome commented 2 years ago

I think the max force should be higher than the max isometric force (e.g., during eccentric muscle contraction). Otherwise this could have an similar effect as clipping the force-velocity curve. I'm not sure how much this would influence the simulation for OpenSim RI usage.

aikkala commented 2 years ago

Thanks @SashaMalysheva. We recently pushed a commit that sets "max_isometric_force" (from opensim) as the "force" muscle parameter (in mujoco). @gpzhaome do you mean that the max in "forcerange" should be higher than "max_isometric_force", or that the "force" muscle parameter should be higher than "max_isometric_force"?

gpzhaome commented 2 years ago

@aikkala Sorry for the confusion. I mean the max in "forcerange" should be higher than "max_isometric_force". The "force" muscle parameter in Mujoco is defined as the peak active force at zero velocity, which is equivalent to the max isometric force.

aikkala commented 2 years ago

Thanks @gpzhaome for confirming this. Closing this PR for now, but if @SashaMalysheva or someone else has an idea how the forcerange should be estimated I'd be happy to hear it.