Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TSS2 Rav4] OpenPilot (engaged) sporadically creeps forward slowly at stopped traffic while at a red light #1270

Closed
dpanych opened this issue Mar 24, 2020 · 19 comments
Labels
bug car vehicle-specific tuning

Comments

@dpanych
Copy link

dpanych commented Mar 24, 2020

Describe the bug
While OP is engaged, when coming to a complete stop (like a red light), OP will first stop at a normal distance, but then it will continue to creep toward the stopped car (very slowly).

How to reproduce or log data
I am unable to create this as it seems to be random. For futures sake, I will save cabana links when it occurs again.
https://my.comma.ai/cabana/?route=7e34a988419b5307%7C2020-03-19--19-19-14&exp=1616186697&sig=GOzZLgLrNbOUJ9FTgnAQF0HTbhe34bN28UfWwi6UJo4%3D&max=16&url=https%3A%2F%2Fchffrprivate-vzn.azureedge.net%2Fchffrprivate3%2Fv2%2F7e34a988419b5307%2F1dac8843e1a4c769857cd329d8baad59_2020-03-19--19-19-14 (Look at 466.405)

Expected behavior
The car should stop and stay stopped until traffic starts moving again.

Device/Version information (please complete the following information):

  • Device: EON GOLD
  • Version: 0.7.4 staging-devel
  • Car make/model: 2020 RAV4 Hybrid XLE
@arne182
Copy link
Contributor

arne182 commented Mar 26, 2020

Have seen this on my fork users.
I have increased this line from longcontrol from 0.5 to
BRAKE_STOPPING_TARGET = 0.75 # apply at least this amount of brake to maintain the vehicle stationary
Giving 25% more braking at standstill.

@arne182
Copy link
Contributor

arne182 commented Mar 26, 2020

I still think we should include lead car distance into longcontrol to avoid this having both the rate and the amount dynamically change based on lead car behaviour.

@pd0wm
Copy link
Contributor

pd0wm commented Mar 27, 2020

How much braking does the stock system send at standstill?

We send an acceleration to the car, 0.5 braking maps to -1.5 m/s^2. Theoretically everything smaller than zero should keep the car stopped, since they have a lower level controller that should keep the brakes applied.

@arne182
Copy link
Contributor

arne182 commented Mar 27, 2020

Yes 0.5 -1.5m/s^2 should keep the car at a standstill but it does not for the TSS2 RAV4 and hybrid. Having -2.0 to -2.5m/s^2, does keep the car from creeping. The funny part is that vEgo is 0 but lead distance is decreasing.

@arne182
Copy link
Contributor

arne182 commented Mar 27, 2020

image

@pd0wm
Copy link
Contributor

pd0wm commented Mar 28, 2020

@arne182 , is that a screenshot from the stock system?

@arne182
Copy link
Contributor

arne182 commented Mar 28, 2020

No. From the open pilot one. Let me ask for a stock drive. BRB

@arne182
Copy link
Contributor

arne182 commented Mar 30, 2020

Stock system uses -3.5m/s and then goes up to 0.
image
I also see a -1.28m/s constantly held without creeping.
in @dpanych drive I see the RELEASE_STANDSTILL is not set to 0 for the whole standstill time
image

arne182 added a commit to arne182/ArnePilot that referenced this issue Mar 30, 2020
This needs to be tested but it looks like it could fix.
commaai#1270
@pd0wm pd0wm changed the title OpenPilot (engaged) sporadically creeps forward slowly at stopped traffic while at a red light [TSS2 Rav4] OpenPilot (engaged) sporadically creeps forward slowly at stopped traffic while at a red light Apr 1, 2020
@pd0wm pd0wm added the tuning label May 11, 2020
@CoPilotPlus
Copy link

I too have this issue on 2020 Corolla Hybrid on 0.7.5. Can this car be tweaked as well?

@directedbit
Copy link

directedbit commented Jun 16, 2020

Had the same issue on 2020 RAV4 TSS2 (0.75 release2)

@Taik
Copy link
Contributor

Taik commented Jul 22, 2020

I can confirm this is still happening on my 2020 RAV4 Hybrid TSS2 (077 release2).

@Volbard
Copy link

Volbard commented Aug 2, 2020

I'm also seeing this.

Device/Version information:
Device: Comma Two
Dongle ID: 93cb227004373575
Route: 93cb227004373575|2020-08-01--13-43-51
Version: release2 0.7.7
Car make/model: 2020 Rav4 Hybrid

Worst case is 13:57 on the route.

Volbard pushed a commit to Volbard/openpilot that referenced this issue Aug 6, 2020
@directedbit
Copy link

@pd0wm - should we wait for this to get fixed or just use @arne182's PR? It's a little unnerving having the car creep forward towards other stationary vehicles

@RightTrash
Copy link

Have a 2020 Rav4 Hybrid XLE with comma 2, 7.10 and at every stop it does on it's own, it comes to the stop, then suddenly creeps and stops again. Sometimes it then creeps more so, the second creeping doesn't always occur, there have been numerous times I've disengaged it because I felt I was creeping too close to the stopped car.
The only time I would say something seemed apparent, perhaps relating, was at an odd 4 lane stop light which was mid sharp bend with a 2 lane one way road, where both roads are intersecting mid sharp bends on a hill also, the car in front of me was at an angle somewhat...

Hope to see this issue/bug worked out, as it seems sketchy in various ways, always ready to disengage in those moments.

@adeebshihadeh
Copy link
Contributor

This should be fixed by d93f07f

@pd0wm pd0wm added bug car vehicle-specific labels Jun 2, 2022
@pd0wm pd0wm removed the car bug label Jun 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug car vehicle-specific tuning
Projects
None yet
Development

No branches or pull requests