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 |
jyoung8607
changed the title
|
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 |
* init * fix * add test route * fix E501 Line too long (167 > 160) * use the new generic SecOC carPlatform --------- Co-authored-by: Jason Young <[email protected]>
* Revert "Toyota: Yaris SecOC is hybrid (commaai#1641)" This reverts commit 3623ea8. * Revert "Toyota: JDM Yaris with SecOC (commaai#1578)" This reverts commit 63d039a.
4 participants
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