mujoco_mpc icon indicating copy to clipboard operation
mujoco_mpc copied to clipboard

a proposal on updating agent state info before mj_step in PhysicsLoop of app.cc

Open starimpact opened this issue 1 year ago • 1 comments

as following (line 359), what do you say about adding sim.agent->state.Set(m, d); before mj_step. my reason is that , controller need instant state info to control robot, instant time info is good for interpolating right control info, or it will be failed in some hard scenarios. 截屏2024-09-25 17 26 35 i am not sure about this suggestion, because the performance does not improve yet.

starimpact avatar Sep 25 '24 09:09 starimpact

The state is set below in this line. The planner has access to the most up-to-date state. However, I am noticing that we might not be initializing the planner state from the model default state since this function does nothing.

thowell avatar Sep 25 '24 10:09 thowell