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

No suppport for WWHOBD #245

Closed
Krash-2504 opened this issue Sep 18, 2024 · 6 comments · Fixed by #264
Closed

No suppport for WWHOBD #245

Krash-2504 opened this issue Sep 18, 2024 · 6 comments · Fixed by #264

Comments

@Krash-2504
Copy link

Hi,
There seems to be no client functions for WWHOBD. Is there any plan of adding support for WWHOBD?

@pylessard
Copy link
Owner

pylessard commented Sep 19, 2024

Was not aware that this existed.
I'm looking at ISO-14229 , I see they added reportWWHOBDDTCByMaskRecord, and reportWWHOBDDTCWithPermanentStatus in the 2013 edition. Is that what you are looking for?

I can surely add the support relatively fast.

@Krash-2504
Copy link
Author

Yes, these are the ones.

@pylessard
Copy link
Owner

Support shouldn't be too hard to add. I'm working on something else these days. Will check eventually. I'm also open to a PR, but it would need all the unit tests.

Cheers

@pylessard
Copy link
Owner

Hey, I'm reopening.
These 4 values are not correctly reported.

Image

Will fix and will also add reportWWHOBDDTCWithPermanentStatus before making a release

@pylessard
Copy link
Owner

Updated in #266

@pylessard
Copy link
Owner

Available in v1.25.0

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 a pull request may close this issue.

2 participants