-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Toyota: JDM Yaris with SecOC #1578
Conversation
Is this the one that needs the radar points parser disabled? |
Yes is a continental new radar! We also tried the approach of disabling the radar TX to gain longitudinal control, but even though the radar TX actually stops and we correctly create the 0x183 messages, the car enters a cruise fault state when trying to engage OpenPilot. |
|
0b54d0594d924cd9/000000d3--aa729fc253/9 |
It looks like the basics are working, but I'm not sure about the lateral control tuning parameters. It's hard to evaluate given the relatively low speed driving in the test route coupled with the model that was in master at the time. Given the special treatment of SecOC cars, we can merge now and revisit the tune later if needed. |
This seems to be the hybrid variant? Can you fix the docs, or are all Yaris hybrid now? |
Yes this is hybrid! And Yaris ICE exists in japan, my bad. Opened: #1641 |
The Japanese toyota yaris has the port (lateral only) almost ready, I believe that just like the rav4 prime should remain excluded from cardocs?
0b54d0594d924cd9/00000041--51e7d1ea2e
user https://github.com/gonzalles125
discord gon0822