search

langemand / SoulEVSpy

Android app for Kia and Hyundai electric cars
Apache License 2.0
13 stars 5 forks source link

VIN not decoded for a Kona. #32

Closed EricYoungdale closed 3 years ago

EricYoungdale commented 4 years ago

My VIN starts with:

KM8K53AG5KU

The code in KiaVinParser.java doesn't handle the case of wmi.equals("KM8")

langemand commented 4 years ago

Thanks, I will add that combination. Merry Christmas! --Best regards / venlig hilsenHenrik ScheelMobil: +45 2720 9828-------- Oprindelig meddelelse --------Emne: [langemand/SoulEVSpy] VIN not decoded for a Kona. (#32)Fra: EricYoungdale Til: langemand/SoulEVSpy CC: Subscribed My VIN starts with: KM8K53AG5KU The code in KiaVinParser.java doesn't handle the case of wmi.equals("KM8")

—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.

langemand commented 4 years ago

I am updating the KiaVinParser code.

The 5th character in the VIN identifies the car trim level. Yours is ‘5', which is new to me. What is the name of the trim level for your car (other trim levels are Trend, Exclusive, SE, Base, …)?

EricYoungdale commented 4 years ago

Hello again. I am updating the KiaVinParser code. The 5th character in the VIN identifies the car trim level. Yours is ‘5', which is new to me. What is the name of the trim level for your car (other trim levels are Trend, Exclusive, SE, Base, …)? I wish you a Merry Christmas. Best regards, Henrik -- Venlig hilsen / best regards Henrik R. Scheel Spjeldager Consult ApS Spjeldager 9 DK-2630 Taastrup CVR: 34491399 T: +45 2720 9828 http://www.spjeldager.dk/ http://www.spjeldager.dk/ … Den 24. dec. 2019 kl. 13.57 skrev EricYoungdale @. @.>>: My VIN starts with: KM8K53AG5KU The code in KiaVinParser.java doesn't handle the case of wmi.equals("KM8") — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#32?email_source=notifications&email_token=AAONDNUJKD46RQPBVNWK3XLQ2IBM5A5CNFSM4J66EVNKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ICQDZJA>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAONDNSKPPACNYXMTXJEZ3TQ2IBM5ANCNFSM4J66EVNA.

It is the "Ultimate" trim. That's what they call it in the States, anyways

EricYoungdale commented 4 years ago

One other thing that might be relevant. The Kona in the US does not have a heat pump (PTC heater only). And there is no battery heater either. The Canadian versions of the car have both of these things.

In 2020, they added the battery heater to US models, so it is only the 2019 models that have this quirk. I don't recall whether all trim levels have this.

The BMS page shows funny numbers for the temperature of the non-existent battery heater. I had not yet gotten around to collecting the logs and creating a 2nd issue.

langemand commented 4 years ago

Hi again. I don’t know if you are using the paid version (Soul EV Spy) or the free version (Soul EV Spy Lite); but I have released a fix for the VIN-parsing in the latest BETA-release of the paid version: 1.0.0-340.

Thanks for informing me of the bug!