Commit Graph

33 Commits

Author SHA1 Message Date
Shane Smiskol
280f70ada5 Revert "IsoTpParallelQuery: extend timeout for every frame type" (#29693)
* Revert "IsoTpParallelQuery: extend timeout for every frame type (#29654)"

This reverts commit bbde2143a809048ff9e5cf1a0deef6b37ee4c9c9.

* bump

* bump to master

* figure out a clean way to exclude segments

* revert

* bump
old-commit-hash: ddddc7a8cb
2023-08-28 23:27:58 -07:00
Shane Smiskol
2c86b69c92 IsoTpParallelQuery: extend timeout for every frame type (#29654)
* recv returns updated

* bump panda

* bump

* bump
old-commit-hash: a160f1a624
2023-08-26 07:00:46 -07:00
Adeeb Shihadeh
16ec56398c add openpilot prefix to imports (#29498)
* add openpilot prefix to imports

* more

* more

* fix docs

* fix linter

* bump submodules

* fix patched tests

* update dynamic imports

* debug

* Revert "debug"

This reverts commit db5e13b9911cc74438bee123bc3430da6c31b24b.

* fix pm test
old-commit-hash: a9626f95b6
2023-08-20 20:49:55 -07:00
Shane Smiskol
e5e1fd12b0 IsoTpParallelQuery: log empty responses (#29014)
* check if None

* log instead

* clean up

* should mark done
old-commit-hash: 29837b8728
2023-07-18 03:43:56 -07:00
Shane Smiskol
5958d046ac IsoTpParallelQuery: remove spammy log (#28372)
* remove

* just a warning

* just comment out for now
old-commit-hash: 96905e52e2
2023-06-02 14:24:54 -07:00
Shane Smiskol
6064fbb5e3 IsoTpParallelQuery: log all timeouts (#28325)
* log

* self explan

* clean up

* revert

* need a better name

* move

* bvack
old-commit-hash: 026c093d05
2023-05-27 22:14:50 -07:00
Shane Smiskol
000db65838 IsoTpParallelQuery: fixup comment and simpler response check (#28326)
* startswith works with bytes

* this comment only explained an exception, was confusing
old-commit-hash: aa3490ac30
2023-05-27 19:25:38 -07:00
Shane Smiskol
09df1f86b2 IsoTpParallelQuery: set timed out requests done (#27764)
* better ordering and comments

* remove

* mark done if timed out

* response_timeouts should be only for extending timeouts, not marking done

* so the print still works

* more clear comment

* that can be removed
old-commit-hash: f5519fdca8
2023-04-11 22:05:00 -07:00
Shane Smiskol
b1bd07ea75 IsoTpParallelQuery: break earlier and add comment (#27883)
* better ordering and comments

* one comment one dream
old-commit-hash: 74f709a59f
2023-04-11 21:05:07 -07:00
Shane Smiskol
3baa87d208 IsoTpParallelQuery: log response pending (#27881)
we really should log this
old-commit-hash: 02798513af
2023-04-11 20:59:03 -07:00
Shane Smiskol
b5d5f59260 IsoTpParallelQuery: extend timeouts when receiving consecutive frames (#27762)
* change to rx_in_progress

* bump panda

* bump panda
old-commit-hash: a0f33658f2
2023-03-31 20:57:32 -07:00
Dean Lee
a837c85ec3 isotp_parallel_query: use drain_sock_raw (#27714)
use drain_sock_raw
old-commit-hash: fb03e225fd
2023-03-28 00:25:20 -07:00
Shane Smiskol
aa5deae8f9 IsoTpParallelQuery: log errors in qlogs (#26609)
log these in qlogs
old-commit-hash: 08b49e5671
2022-11-28 15:00:37 -08:00
Shane Smiskol
098ef11824 IsoTpParallelQuery: set separation time (#25978)
* Specify separation time in openpilot

* comment

* Update selfdrive/car/isotp_parallel_query.py

* Update selfdrive/car/isotp_parallel_query.py
old-commit-hash: 8b41d817e4
2022-10-05 17:06:52 -07:00
Shane Smiskol
bfc4d72402 IsoTpParallelQuery: don't return rx_addr (#25934)
* revert isotpparallelquery returning rx addr for functional special case

* we don't really use the tx addr (and soon won't make sense with fun querying)
old-commit-hash: 4d7f4b4c9d
2022-09-30 16:01:22 -07:00
Shane Smiskol
02428ca744 IsoTpParallelQuery: process all functional responses (#25939)
* stash

* process all functional addrs (stash)

* clean up

* rm

* simplify

* let user pass in tx for rx addrs

* revert panda

* simplify

* comment

order

* need to go by rx_addr now

* Revert "need to go by rx_addr now"

This reverts commit 1197ecfbc5b9e5df20b523a0623f644cd8cae1ef.

* stash

* should also work

* this seems pretty clean

* not used

* properly use

* comment

* some fixes

* some fixes

* send consecutive frames on physical addrs

* bump panda

* looks better

* setup_only

* Revert VIN changes

* rev

* bump panda to master

* Update selfdrive/car/isotp_parallel_query.py
old-commit-hash: bea960675f
2022-09-30 15:15:13 -07:00
Shane Smiskol
d7acc9fe75 Revert "IsoTpParallelQuery: process all functional responses (#25930)"
This reverts commit 39e98b0456047adc253c611b1015fd1aa38f6dd2.

old-commit-hash: 9e2cca23cf
2022-09-29 21:15:34 -07:00
Shane Smiskol
f3211a9dfb IsoTpParallelQuery: process all functional responses (#25930)
* Revert "VIN: query physical addresses (#25122)"

This reverts commit 452c155a49b068891937450b7389a2154881160a.

* try sending tester present

* do CAN fingerprinting first

* looks like we can get rid of this!

* remove import

* no cache for testing

* revert

* revert

* move function to fw_versions

* Exception

* Revert fp order, sleep to let PubSocket connect

* comment

comment

* space

* Update selfdrive/car/car_helpers.py

* at 0.06 is where it becomes more consistent

* treat functional addrs like physical addrs (process all responses)

* fixes and debugging

* fix

* revert other changes

* Update selfdrive/car/isotp_parallel_query.py

* caps
old-commit-hash: c5514f3440
2022-09-29 19:07:28 -07:00
Shane Smiskol
b0d03acf2a IsoTpParallelQuery: extend timeout for each ISO-TP frame (#25897)
* raise timeouts

* extend timeout for each message/frame (not multi-frame full response)

* bump panda

* 100 ms timeout between frames (max I've seen is 20, should be good)

* bump panda

* remove unused

* Add tester present to HKG queries (temp)

* send tester present to all ecus first

* vin and fw_versions.py sleep, should figure out why sendcan drops packets

* None is wildcard (some ecus respond with negative code, that's fine)

* typing

* try bus 0

* revert brand-specific changes

* Update selfdrive/car/isotp_parallel_query.py
old-commit-hash: 870c5f383d
2022-09-27 18:13:13 -07:00
Shane Smiskol
a27814c513 Log tx addr for IsoTpMessage exceptions
old-commit-hash: 29b9a07393
2022-09-26 23:22:50 -07:00
Jason Young
814afc5308 Clean up timers on receipt of a non-0x78 UDS error (#25398)
* clean up timers on receipt of a non-0x78 UDS error

* Update selfdrive/car/isotp_parallel_query.py

Co-authored-by: Shane Smiskol <shane@smiskol.com>
old-commit-hash: 5d699a578c
2022-08-11 15:24:23 -07:00
Shane Smiskol
661655b4cf Log VIN response address (#25148)
* log vin rx addr

* clean up
old-commit-hash: 3a8f17111c
2022-07-13 01:10:56 -07:00
Adeeb Shihadeh
3c48bc47b7 Move a bunch of stuff to system/ part 3 (#24829)
* move swaglog.py

* timezoned

* logmessaged

* version.py

* fix linter
old-commit-hash: 0fce5d9045
2022-06-11 23:19:27 -07:00
Shane Smiskol
fa9528e119 IsoTpParallelQuery: handle response pending (#24724)
* handle response pending

* match commit

* remove total timeout, just keep track of individual response timeouts

* fix

* add back total timeout

* this isn't reliable enough

* keep track of pending responses to print warning

* tx_addr is (addr, subaddr)

* debug

only query hyundai

import time

reponse pending

no cache

all cars

no timeout to test before

* Revert "debug"

This reverts commit abe9cfc1b668034d7fa5ca5cbe9efe8834db3e7b.

* always print pending

always debug

* Only debug

* Update selfdrive/car/isotp_parallel_query.py

* remove variable only for debugging
old-commit-hash: 30e1f28213
2022-06-07 17:49:07 -07:00
Shane Smiskol
f9772e624b FPv2: log query request and response address (#24733)
* log fw query response offset

* one line still

* log request bytes as well

* bump cereal

* prettier order

* bump cereal

* I wonder if the radar sends a response pending...

debug

dont cache

fix

debug

* Revert "I wonder if the radar sends a response pending..."

This reverts commit 12a9552c27ac427c161c27f916fb4fd1a8e4b44c.

* log addr too

* bump cereal
old-commit-hash: 124f3f02d5
2022-06-03 12:05:13 -07:00
Willem Melching
82f0522647 FW query: catch asserts from uds.py (#22456)
old-commit-hash: 95957bc188
2021-10-07 10:34:59 +02:00
Willem Melching
03da450865 Improve Hyundai FW query stability (#22344)
* extend timeout on valid response

* extend total timeout in fw query

* redundant query

* cleanup

* add 2 new sonata fw

* add cloudlog

* total timeout always 10x

* update ref

* time.monotonic()

* more logging
old-commit-hash: 2d7d8fba99
2021-09-24 17:32:09 -07:00
Josh Smith
2cae3a3799 Add type hints, small cleanups (#21080)
* improve tools.lib.kbhit and tools.sim.lib.keyboard_ctrl

* unpack more efficiently

* minor improvements

* agnos.py match spec better

* manual_ctrl test missing queue arg

* fix incorrect type annotation

* queues are generic

* varname reuse resulting in incorrect type inference

* bytes().hex() rather than bytes.hex(bytes())

* a bit of type hinting stuff
old-commit-hash: 77321dbac4
2021-06-03 12:21:04 +02:00
Jason Young
76fc2d5206 VW MQB: UDS fingerprinting support (#20271)
* FPv2 support for MQB

* Comment touch-ups

* Trim extra newlines

* Add Atlas and Tiguan

* Update submodule ref

* Revert "Add Atlas and Tiguan"

This reverts commit a47bc1bca7a5b72df44578051379859eefa5fb39.

* Curb my enthusiasm

* Drop ABS/ESP from UDS fingerprinting for now

* Use multi-query to grab SW build ID and parameterization

* Resolve merge oops

* Resolve merge conflicts

* Make Ecu.fwdCamera optional for VW

* Where'd that come from?

* Propitiate process replay checks until ready for model dump

* FW values for Audi A3 Prestige

* Revert "Make Ecu.fwdCamera optional for VW"

This reverts commit 56852b99

* Don't try to FP fwdCamera for VW

* Remove FW comments

* Simplify RX offset handling

* Corrected scoping for MQB trans detection

* Add default response_offset

* Remove explicit list of MQB vehicles for now

* Deal with pylint warning

* Clarify and simplify comments

Co-authored-by: Comma Device <device@comma.ai>
Co-authored-by: Adeeb Shihadeh <adeebshihadeh@gmail.com>
old-commit-hash: e7fc11c7fb
2021-03-25 16:06:59 -07:00
Adeeb
9027018368 Update Flake8 config (#1624)
* update flake8 checks

* add E502

* no whitespace warnings

* fix violations

* no W391

Co-authored-by: Jason Young <jason@comma.ai>
old-commit-hash: 367155168a
2020-06-03 16:13:34 -07:00
George Hotz
9ef085076b selfdrive/car
old-commit-hash: 71ead9adea
2020-01-17 10:58:43 -08:00
George Hotz
81dd5a50cf root commit
old-commit-hash: 6c33a5c1f3
2020-01-17 10:02:52 -08:00
Vehicle Researcher
fa4b6f33d1 openpilot v0.7.1 release
old-commit-hash: 8da8b6135c
2020-01-15 14:05:04 -08:00