Skip to content

Rivian: correct SCCM_WheelTouch - #3510

Closed
lukasloetkolben wants to merge 1 commit into
commaai:masterfrom
lukasloetkolben:rivian-touch
Closed

Rivian: correct SCCM_WheelTouch#3510
lukasloetkolben wants to merge 1 commit into
commaai:masterfrom
lukasloetkolben:rivian-touch

Conversation

@lukasloetkolben

Copy link
Copy Markdown
Contributor

SCCM_WheelTouch_HandsOn becomes true once SCCM_WheelTouch_CapacitiveValue rises above SCCM_WheelTouch_Calibration.

I split the 12-bit CapacitiveValue into two 8-bit signals: SCCM_WheelTouch_CapacitiveValue and a new SCCM_WheelTouch_ResistiveValue.
ResistiveValue seems to reflects the resistive component of the touch. it changes depending on whether you are wearing gloves or have wet hands.

@github-actions github-actions Bot added DBC signals car related to opendbc/car/ rivian car safety vehicle-specific safety code labels Jun 28, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Car behavior report

Replays driving segments through this PR and compares the behavior to master.
Please review any changes carefully to ensure they are expected.

Testing 10 segments for: RIVIAN_R1

✅ 0 changed, 10 passed, 0 errors

@sshane

sshane commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Do you have a route showing these differences?

@lukasloetkolben

Copy link
Copy Markdown
Contributor Author

This is actually visible in every route. In particular SCCM_WheelTouch_Calibration is obvious since we have the SCCM_WheelTouch_HandsOn bit

@lukasloetkolben

Copy link
Copy Markdown
Contributor Author

No stock openpilot route on hand right niw. Screenshot below shows Calibration vs HandsOn

Bildschirmfoto 2026-07-28 um 10 54 09

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

car safety vehicle-specific safety code car related to opendbc/car/ DBC signals rivian

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants