-
Notifications
You must be signed in to change notification settings - Fork 168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deprecating OBD-branch #701
Conversation
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. |
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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is good. Putting deprecation, but already migrating DTC (which is more or less the only point in OBD I have seen continuously being used"
I do agree with @ppb2020 that we should double check we are not "loosing" any potentially useful information from the OBD branch, but as deprecation does not imply immediate removall, we might want to do this in a seperate issue/PR?
Meeting notes:
|
This is a follow up to COVESA#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. Signed-off-by: Erik Jaegervall <[email protected]>
MoM: OK to merge |
This is a follow up to #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.