You can not select more than 25 topics
			Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
		
		
		
		
		
			
		
			
				
					
					
						
							261 lines
						
					
					
						
							11 KiB
						
					
					
				
			
		
		
	
	
							261 lines
						
					
					
						
							11 KiB
						
					
					
				#!/usr/bin/env python3
 | 
						|
import argparse
 | 
						|
import concurrent.futures
 | 
						|
import os
 | 
						|
import sys
 | 
						|
from collections import defaultdict
 | 
						|
from tqdm import tqdm
 | 
						|
from typing import Any, DefaultDict, Dict
 | 
						|
 | 
						|
from selfdrive.car.car_helpers import interface_names
 | 
						|
from selfdrive.test.openpilotci import get_url, upload_file
 | 
						|
from selfdrive.test.process_replay.compare_logs import compare_logs, save_log
 | 
						|
from selfdrive.test.process_replay.process_replay import CONFIGS, PROC_REPLAY_DIR, FAKEDATA, check_enabled, replay_process
 | 
						|
from system.version import get_commit
 | 
						|
from tools.lib.filereader import FileReader
 | 
						|
from tools.lib.logreader import LogReader
 | 
						|
 | 
						|
source_segments = [
 | 
						|
  ("BODY", "937ccb7243511b65|2022-05-24--16-03-09--1"),        # COMMA.BODY
 | 
						|
  ("HYUNDAI", "02c45f73a2e5c6e9|2021-01-01--19-08-22--1"),     # HYUNDAI.SONATA
 | 
						|
  ("HYUNDAI2", "d545129f3ca90f28|2022-11-07--20-43-08--3"),    # HYUNDAI.KIA_EV6 (+ QCOM GPS)
 | 
						|
  ("TOYOTA", "0982d79ebb0de295|2021-01-04--17-13-21--13"),     # TOYOTA.PRIUS (INDI)
 | 
						|
  ("TOYOTA2", "0982d79ebb0de295|2021-01-03--20-03-36--6"),     # TOYOTA.RAV4  (LQR)
 | 
						|
  ("TOYOTA3", "f7d7e3538cda1a2a|2021-08-16--08-55-34--6"),     # TOYOTA.COROLLA_TSS2
 | 
						|
  ("HONDA", "eb140f119469d9ab|2021-06-12--10-46-24--27"),      # HONDA.CIVIC (NIDEC)
 | 
						|
  ("HONDA2", "7d2244f34d1bbcda|2021-06-25--12-25-37--26"),     # HONDA.ACCORD (BOSCH)
 | 
						|
  ("CHRYSLER", "4deb27de11bee626|2021-02-20--11-28-55--8"),    # CHRYSLER.PACIFICA_2018_HYBRID
 | 
						|
  ("RAM", "2f4452b03ccb98f0|2022-09-07--13-55-08--10"),        # CHRYSLER.RAM_1500
 | 
						|
  ("SUBARU", "341dccd5359e3c97|2022-09-12--10-35-33--3"),      # SUBARU.OUTBACK
 | 
						|
  ("GM", "0c58b6a25109da2b|2021-02-23--16-35-50--11"),         # GM.VOLT
 | 
						|
  ("GM2", "376bf99325883932|2022-10-27--13-41-22--1"),         # GM.BOLT_EUV
 | 
						|
  ("NISSAN", "35336926920f3571|2021-02-12--18-38-48--46"),     # NISSAN.XTRAIL
 | 
						|
  ("VOLKSWAGEN", "de9592456ad7d144|2021-06-29--11-00-15--6"),  # VOLKSWAGEN.GOLF
 | 
						|
  ("MAZDA", "bd6a637565e91581|2021-10-30--15-14-53--4"),       # MAZDA.CX9_2021
 | 
						|
 | 
						|
  # Enable when port is tested and dashcamOnly is no longer set
 | 
						|
  #("FORD", "54827bf84c38b14f|2023-01-26--21-59-07--4"),  # FORD.BRONCO_SPORT_MK1
 | 
						|
  #("TESLA", "bb50caf5f0945ab1|2021-06-19--17-20-18--3"),      # TESLA.AP2_MODELS
 | 
						|
  #("VOLKSWAGEN2", "3cfdec54aa035f3f|2022-07-19--23-45-10--2"),  # VOLKSWAGEN.PASSAT_NMS
 | 
						|
]
 | 
						|
 | 
						|
