Skip to content

Mazda: use torque controller - #25265

Merged
adeebshihadeh merged 4 commits into
masterfrom
mazda-torque
Jul 25, 2022
Merged

Mazda: use torque controller#25265
adeebshihadeh merged 4 commits into
masterfrom
mazda-torque

Conversation

@adeebshihadeh

@adeebshihadeh adeebshihadeh commented Jul 23, 2022

Copy link
Copy Markdown
Contributor

Need to test on CX5 and CX9, rest are dashcam only.

@adeebshihadeh adeebshihadeh added tuning car vehicle-specific labels Jul 23, 2022
@alancyau

Copy link
Copy Markdown
Contributor

Here’s a log from a 2022 Mazda CX-5 running this branch.

a4af1602d8e668ac|2022-07-23--23-17-42--0

@Jafaral

Jafaral commented Jul 25, 2022

Copy link
Copy Markdown
Contributor

CX-9 2021 : 8c521516df3527d8|2022-07-24--18-57-08

clips 2 through 10

@adeebshihadeh

Copy link
Copy Markdown
Contributor Author

Both of your plots look pretty good. How does it feel compared to the current tuning on master?

@Jafaral

Jafaral commented Jul 25, 2022

Copy link
Copy Markdown
Contributor

The drive was smooth overall. I did not test current master to make a direct comparison.

@adeebshihadeh
adeebshihadeh marked this pull request as ready for review July 25, 2022 20:54
@alancyau

Copy link
Copy Markdown
Contributor

Smooth overall. Pretty similar to master. It seems like lane change might be a bit jerky though.

@adeebshihadeh
adeebshihadeh merged commit 3474553 into master Jul 25, 2022
@adeebshihadeh
adeebshihadeh deleted the mazda-torque branch July 25, 2022 22:16
Jafaral pushed a commit to Jafaral/openpilot that referenced this pull request Jul 26, 2022
* Mazda: use torque controller
66thparallel pushed a commit to 66thparallel/janes_openpilot_fork that referenced this pull request May 1, 2026
* Mazda: use torque controller

* update refs

* update refs
old-commit-hash: 3474553
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

car vehicle-specific tuning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants