COVESA / vehicle_signal_specification

Vehicle Signal Specification - standardized way to describe automotive data
Mozilla Public License 2.0
307 stars 157 forks source link

Deprecating OBD-branch #701

Closed erikbosch closed 4 months ago

erikbosch commented 6 months ago

This is a follow up to https://github.com/COVESA/vehicle_signal_specification/issues/635. Intention is to have them deprecated and remove them first in VSS 6.0 Until then anyone can create PRs with replacement signals as needed. This PR contains replacement signals for diagnostics.

As a first point we should we discuss if this is a good approach for removing the OBD branch.

ppb2020 commented 5 months ago

IIRC, our reasoning for deprecating OBD was that the signals were generally duplicates of other signals. I would have expected that some signals are not duplicated and would need to be added elsewhere in the catalog as part of the removal. I don't see other signals added...

I would also have expected that some ODB-specific signals might remain, for example a signal indicating that a device is connected to the OBD port. Or one where some "commands" can be sent in specifically to the OBD using certain pins on the diagnostic connector, which could be represented as signals specific to OBD... I don't know if OBD specifies this, however.

erikbosch commented 5 months ago

IIRC, our reasoning for deprecating OBD was that the signals were generally duplicates of other signals. I would have expected that some signals are not duplicated and would need to be added elsewhere in the catalog as part of the removal. I don't see other signals added...

In this PR I add signals for DTCCount and DTCList as we do not have DTCs elsewhere and several people have said that they could be useful. But you are correct, there are more signals that is not represented in VSS, especially in the area of combustion engine status (fuel trim, timing advance, lambda sensor readings and so on). But the question is if those signals are useful in VSS.

I can see multiple approaches here:

My original idea is to use the last approach, but allow long time for replacement signals to be created, i.e. remove OBD signals first in VSS 6.0. But your points are good, and we should align on the approach.

erikbosch commented 5 months ago

Meeting notes:

erikbosch commented 4 months ago

MoM: OK to merge