segments = [
 | 
						|
  ("BODY", "regenFA002A80700|2022-09-27--15-37-02--0"),
 | 
						|
  ("HYUNDAI", "regenBE53A59065B|2022-09-27--16-52-03--0"),
 | 
						|
  ("HYUNDAI2", "d545129f3ca90f28|2022-11-07--20-43-08--3"),
 | 
						|
  ("TOYOTA", "regen929C5790007|2022-09-27--16-27-47--0"),
 | 
						|
  ("TOYOTA2", "regenEA3950D7F22|2022-09-27--15-43-24--0"),
 | 
						|
  ("TOYOTA3", "regen89026F6BD8D|2022-09-27--15-45-37--0"),
 | 
						|
  ("HONDA", "regenC7D5645EB17|2022-09-27--15-47-29--0"),
 | 
						|
  ("HONDA2", "regenCC2ECCE5742|2022-09-27--16-18-01--0"),
 | 
						|
  ("CHRYSLER", "regenC253C4DAC90|2022-09-27--15-51-03--0"),
 | 
						|
  ("RAM", "regen20490083AE7|2022-09-27--15-53-15--0"),
 | 
						|
  ("SUBARU", "regen1E72BBDCED5|2022-09-27--15-55-31--0"),
 | 
						|
  ("GM", "regen45B05A80EF6|2022-09-27--15-57-22--0"),
 | 
						|
  ("GM2", "376bf99325883932|2022-10-27--13-41-22--1"),
 | 
						|
  ("NISSAN", "regenC19D899B46D|2022-09-27--15-59-13--0"),
 | 
						|
  ("VOLKSWAGEN", "regenD8F7AC4BD0D|2022-09-27--16-41-45--0"),
 | 
						|
  ("MAZDA", "regenFC3F9ECBB64|2022-09-27--16-03-09--0"),
 | 
						|
]
 | 
						|
 | 
						|
# dashcamOnly makes don't need to be tested until a full port is done
 | 
						|
excluded_interfaces = ["mock", "ford", "mazda", "tesla"]
 | 
						|
 | 
						|
BASE_URL = "https://commadataci.blob.core.windows.net/openpilotci/"
 | 
						|
REF_COMMIT_FN = os.path.join(PROC_REPLAY_DIR, "ref_commit")
 | 
						|
 | 
						|
 | 
						|
def run_test_process(data):
 | 
						|
  segment, cfg, args, cur_log_fn, ref_log_path, lr_dat = data
 | 
						|
  res = None
 | 
						|
  if not args.upload_only:
 | 
						|
    lr = LogReader.from_bytes(lr_dat)
 | 
						|
    res, log_msgs = test_process(cfg, lr, segment, ref_log_path, cur_log_fn, args.ignore_fields, args.ignore_msgs)
 | 
						|
    # save logs so we can upload when updating refs
 | 
						|
    save_log(cur_log_fn, log_msgs)
 | 
						|
 | 
						|
  if args.update_refs or args.upload_only:
 | 
						|
    print(f'Uploading: {os.path.basename(cur_log_fn)}')
 | 
						|
    assert os.path.exists(cur_log_fn), f"Cannot find log to upload: {cur_log_fn}"
 | 
						|
    upload_file(cur_log_fn, os.path.basename(cur_log_fn))
 | 
						|
    os.remove(cur_log_fn)
 | 
						|
  return (segment, cfg.proc_name, cfg.subtest_name, res)
 | 
						|
 | 
						|
 | 
						|
def get_log_data(segment):
 | 
						|
  r, n = segment.rsplit("--", 1)
 | 
						|
  with FileReader(get_url(r, n)) as f:
 | 
						|
    return (segment, f.read())
 | 
						|
 | 
						|
 | 
						|
def test_process(cfg, lr, segment, ref_log_path, new_log_path, ignore_fields=None, ignore_msgs=None):
 | 
						|
  if ignore_fields is None:
 | 
						|
    ignore_fields = []
 | 
						|
  if ignore_msgs is None:
 | 
						|
    ignore_msgs = []
 | 
						|
 | 
						|
  ref_log_msgs = list(LogReader(ref_log_path))
 | 
						|
 | 
						|
  try:
 | 
						|
    log_msgs = replay_process(cfg, lr)
 | 
						|
  except Exception as e:
 | 
						|
    raise Exception("failed on segment: " + segment) from e
 | 
						|
 | 
						|
  # check to make sure openpilot is engaged in the route
 | 
						|
  if cfg.proc_name == "controlsd":
 | 
						|
    if not check_enabled(log_msgs):
 | 
						|
      return f"Route did not enable at all or for long enough: {new_log_path}", log_msgs
 | 
						|
 | 
						|
  try:
 | 
						|
    return compare_logs(ref_log_msgs, log_msgs, ignore_fields + cfg.ignore, ignore_msgs, cfg.tolerance, cfg.field_tolerances), log_msgs
 | 
						|
  except Exception as e:
 | 
						|
    return str(e), log_msgs
 | 
						|
 | 
						|
 | 
						|
