artofnothingness / mppic

MIT License
92 stars 22 forks source link

Final steps: Tune MPPI for release on reference hardware platform #56

Closed artofnothingness closed 1 year ago

artofnothingness commented 2 years ago

Too high move forward cost after addition prefer forward critic i guess

SteveMacenski commented 2 years ago

It seemed fine to me last time I tested, but I can take a look again. If its just gain tuning and not code changes required, we can push it off til we finish a stable version

artofnothingness commented 2 years ago

close this temporary till new params tuning iteration

SteveMacenski commented 2 years ago

I just refocused the goal of the ticket as a reminder to do final tuning, which hopefully will be done on hardware if/when I can get access and time to do so

SteveMacenski commented 2 years ago

https://github.com/artofnothingness/mppic/issues/41#issuecomment-1165983035 tuning comments -- probably good for readme or at least when deciding on final parameters to ship with

SteveMacenski commented 2 years ago

So if we're going to aim for what would be the best, I think 40hz with 2000 batches would be the thing to aim for.

with a temp of 0.35, time increment of 0.075, and projection time of 3 seconds with the otherwise default critics, the controller takes about 30ms for 2,000 batches, which corresponds to about 33 hz (it takes about 40-50ms when a new path is given).

That gives us a goal of improving performance by about 20-30% to achieve that.

SteveMacenski commented 2 years ago

Add comment about higher speeds and further prune distances to enable. Review comments in the Smoothing ticket and the Tuning PR to summarize tuning / configuration assistance notes for users.

SteveMacenski commented 2 years ago

Things to evaluate on hardware

SteveMacenski commented 1 year ago

Update defaults in code + readme defaults table + readme xonfiguration example

SteveMacenski commented 1 year ago

104 does docs and testing