trajopt icon indicating copy to clipboard operation
trajopt copied to clipboard

Use applyContactManagerConfig instead of manually setting

Open mpowelson opened this issue 3 years ago • 5 comments

Switch to using applyContactManagerConfig rather than doing it manually (and partially)

mpowelson avatar Dec 10 '21 18:12 mpowelson

This is failing because it doesn't handle the collision buffer. I'll have to look more into what to do here. There is also interplay with the CollisionCoeffData

mpowelson avatar Dec 10 '21 20:12 mpowelson

I think to apply the buffer it should probably internally create a new margin data by looping over the provided one and increasing the margin buffer. Though we would need to make sure that pairs with custom distance other than the default are handled correctly with regards to the buffer.

Levi-Armstrong avatar Dec 15 '21 16:12 Levi-Armstrong

@mpowelson what are the next steps on this PR?

Levi-Armstrong avatar Jan 19 '22 14:01 Levi-Armstrong

@mpowelson what are the next steps on this PR?

I'm not sure. We'll have to figure out how we want to handle the CollisionCoeffData and the Collision margin data. Maybe we should add an apply method to the config. Otherwise, I'm not sure how to make the contact manager apply derived configs like this correctly.

mpowelson avatar Jan 19 '22 15:01 mpowelson

@Levi-Armstrong https://m.youtube.com/watch?v=v6MhPl2ICsc don't have your current email. Saw this video, seems like control of walking is a real time nonlinear constrained optimization problem using distance field.

drchrislewis avatar Aug 23 '22 12:08 drchrislewis