def format_diff(results, log_paths, ref_commit):
 | 
						|
  diff1, diff2 = "", ""
 | 
						|
  diff2 += f"***** tested against commit {ref_commit} *****\n"
 | 
						|
 | 
						|
  failed = False
 | 
						|
  for segment, result in list(results.items()):
 | 
						|
    diff1 += f"***** results for segment {segment} *****\n"
 | 
						|
    diff2 += f"***** differences for segment {segment} *****\n"
 | 
						|
 | 
						|
    for proc, diff in list(result.items()):
 | 
						|
      # long diff
 | 
						|
      diff2 += f"*** process: {proc} ***\n"
 | 
						|
      diff2 += f"\tref: {log_paths[segment][proc]['ref']}\n"
 | 
						|
      diff2 += f"\tnew: {log_paths[segment][proc]['new']}\n\n"
 | 
						|
 | 
						|
      # short diff
 | 
						|
      diff1 += f"    {proc}\n"
 | 
						|
      if isinstance(diff, str):
 | 
						|
        diff1 += f"        ref: {log_paths[segment][proc]['ref']}\n"
 | 
						|
        diff1 += f"        new: {log_paths[segment][proc]['new']}\n\n"
 | 
						|
        diff1 += f"        {diff}\n"
 | 
						|
        failed = True
 | 
						|
      elif len(diff):
 | 
						|
        diff1 += f"        ref: {log_paths[segment][proc]['ref']}\n"
 | 
						|
        diff1 += f"        new: {log_paths[segment][proc]['new']}\n\n"
 | 
						|
 | 
						|
        cnt: Dict[str, int] = {}
 | 
						|
        for d in diff:
 | 
						|
          diff2 += f"\t{str(d)}\n"
 | 
						|
 | 
						|
          k = str(d[1])
 | 
						|
          cnt[k] = 1 if k not in cnt else cnt[k] + 1
 | 
						|
 | 
						|
        for k, v in sorted(cnt.items()):
 | 
						|
          diff1 += f"        {k}: {v}\n"
 | 
						|
        failed = True
 | 
						|
  return diff1, diff2, failed
 | 
						|
 | 
						|
 | 
						|
