opendbc icon indicating copy to clipboard operation
opendbc copied to clipboard

2024 Elantra car Unrecognized

Open 1okko opened this issue 1 year ago • 21 comments

Describe the bug

2024 Elantra car Unrecognized

Which car does this affect?

2024 HYUNDAI Elantra

Provide a route where the issue occurs

d01e5725e5587ae3/00000001--a4126f0d93/0

openpilot version

0.9.7

Additional info

It can be confirmed that the vehicle is a CAN and a camera SCC

1okko avatar May 06 '24 02:05 1okko

Has a CAN FD radar bus. Please try this branch: https://github.com/commaai/openpilot/tree/cn7c-elantra

sshane avatar May 06 '24 19:05 sshane

ok.i will try and let you know.

1okko avatar May 07 '24 00:05 1okko

d01e5725e5587ae3/00000000--178533cb2e/1

it works well,thanks

1okko avatar May 08 '24 00:05 1okko

拥有CAN FD雷达总线。请尝试这个分支:https://github.com/commaai/openpilot/tree/cn7c-elantra

d01e5725e5587ae3/00000000--178533cb2e/1

效果很好,谢谢

1okko avatar May 08 '24 03:05 1okko

d01e5725e5587ae3/00000000--178533cb2e/1

it works well,thanks

d01e5725e5587ae3/00000000--178533cb2e/1

it works well,thanks

1okko avatar May 08 '24 03:05 1okko

Nice! Can you drive more to gather more lateral control data? I'll get https://github.com/commaai/openpilot/pull/30219 cleaned up and merged shortly after once we verify that lateral control is as we expect (not too much or too little lateral acceleration)

sshane avatar May 08 '24 09:05 sshane

d01e5725e5587ae3/00000000--178533cb2e/1

d01e5725e5587ae3/00000000--178533cb2e/1 this not enough?

1okko avatar May 08 '24 11:05 1okko

Need some more curves I believe!

sshane avatar May 09 '24 01:05 sshane

Need some more curves I believe!

ok, i will drive it today

1okko avatar May 09 '24 03:05 1okko

Need some more curves I believe!

d01e5725e5587ae3/00000002--3ec9e0ec2a/0

1okko avatar May 10 '24 12:05 1okko

High speed (> 80 km/h) and engaged curves I mean

sshane avatar May 11 '24 00:05 sshane

@1okko can you get more driving data please? Then we can merge!

sshane avatar May 22 '24 00:05 sshane

@1okko can you get more driving data please? Then we can merge!

d01e5725e5587ae3/00000003--ecb5e1500a/2

1okko avatar May 25 '24 06:05 1okko

Dongle, platform, VIN Name from VIN 🆚 CarDocs Segments Bad seg tags Good seg tags
d01e5725e5587ae3
HYUNDAI_ELANTRA_2021
000000000........
None 🆚
None
4 routes,
94 segments
(1.6 hours)
- all lat active: 51
- no input all lat active: 42

sshane avatar May 25 '24 09:05 sshane

Need a little more sharper turns at high speeds, the torqued learner doesn't have many data points there.

sshane avatar May 25 '24 09:05 sshane

Need a little more sharper turns at high speeds, the torqued learner doesn't have many data points there.

ok

1okko avatar May 25 '24 09:05 1okko

Need a little more sharper turns at high speeds, the torqued learner doesn't have many data points there.

d01e5725e5587ae3/00000004--9eae7430ad/0

1okko avatar Jun 01 '24 10:06 1okko

Dongle, platform, VIN Name from VIN 🆚 CarDocs Segments Bad seg tags Good seg tags
d01e5725e5587ae3
HYUNDAI_ELANTRA_2021
000000000........
None 🆚
None
6 routes,
181 segments
(3.0 hours)
- high lateral accel factor: 1 - all lat active: 87
- no input all lat active: 57
- valid torque params: 78

It starts high (still learning), but settles into 3.5 and falling (still hasn't fully learned), so this looks expected.

image

sshane avatar Jun 04 '24 23:06 sshane

I forgot this port requires changing the length of LFAHDA_MFC, we'll have to put this off for now until we have time for a car porting sprint. Feel free to keep driving on the branch!

sshane avatar Jun 04 '24 23:06 sshane

I forgot this port requires changing the length of LFAHDA_MFC, we'll have to put this off for now until we have time for a car porting sprint. Feel free to keep driving on the branch!

How can I test this fork on c3+red panda kit?

1okko avatar Jun 15 '24 06:06 1okko

Is this ready yet? I just got a Hyundai Elantra Hybrid 2024 and looking to test this out or help out if needed, lmk.

0xDeadcell avatar Sep 15 '24 04:09 0xDeadcell