commaai / commaai/opendbc

COROLLA: investigating steering wobble

Open
#1,645 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
3.4k
Forks
2.3k
Avg merge
3d 2h
Merged PRs (30d)
38

Description

## `9b84658caefee71a/00000157--80aed67dfb`

This route has code to enable SDSU OP long on TSSP corolla but same lateral parameters as upstream OP for corolla.

I've been messing with lateral tuning to allow for driving 70-80mph without death wobble.
There is also some notable steering wobble at lower speeds as I will show below.

I've noticed that maybe this isn't as important as the steeringActuatorDelay/steeringDeadzone.

I've been happiest with scaled torque controller params `kp = 1.0 * 0.4`, `kf = 1.0 * 1.2`, and `ki = 0.1 * 0.7` when driving at highway speeds but think all that might be necessary is steeringActuatorDelay/steeringDeadzone similar to PRIUS (with bad steering angle sensor?).

![Image](https://github.com/user-attachments/assets/af5c7f64-67e0-4329-91a5-5d64f8ec9169)

![Image](https://github.com/user-attachments/assets/e7a7d1e3-4935-4539-b8f1-1ac8a8e3f64e)

![Image](https://github.com/user-attachments/assets/0860c419-8216-46b9-be18-4427ca487932)

## 9b84658caefee71a/00000156--a5f4c4aa3c
is a route with steerActuatorDelay increased from 0.12 to 0.18 and steeringAngleDeadzoneDeg of 0.2 (with stock torque tuning PIDF params).

I'll be gathering data from master (no OP long for COROLLA) showing the difference between stock lateral params and the effects of 0.2 degrees deadzone as well as maybe ActuatorDelay.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.