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

Toyota: japanese Yaris basic car port #1578

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

AlexandreSato
Copy link
Contributor

@AlexandreSato AlexandreSato commented Dec 14, 2024

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

@github-actions github-actions bot added car related to opendbc/car/ toyota labels Dec 14, 2024
nelsonjchen added a commit to optskug/docs that referenced this pull request Dec 14, 2024
@jyoung8607
Copy link
Collaborator

Is this the one that needs the radar points parser disabled?

@AlexandreSato
Copy link
Contributor Author

AlexandreSato commented Dec 14, 2024

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.

@AlexandreSato AlexandreSato marked this pull request as ready for review December 15, 2024 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
car related to opendbc/car/ toyota
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants