Nissan: FW query tweaks for 2023 Nissan Altima#3202
Nissan: FW query tweaks for 2023 Nissan Altima#3202royjr wants to merge 12 commits intocommaai:masterfrom
Conversation
Car behavior reportReplays driving segments through this PR and compares the behavior to master. Testing 50 segments for: NISSAN_XTRAIL, NISSAN_LEAF, NISSAN_LEAF_IC, NISSAN_ROGUE, NISSAN_ALTIMA ✅ 0 changed, 50 passed, 0 errors |
|
@sshane This looks correct to me, insofar as it matches the data we're able to get back to the car, but I'm not sure how this was working for other Nissan without the change. There's a MY 2022 facelift of the existing 6th gen cars we support, but allegedly we already support a MY2024, so I'm not sure if the behavior change is related. This PR seems fine to me, but I have no way to validate the change against other Nissan in bulk. |
|
All the Nissan variants responded slightly differently to requests afaik. This only adds a request, probably fine but letr me check it out |
|
|
||
| FW_QUERY_CONFIG = FwQueryConfig( | ||
| requests=[request for bus, logging in ((0, False), (1, True)) for request in [ | ||
| requests=[request for bus, logging in ((0, False), (1, False)) for request in [ |
There was a problem hiding this comment.
1 is w/ OBD port, we want to only FP at camera commaai/openpilot#31243
|
@royjr can you have user try this? very strange nothing returns without comma power on bus 0 since all the other Nissans work. Is this the normal harness setup? |
Nissan Altima 2023
Nissan B
1414bd4bad5acc8f/00000007--2e04252f32