VehicleModel and controller still takes capnp CP since they get it from Params()

pull/33208/head
Shane Smiskol 11 months ago
parent b8a8b4349a
commit d4700d1ba9
  1. 2
      selfdrive/controls/lib/tests/test_latcontrol.py
  2. 3
      selfdrive/controls/lib/tests/test_vehicle_model.py

@ -1,6 +1,7 @@
from parameterized import parameterized from parameterized import parameterized
from cereal import car, log from cereal import car, log
from openpilot.selfdrive.car.card import convert_to_capnp
from openpilot.selfdrive.car.car_helpers import interfaces from openpilot.selfdrive.car.car_helpers import interfaces
from openpilot.selfdrive.car.honda.values import CAR as HONDA from openpilot.selfdrive.car.honda.values import CAR as HONDA
from openpilot.selfdrive.car.toyota.values import CAR as TOYOTA from openpilot.selfdrive.car.toyota.values import CAR as TOYOTA
@ -19,6 +20,7 @@ class TestLatControl:
CarInterface, CarController, CarState = interfaces[car_name] CarInterface, CarController, CarState = interfaces[car_name]
CP = CarInterface.get_non_essential_params(car_name) CP = CarInterface.get_non_essential_params(car_name)
CI = CarInterface(CP, CarController, CarState) CI = CarInterface(CP, CarController, CarState)
CP = convert_to_capnp(CP)
VM = VehicleModel(CP) VM = VehicleModel(CP)
controller = controller(CP.as_reader(), CI) controller = controller(CP.as_reader(), CI)

@ -3,6 +3,7 @@ import math
import numpy as np import numpy as np
from openpilot.selfdrive.car.card import convert_to_capnp
from openpilot.selfdrive.car.honda.interface import CarInterface from openpilot.selfdrive.car.honda.interface import CarInterface
from openpilot.selfdrive.car.honda.values import CAR from openpilot.selfdrive.car.honda.values import CAR
from openpilot.selfdrive.controls.lib.vehicle_model import VehicleModel, dyn_ss_sol, create_dyn_state_matrices from openpilot.selfdrive.controls.lib.vehicle_model import VehicleModel, dyn_ss_sol, create_dyn_state_matrices
@ -11,7 +12,7 @@ from openpilot.selfdrive.controls.lib.vehicle_model import VehicleModel, dyn_ss_
class TestVehicleModel: class TestVehicleModel:
def setup_method(self): def setup_method(self):
CP = CarInterface.get_non_essential_params(CAR.HONDA_CIVIC) CP = CarInterface.get_non_essential_params(CAR.HONDA_CIVIC)
self.VM = VehicleModel(CP) self.VM = VehicleModel(convert_to_capnp(CP))
def test_round_trip_yaw_rate(self): def test_round_trip_yaw_rate(self):
# TODO: fix VM to work at zero speed # TODO: fix VM to work at zero speed

Loading…
Cancel
Save