openpilot icon indicating copy to clipboard operation
openpilot copied to clipboard

GM: Add BSM from Powertrain Bus

Open garrettpall opened this issue 1 year ago • 18 comments

Using fingerprint info, the message is there on most upstream GM car’s except Volts, Escalades ESV, Equinox, and Holden Astra.

Confirmed working on: Bolt Silverado XT4

Prerequisites: https://github.com/commaai/opendbc/pull/993

garrettpall avatar Dec 27 '23 23:12 garrettpall

GMC 1500(fingerprints as a Silverado) route: dc7716b32bf25574|2023-12-27--21-15-59--0

Bolt EUV route: 3140e1c6195c880e|2023-12-27--21-15-14--0

https://github.com/commaai/openpilot/assets/63013446/b554b0a6-dff7-4969-84ad-20b2c3a35a02

ServerDummy avatar Dec 28 '23 18:12 ServerDummy

On the 2018 Arcadia, the BSM signal is only 1 when the BSM light is on and the turn signal is on.

garrettpall avatar Dec 29 '23 00:12 garrettpall

Nice find!

On the 2018 Arcadia, the BSM signal is only 1 when the BSM light is on and the turn signal is on.

Maybe another bit tells us this?

sshane avatar Jan 02 '24 19:01 sshane

Nice find!

On the 2018 Arcadia, the BSM signal is only 1 when the BSM light is on and the turn signal is on.

Maybe another bit tells us this?

@sshane None in that message, there are times where those are the only bits that change.

garrettpall avatar Jan 03 '24 01:01 garrettpall

Have any routes of this?

sshane avatar Jan 04 '24 01:01 sshane

@sshane df51b2195de66a9b|2023-12-04--15-55-16

garrettpall avatar Jan 04 '24 11:01 garrettpall

Looks like the original signals are

 SG_ RightLaneChangeThreat : 7|1@0+ (1,0) [0|1] ""  XXX
 SG_ LeftLaneChangeThreat : 6|1@0+ (1,0) [0|1] ""  XXX

so this makes sense. Newer cars likely just changed the behavior slightly to always show this

There's also some speed signals for the BSMs in there, but no distance. Can you add a comment to the DBC explaining it can be only when the blinker is on, for some cars?

sshane avatar Jan 04 '24 22:01 sshane

@sshane I think I added the note correctly, can you check?

garrettpall avatar Jan 04 '24 22:01 garrettpall

lgtm, vivek is trying out the branch on the Bolt for the weekend

sshane avatar Jan 06 '24 07:01 sshane

@sshane Everything work out on the Bolt?

garrettpall avatar Jan 11 '24 22:01 garrettpall

It looks like you didn't use one of the Pull Request templates. Please check the contributing docs. Also make sure that you didn't modify any of the checkboxes or headings within the template.

github-actions[bot] avatar Jan 17 '24 08:01 github-actions[bot]

There's also some speed signals for the BSMs in there, but no distance. Can you add a comment to the DBC explaining it can be only when the blinker is on, for some cars?

On the Acadia, the Blind Spot warning light on the mirror lights up when a vehicle is in your blindspot and the warning light flashes if the turn signal is turned on while a vehicle is there. GM calls this the Side Blind Zone Alert system (SBZA) and is included in the Lane Change Alert system (LCA).

Verylukyguy avatar Jan 23 '24 13:01 Verylukyguy

@Verylukyguy if you want to help us find the bit that has the BSM indicator at all times, you'll need to go get a route where you turn the turn signal on in the direction that it shows the BSM alert for as long as it does (so we can cross reference it with Cabana). This might confuse some drivers around you though

sshane avatar Jan 24 '24 10:01 sshane

@Verylukyguy if you want to help us find the bit that has the BSM indicator at all times, you'll need to go get a route where you turn the turn signal on in the direction that it shows the BSM alert for as long as it does (so we can cross reference it with Cabana). This might confuse some drivers around you though

I definitely want to help improve the overall OpenPilot experience whenever I can. I'll make a route today and let you guys know when it ready.

Verylukyguy avatar Jan 24 '24 13:01 Verylukyguy

Right Side: df51b2195de66a9b|2024-01-24--17-19-57--8 Left Side: df51b2195de66a9b|2024-01-24--18-01-06--1

Verylukyguy avatar Jan 24 '24 23:01 Verylukyguy

I can't find it from that, we'll need at least 10 total seconds throughout a very short route to filter out the noise. Remember, whenever the light is on, you need to perform some action, and whenever the light is off, you should stop that action. Try to match the light very closely. Maybe put the car in L or N gear so you don't have to use the blinkers?

Also needs to only be for one light per action, so:

left bsm light goes on -> put car in low left bsm light goes off -> put car back in drive

a few cases for multiple seconds and we should be able to find it, if it exists

sshane avatar Jan 25 '24 12:01 sshane

Try this route: df51b2195de66a9b|2024-01-26--18-54-15--0

Verylukyguy avatar Jan 27 '24 00:01 Verylukyguy

Updated BCMBlindSpotMonitors to BCMBlindSpotMonitor to match merged opendbc code.

garrettpall avatar Jan 27 '24 21:01 garrettpall

Try this route: df51b2195de66a9b|2024-01-26--18-54-15--0

Thanks, unfortunately I still didn't find it. It may only be on another bus on your car. Besides the original signal name is something to the effect of 'Lane Change Threat', so it only being 1 when the turn signal is on doesn't seem crazy.

sshane avatar Jan 28 '24 04:01 sshane

thanks for the PR @garrettpall!

sshane avatar Jan 28 '24 04:01 sshane