Skip to content

Conversation

@Ian-Boraks
Copy link

@Ian-Boraks Ian-Boraks commented Dec 7, 2025

On Jafaral's custom Mazda forks, they have disabled the steer_required alert for newer Mazdas as they have the ability to steer down to zero.

From Jafaral on the comma.ai community discord (message link):

I chose to do alerts only when it actually make sense. When I added the official support, Comma requested to keep those alerts, so I kept/added them. However, there was so much unnecessary noise and I actually polled the users here and all requested no audiable alrers . . . and for the new CX-5, there is no practical reason to annoy the user when the car is driving at 0.000005 mph with no torque, who cares

I cherry picked their edits that appear in all of the custom Mazda branches. Here are a few examples:
https://github.com/Jafaral/openpilot/tree/mazda-eps22-swap-new/
https://github.com/Jafaral/openpilot/commits/mazda-0.9.9/
https://github.com/Jafaral/openpilot/commits/mazda-0.8.16

I decided not to cherry pick the edits to opendbc_repo/opendbc/car/mazda/interface.py:

- ret.dashcamOnly = candidate not in (CAR.MAZDA_CX5_2022, CAR.MAZDA_CX9_2021)
+ ret.dashcamOnly = False # candidate not in (CAR.MAZDA_CX5_2022, CAR.MAZDA_CX9_2021)

@github-actions
Copy link

github-actions bot commented Dec 7, 2025

Thank you for your PR! If you're not already in the FrogPilot Discord, feel free to join and let me know you've opened a PR!

@Ian-Boraks Ian-Boraks changed the title MAZDA: Include Jafaral's "No Warning" Changes for EPS Swaps MAZDA: Include Jafaral's "No Warning" Changes Dec 7, 2025
@Ian-Boraks Ian-Boraks force-pushed the frogpilot-make-prs-here branch from 896d382 to 99fe801 Compare December 8, 2025 00:55
@FrogAi FrogAi force-pushed the MAKE-PRS-HERE branch 17 times, most recently from d4a7400 to cc903bd Compare December 16, 2025 05:37
@FrogAi FrogAi force-pushed the MAKE-PRS-HERE branch 7 times, most recently from 0823b67 to 9f4a801 Compare January 6, 2026 06:54
@FrogAi FrogAi force-pushed the MAKE-PRS-HERE branch 5 times, most recently from 43ac771 to 1322986 Compare January 11, 2026 04:18
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.

2 participants