martinl / openpilot

Open source driving agent - Subaru fork
https://github.com/martinl/openpilot/wiki/
MIT License
70 stars 72 forks source link

first draft working retune for my crosstrek. primarily actuator delay #52

Closed ClockeNessMnstr closed 3 years ago

ClockeNessMnstr commented 3 years ago

I was messing with the basePoints as well, bumped to about 70mph. seemed strange that the interpolation stopped at ~45mph before.

Otherwise this is a tune that is working pretty well on my crosstrek which was having a slow weave/drift on the Hwy @ 70+mph It was also previously taking turns late (Actuator delay seemed to take care of that.)

If this tune doesn't work for you then we could try just adjusting the Actuator Delay. (0.2 was still a touch late for me. 0.15 was cutting the corner) 0.18 seemed about right.

I started with Actuator delay watching the timing on OP taking a corner/turn. FF alone should be an open loop controller so, Then I bumped "FF" up until it oversteered and brought it back down.

I then bumped "P" up until it oversteered and brought it back down. I did not do a lot to "I" yet but bumping it up helped smooth out the straight line performance so far.

I also will need to try more P/I values at highway speed. I checked them to make sure it was performing well but did not do a second set of tuning for the 32m/s breakpoint I have currently.

ClockeNessMnstr commented 3 years ago

Started Tuning with actual PID instead of PI. feels much better. not sure why it's not something upstream already.

I did notice the steering ratio and weight seemed off for impreza. SR = 13:1 , weight = ~1400

still using actuator delay = .2 I'm not sure why it was so high before.