Skip to content
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

implement sub-components #352

Merged
merged 2 commits into from
Oct 18, 2024
Merged

Conversation

andlaus
Copy link
Collaborator

@andlaus andlaus commented Oct 17, 2024

Sub-components are a mechanism to group physically or logically related diagnostic variables of an ECU. As far as I can see, sub-components are purely informative and optional. Also, they are (probably) only relevant for the (somewhat uncommon) communication mechanism based on diagnostic variables.

Since I do not have any datasets which use this feature in earnest and I'm too lazy to write a comprehensive unit test, your mileage may vary a bit. (there should be no major screwups, though...)

Andreas Lauser <andreas.lauser@mercedes-benz.com>, on behalf of MBition GmbH.
Provider Information

@andlaus andlaus requested a review from kayoub5 October 17, 2024 13:40
Sub-components are a mechanism to group physically or logically
related diagnostic variables of an ECU. As far as I can see,
sub-components are purely informative and optional. Also, they are
(probably) only relevant for the (somewhat uncommon) communication
mechanism based on diagnostic variables.

Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
@andlaus andlaus force-pushed the implement_subcomponents branch from b70afb9 to fa65cb7 Compare October 17, 2024 14:25
@andlaus andlaus merged commit e2a3010 into mercedes-benz:main Oct 18, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants