commaai / commaai/openpilot

torqued: filter out high longitudinal acceleration

Open
#32,953 1 comment 0 reactions 0 assignees View on GitHub
bug controls
Dominant language
Python
Stars
63.7k
Forks
11.4k
Avg merge
17h 53m
Merged PRs (30d)
171

Description

For example, when abs(accel) > 1 m/s^2, lateral acceleration achieved may be different than at that speed with an accel of 0 m/s^2. We also use the past speed when calculating the lateral accel, so this is further undefined behavior

![image](https://github.com/commaai/openpilot/assets/25857203/a597ce06-98f3-4fc3-befa-475e1bf98a6b)

Contributor guide

Open the contributing guide

Research direction

Start by locating the torqued lateral-acceleration calculation and trace how longitudinal acceleration and past speed affect it. Confirm the intended handling for cases where abs(accel) exceeds 1 m/s^2, then add coverage showing that those cases are filtered out and verify the resulting lateral-acceleration behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
robotics
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.