if __name__ == "__main__":
 | 
						|
  all_cars = {car for car, _ in segments}
 | 
						|
  all_procs = {cfg.proc_name for cfg in CONFIGS}
 | 
						|
 | 
						|
  parser = argparse.ArgumentParser(description="Regression test to identify changes in a process's output")
 | 
						|
  parser.add_argument("--whitelist-procs", type=str, nargs="*", default=all_procs,
 | 
						|
                      help="Whitelist given processes from the test (e.g. controlsd)")
 | 
						|
  parser.add_argument("--whitelist-cars", type=str, nargs="*", default=all_cars,
 | 
						|
                      help="Whitelist given cars from the test (e.g. HONDA)")
 | 
						|
  parser.add_argument("--blacklist-procs", type=str, nargs="*", default=[],
 | 
						|
                      help="Blacklist given processes from the test (e.g. controlsd)")
 | 
						|
  parser.add_argument("--blacklist-cars", type=str, nargs="*", default=[],
 | 
						|
                      help="Blacklist given cars from the test (e.g. HONDA)")
 | 
						|
  parser.add_argument("--ignore-fields", type=str, nargs="*", default=[],
 | 
						|
                      help="Extra fields or msgs to ignore (e.g. carState.events)")
 | 
						|
  parser.add_argument("--ignore-msgs", type=str, nargs="*", default=[],
 | 
						|
                      help="Msgs to ignore (e.g. carEvents)")
 | 
						|
  parser.add_argument("--update-refs", action="store_true",
 | 
						|
                      help="Updates reference logs using current commit")
 | 
						|
  parser.add_argument("--upload-only", action="store_true",
 | 
						|
                      help="Skips testing processes and uploads logs from previous test run")
 | 
						|
  parser.add_argument("-j", "--jobs", type=int, default=1)
 | 
						|
  args = parser.parse_args()
 | 
						|
 | 
						|
  tested_procs = set(args.whitelist_procs) - set(args.blacklist_procs)
 | 
						|
  tested_cars = set(args.whitelist_cars) - set(args.blacklist_cars)
 | 
						|
  tested_cars = {c.upper() for c in tested_cars}
 | 
						|
 | 
						|
  full_test = (tested_procs == all_procs) and (tested_cars == all_cars) and all(len(x) == 0 for x in (args.ignore_fields, args.ignore_msgs))
 | 
						|
  upload = args.update_refs or args.upload_only
 | 
						|
  os.makedirs(os.path.dirname(FAKEDATA), exist_ok=True)
 | 
						|
 | 
						|
  if upload:
 | 
						|
    assert full_test, "Need to run full test when updating refs"
 | 
						|
 | 
						|
  try:
 | 
						|
    ref_commit = open(REF_COMMIT_FN).read().strip()
 | 
						|
  except FileNotFoundError:
 | 
						|
    print("Couldn't find reference commit")
 | 
						|
    sys.exit(1)
 | 
						|
 | 
						|
  cur_commit = get_commit()
 | 
						|
  if cur_commit is None:
 | 
						|
    raise Exception("Couldn't get current commit")
 | 
						|
 | 
						|
  print(f"***** testing against commit {ref_commit} *****")
 | 
						|
 | 
						|
  # check to make sure all car brands are tested
 | 
						|
  if full_test:
 | 
						|
    untested = (set(interface_names) - set(excluded_interfaces)) - {c.lower() for c in tested_cars}
 | 
						|
    assert len(untested) == 0, f"Cars missing routes: {str(untested)}"
 | 
						|
 | 
						|
  log_paths: DefaultDict[str, Dict[str, Dict[str, str]]] = defaultdict(lambda: defaultdict(dict))
 | 
						|
  with concurrent.futures.ProcessPoolExecutor(max_workers=args.jobs) as pool:
 | 
						|
    if not args.upload_only:
 | 
						|
      download_segments = [seg for car, seg in segments if car in tested_cars]
 | 
						|
      log_data: Dict[str, LogReader] = {}
 | 
						|
      p1 = pool.map(get_log_data, download_segments)
 | 
						|
      for segment, lr in tqdm(p1, desc="Getting Logs", total=len(download_segments)):
 | 
						|
        log_data[segment] = lr
 | 
						|
 | 
						|
    pool_args: Any = []
 | 
						|
    for car_brand, segment in segments:
 | 
						|
      if car_brand not in tested_cars:
 | 
						|
        continue
 | 
						|
 | 
						|
      for cfg in CONFIGS:
 | 
						|
        if cfg.proc_name not in tested_procs:
 | 
						|
          continue
 | 
						|
 | 
						|
        cur_log_fn = os.path.join(FAKEDATA, f"{segment}_{cfg.proc_name}{cfg.subtest_name}_{cur_commit}.bz2")
 | 
						|
        if args.update_refs:  # reference logs will not exist if routes were just regenerated
 | 
						|
          ref_log_path = get_url(*segment.rsplit("--", 1))
 | 
						|
        else:
 | 
						|
          ref_log_fn = os.path.join(FAKEDATA, f"{segment}_{cfg.proc_name}{cfg.subtest_name}_{ref_commit}.bz2")
 | 
						|
          ref_log_path = ref_log_fn if os.path.exists(ref_log_fn) else BASE_URL + os.path.basename(ref_log_fn)
 | 
						|
 | 
						|
        dat = None if args.upload_only else log_data[segment]
 | 
						|
        pool_args.append((segment, cfg, args, cur_log_fn, ref_log_path, dat))
 | 
						|
 | 
						|
        log_paths[segment][cfg.proc_name + cfg.subtest_name]['ref'] = ref_log_path
 | 
						|
        log_paths[segment][cfg.proc_name + cfg.subtest_name]['new'] = cur_log_fn
 | 
						|
 | 
						|
    results: Any = defaultdict(dict)
 | 
						|
    p2 = pool.map(run_test_process, pool_args)
 | 
						|
    for (segment, proc, subtest_name, result) in tqdm(p2, desc="Running Tests", total=len(pool_args)):
 | 
						|
      if not args.upload_only:
 | 
						|
        results[segment][proc + subtest_name] = result
 | 
						|
 | 
						|
  diff1, diff2, failed = format_diff(results, log_paths, ref_commit)
 | 
						|
  if not upload:
 | 
						|
    with open(os.path.join(PROC_REPLAY_DIR, "diff.txt"), "w") as f:
 | 
						|
      f.write(diff2)
 | 
						|
    print(diff1)
 | 
						|
 | 
						|
    if failed:
 | 
						|
      print("TEST FAILED")
 | 
						|
      print("\n\nTo push the new reference logs for this commit run:")
 | 
						|
      print("./test_processes.py --upload-only")
 | 
						|
    else:
 | 
						|
      print("TEST SUCCEEDED")
 | 
						|
 | 
						|
  else:
 | 
						|
    with open(REF_COMMIT_FN, "w") as f:
 | 
						|
      f.write(cur_commit)
 | 
						|
    print(f"\n\nUpdated reference logs for commit: {cur_commit}")
 | 
						|
 | 
						|
  sys.exit(int(failed))
 | 
						|
 |