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

Hyundai VDO TPMS sensors with CRC #1643

Merged
merged 1 commit into from
Apr 14, 2021
Merged

Conversation

teou1
Copy link
Contributor

@teou1 teou1 commented Feb 1, 2021

#1618

Hi again, am kind of still learning Git... now we have 2 topics about that...sorry.
CRC worked like a charm.
I think it is ready to be merged?

@teou1
Copy link
Contributor Author

teou1 commented Feb 1, 2021

tpms-final.txt

@zuckschwerdt
Copy link
Collaborator

The code needs an update to our style. I'll do that and merge in the next few days.

@zuckschwerdt
Copy link
Collaborator

Can you PR sample files to rtl_433_tests? Or upload a zip with test samples here.

@teou1
Copy link
Contributor Author

teou1 commented Feb 25, 2021

Here it is. But the old logs around Christmas are made with older test versions of the code. The last 1-2 are with the current version of the code (with crc)
tpms.zip

@merbanan
Copy link
Owner

merbanan commented Apr 9, 2021

@zuckschwerdt will you update and merge the code ?

@zuckschwerdt zuckschwerdt merged commit 3537257 into merbanan:master Apr 14, 2021
@zuckschwerdt
Copy link
Collaborator

Enabled in f3f04c0.

@teou1 can you upload a zip with 2-3 .cu8 files of this sensor?

@Mindavi
Copy link
Collaborator

Mindavi commented Aug 22, 2021

hyundai-vdo-samples.zip

Got some samples in the wild, which is not ideal. Just posting them here so they don't get lost, feel free to push them to rtl_433_tests if I forget.

I cannot validate if these are actually good samples, but the data seems sane-ish and the CRC checks out... good enough for me :)

{"time" : "@0.186260s", "model" : "Hyundai-VDO", "type" : "TPMS", "id" : "93e32a92", "state" : 220, "flags" : 12, "repeat" : 2, "pressure_kPa" : 144.375, "temperature_C" : 28.000, "maybe_battery" : 25, "mic" : "CRC"}
{"time" : "@0.345164s", "model" : "Hyundai-VDO", "type" : "TPMS", "id" : "84b3a66a", "state" : 32, "flags" : 12, "repeat" : 3, "pressure_kPa" : 262.625, "temperature_C" : 24.000, "maybe_battery" : 41, "mic" : "CRC"}

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.

4 participants