Fix overlay merge error

COVESA / vss-tools

Software for working with VSS (https://github.com/COVESA/vehicle_signal_specification)

Mozilla Public License 2.0

49 stars 51 forks source link

Fix overlay merge error #346

Closed erikbosch closed 2 months ago

erikbosch commented 3 months ago

This PR solves a problem when a signal is added in an overlay and the signal is "deeper" down in an instantiated tree. A hypothetical example below, adding a temperature signal for front left brake in an overlay. Current result is that Temperature signal is not included in the final result.

Vehicle.Chassis.Axle.Row1.Wheel.Left.Brake.Temperature:
  datatype: float
  type: sensor
  description: Brake Temperature FL
  dbc2vss:
    interval_ms: 1000
    signal: BrakeTempFL3FE
    transform:
      math: "floor(x-40)"

erikbosch commented 3 months ago

MoM:

Please review until Thursday EOB
Merge Friday if no remarks

erikbosch commented 2 months ago

MoM: Merge