mawei/sunnypilot-tesla
1
0
Fork 0
mirror of https://github.com/dzid26/sunnypilot.git synced 2026-06-08 07:44:55 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix compare_logs crash on list field size changes (#37829)

Browse Source 
fix crash on list-length changes
This commit is contained in:
Daniel Koepping
2026-04-14 20:18:31 -07:00
committed by GitHub
parent e16d422cf4
commit d5f1d8c33a
2 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
selfdrive/test/process_replay/compare_logs.py
View File

@@ -76,7 +76,7 @@ def _diff_capnp_values(v1, v2, path, tolerance):
for i in range(n):
yield from _diff_capnp_values(v1[i], v2[i], path + (str(i),), tolerance)
if n2 > n:
yield 'add', dot, list(enumerate(v2[n:], n))
yield 'add', dot, [(i, v2[i]) for i in range(n, n2)]
if n1 > n:
yield 'remove', dot, list(reversed([(i, v1[i]) for i in range(n, n1)]))

2
selfdrive/test/process_replay/diff_report.py
View File

@@ -49,6 +49,8 @@ def diff_format(diffs, ref, new, field) -> list[str]:
msg_type = field.split(".")[0]
ref_ts = [(m.logMonoTime, MsgWrap(m)) for m in ref.get(msg_type, [])]
new_wrapped = [MsgWrap(m) for m in new.get(msg_type, [])]
if not ref_ts or not new_wrapped:
return format_numeric_diffs(diffs)
return format_diff(diffs, ref_ts, new_wrapped, field)