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

FCA: Only spoof EPS when steering command is active #15

Open
wants to merge 10 commits into
base: sunnypilot_wp_chrysler_advanced
Choose a base branch
from

Conversation

visualage
Copy link

No description provided.

@visualage visualage force-pushed the visualage_wp_chrysler_advanced branch from c3fcf15 to a1df079 Compare February 26, 2024 16:54
@sunnyhaibin sunnyhaibin changed the title spoofing the speed only when steering command is active FCA: Only spoof vEgo when steering command is active Feb 28, 2024
@visualage visualage force-pushed the visualage_wp_chrysler_advanced branch from 5be8a35 to 9691b4e Compare February 29, 2024 17:35
@visualage visualage changed the title FCA: Only spoof vEgo when steering command is active FCA: Only spoof EPS when steering command is active Feb 29, 2024
@visualage visualage force-pushed the visualage_wp_chrysler_advanced branch from 9691b4e to ad623e1 Compare February 29, 2024 17:37
it seems EPS will fault when the speed is low (e.g. <15mph) and LKAS torque is sent too soon. introduce a way to dyanmically adjust the wait time (cycles) based on the speed before spoofing

for speed <3mph, the wait time is 1s.
for speed <9mph, the wait time is 0.5s.
for speed <15mph, the wait time is 0.2s.
for speed <39mph, the wait time is 0.1s.
for speed >=39mph, no wait time.
<5kph, delay 0.7s
<10kph, delay 0.6s
<15kph, delay 0.4s
<25kph, delay 0.2s
<65kph, delay 0.1s
<5kph, delay 0.7s
<10kph, delay 0.6s
<15kph, delay 0.4s
<20kph, delay 0.3s
<25kph, delay 0.2s
<65kph, delay 0.1s
sunnyhaibin added a commit that referenced this pull request Jul 29, 2024
* MADS: cruise main button check cleanup

* exit and resume controls cleanup

* shorten and simplier

* Update var names

* simplify LKAS button press

* more cleanup

* Hyundai longitudinal: revert back for now

* Add todo for future PR
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant