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

Fix Steelmate pressure formula #3209

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ProfBoc75
Copy link
Collaborator

As per #3200 a new formula for the tire pressure is guessed and applied in this PR.
The previous formula got diverged values with TPMS display.

@zuckschwerdt
Copy link
Collaborator

Very nice! I think pressure_psi should be removed and replaced by pressure_kpa in the output.

@ProfBoc75
Copy link
Collaborator Author

Very nice! I think pressure_psi should be removed and replaced by pressure_kpa in the output.

In that case, it's a breaking change, right ?
Or to provide both , but, I'm not sure that is properly managed by smart home automation clients, like HA ?

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