Skip to content

Nissan: FW query tweaks for 2023 Nissan Altima#3202

Draft
royjr wants to merge 12 commits intocommaai:masterfrom
royjr:fp-NISSAN_ALTIMA
Draft

Nissan: FW query tweaks for 2023 Nissan Altima#3202
royjr wants to merge 12 commits intocommaai:masterfrom
royjr:fp-NISSAN_ALTIMA

Conversation

@royjr
Copy link
Copy Markdown
Contributor

@royjr royjr commented Mar 12, 2026

Nissan Altima 2023
Nissan B
1414bd4bad5acc8f/00000007--2e04252f32

@github-actions github-actions Bot added car related to opendbc/car/ nissan labels Mar 12, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 12, 2026

Car behavior report

Replays driving segments through this PR and compares the behavior to master.
Please review any changes carefully to ensure they are expected.

Testing 50 segments for: NISSAN_XTRAIL, NISSAN_LEAF, NISSAN_LEAF_IC, NISSAN_ROGUE, NISSAN_ALTIMA

✅ 0 changed, 50 passed, 0 errors

@royjr royjr marked this pull request as ready for review March 13, 2026 02:14
@royjr royjr changed the title Nissan: add FW for Nissan Altima 2023 Nissan: Nissan Altima 2023 Fingerprint Mar 17, 2026
@jyoung8607 jyoung8607 changed the title Nissan: Nissan Altima 2023 Fingerprint Nissan: FW query tweaks for 2023 Nissan Altima Mar 26, 2026
@jyoung8607 jyoung8607 requested a review from sshane March 26, 2026 23:14
@jyoung8607
Copy link
Copy Markdown
Collaborator

@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.

@sshane
Copy link
Copy Markdown
Contributor

sshane commented Mar 27, 2026

All the Nissan variants responded slightly differently to requests afaik. This only adds a request, probably fine but letr me check it out

Comment thread opendbc/car/nissan/values.py Outdated

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 [
Copy link
Copy Markdown
Contributor

@sshane sshane Mar 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 is w/ OBD port, we want to only FP at camera commaai/openpilot#31243

@sshane
Copy link
Copy Markdown
Contributor

sshane commented Mar 27, 2026

@sshane
Copy link
Copy Markdown
Contributor

sshane commented Mar 27, 2026

@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?

sendcan: rxaddr=1858, bus=0,   1936 ms, 0x023e000000000000, b'\x02>\x00\x00\x00\x00\x00\x00', len(can.dat)=8
sendcan: rxaddr=1858, bus=1,   1936 ms, 0x023e000000000000, b'\x02>\x00\x00\x00\x00\x00\x00', len(can.dat)=8

    can: rxaddr=1858, bus=128, 1946 ms, 0x023e000000000000, b'\x02>\x00\x00\x00\x00\x00\x00', len(can.dat)=8
    can: rxaddr=1858, bus=2,   1946 ms, 0x023e000000000000, b'\x02>\x00\x00\x00\x00\x00\x00', len(can.dat)=8

    can: rxaddr=1858, bus=129, 1957 ms, 0x023e000000000000, b'\x02>\x00\x00\x00\x00\x00\x00', len(can.dat)=8
    can: rxaddr=1890, bus=1,   1957 ms, 0x037f3e1200000000, b'\x03\x7f>\x12\x00\x00\x00\x00', len(can.dat)=8

sendcan: rxaddr=1858, bus=0,   2483 ms, 0x0210810000000000, b'\x02\x10\x81\x00\x00\x00\x00\x00', len(can.dat)=8

    can: rxaddr=1858, bus=128, 2508 ms, 0x0210810000000000, b'\x02\x10\x81\x00\x00\x00\x00\x00', len(can.dat)=8
    can: rxaddr=1858, bus=2,   2508 ms, 0x0210810000000000, b'\x02\x10\x81\x00\x00\x00\x00\x00', len(can.dat)=8

@sshane sshane marked this pull request as draft March 27, 2026 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

car related to opendbc/car/ nissan

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants