Farfar / homeassistant-seatconnect

Seat Connect - A home assistant plugin to add integration with your car
Apache License 2.0
61 stars 14 forks source link

Kilometer to Mileage conversion incorrect #24

Closed greghesp closed 2 years ago

greghesp commented 2 years ago

Hey,

It seems a few of the sensor readings are incorrect. Below are just a few examples, but I think this is the KM to MI conversion

Sensor Seat Portal Home Assistant
Range/Combined Range 75miles 12.1miles
Last Trip Length 15miles 2.4miles
Odometer 4247miles 683.5miles

For example, below is the response from the API request when accessing the portal. I believe these are all in KM. For example, 121 KM is 75 MI which the UI shows, however HA reports this as 12.1 miles

Same for Odometer, below its reported at 6835KM which is 4247MI but the conversion is wrong in HA

centralLockStatus: "locked"
daysToOil: 273
daysToService: 498
distanceToOilValue: 14001
distanceToServiceValue: 23175
doorsWindowsOverallStatus: "closed"
eLevel: 0
eRange: 0
fuelLevel: 26
mileage: 6835
primaryEngineRange: 121
showOnlySOC: false
totalEngineRange: 121
windowStatusSupported: true
greghesp commented 2 years ago

Imperial Units not KM to MI needed