Skip to content

HKG: Can-fd resume from standstill improvement - #26070

Closed
hoomoose wants to merge 0 commit into
commaai:masterfrom
hoomoose:canfd-resume-improvement
Closed

hoomoose wants to merge 0 commit into
commaai:masterfrom
hoomoose:canfd-resume-improvement

Conversation

@hoomoose

@hoomoose hoomoose commented Oct 13, 2022 •

Copy link
Copy Markdown
Contributor

Description
Resume from standstill fails to work 1/5 times in a given drive in my hda1 EV6 (and others as well). To solve this problem I have employed the same technique as used in non-canfd hkg cars. I propose sending 5 "resume" button presses at a time. I attempted sending 25 button presses as is currently done with non-canfd cars, but this resulted in a communications error. 5 button presses appears to improve the resume from standstill (no failures in 3 days of driving) and does not give the communications error.

Verification
This fix has been employed for a few days now by a few users in a fork created specifically for hda1 ev6 and I5 cars (hoomoose:hda1ev6i5) since master does not have the prerequisite resume from standstill fix needed for hda1 ev6 and I5. In three days of driving with this fix, I have not had a failure of resume from standstill (yet...). However, this fix would affect all can-fd cars with the cruise_buttons signal and can/should be tested by hda1 and hda2 cars.

Prerequisite
For testing on an HDA1 car, the hda1 fix is a prerequisite:
#26069

@hoomoose

Copy link
Copy Markdown
Contributor Author

I had my first failure with this method, so it is not perfect. Just thought I would report this.

@incognitojam incognitojam added car vehicle-specific hyundai labels Oct 17, 2022
@sunnyhaibin

Copy link
Copy Markdown
Contributor

@hoomoose Have you tried something similar to this PR?

@hoomoose

Copy link
Copy Markdown
Contributor Author

I just tested out the solution offered by Sunnyhaibin, and have committed the changes.
In one day of driving through rush hour traffic, I had no errors and 100% resume rate. I would estimate success was 14/14. I do not think a conclusion was reached whether to merge the previous PR, but it certainly seems to cause no negative issues with an EV6 and would appear (with limited data from one can-fd car) to offer success. What would be a next step - have more cars try this out?

@sunnyhaibin

Copy link
Copy Markdown
Contributor

I just tested out the solution offered by Sunnyhaibin, and have committed the changes. In one day of driving through rush hour traffic, I had no errors and 100% resume rate. I would estimate success was 14/14. I do not think a conclusion was reached whether to merge the previous PR, but it certainly seems to cause no negative issues with an EV6 and would appear (with limited data from one can-fd car) to offer success. What would be a next step - have more cars try this out?

@hoomoose Provide the route(s) with the resume tests should be sufficient for review.

@adeebshihadeh

Copy link
Copy Markdown
Contributor

Now that HDA1 resume is in master, can you try master for a bit and see if you still have problems?

@hoomoose

Copy link
Copy Markdown
Contributor Author

Installing as I type - thanks!

@hoomoose

Copy link
Copy Markdown
Contributor Author

In driving through downtown to and from my rec center on the new Master with no modifications from me (which is awesome by the way!), I had the following results.
To gym 4/6 successful resumes (Route 68d6a96e703c00c9|2022-10-19--16-28-36)
From Gym 7/8 successful resumes (Route 68d6a96e703c00c9|2022-10-19--17-49-42)
(and 1 false positive where the car resumed but the car in front wasn't moving, but that's a success in my book.)
Overall successful resume rate 11/14.
I am happy to keep driving as is to get more data, but I would say this has been typical of my experience all along before trying to do improvements. Others who post (can-fd with hda1 and hda2) seem to have similar experiences of the ~15 -25% failure rate. Do you have a similar experience with your EV6? I will also try this fix again after re-basing with current master to see if I have the same success rate as I previously posted and I will post routes.

@hoomoose hoomoose closed this Oct 20, 2022
@hoomoose
hoomoose force-pushed the canfd-resume-improvement branch from b43ed61 to 1800592 Compare October 20, 2022 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

car vehicle-specific hyundai

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants