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.

Sign up for GitHub

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

Jump to bottom

Battery kilometers not correct #337

Closed
HarmOtten opened this issue Apr 16, 2016 · 4 comments
Closed

Battery kilometers not correct #337

HarmOtten opened this issue Apr 16, 2016 · 4 comments

Comments

@HarmOtten
Copy link
Contributor

CanZE v1.12
I checked the new value [Battery kilometers] on the Detailed Charging screen, and it showed 8565,0.
But my battery has never been replaced, so it should be 74101km. (I double checked with i907)
@fesch
Copy link
Owner

fesch commented Apr 18, 2016

On my Zoé the value is the same than the one displayed on the car ....

@yoh-there
Copy link
Collaborator

yoh-there commented Apr 18, 2016 •
edited
Loading

Rollover on 16 bit number. Nothing we can do about that I am afraid. Maybe add in description

@HarmOtten
Copy link
Contributor Author

We will find the most significant byte/word of this value somehow, because I know it is there.

@yoh-there yoh-there mentioned this issue May 14, 2016
Merged
yoh-there added a commit that referenced this issue May 14, 2016
@yoh-there
Merge pull request #339 from yoh-there/master
cca2263 
Added 8 bits to HV kilometers (bringing it in line with car kms), attempted fix issue #337
@yoh-there
Copy link
Collaborator

Fixed in #339 I cannot test it as I haven't passed 65K kms yet. I will close this issue. Please test your personal pull/clone, or after the next release. Thanks!

@yoh-there yoh-there mentioned this issue May 31, 2016
Merged
yoh-there added a commit that referenced this issue May 31, 2016
@yoh-there
Merge pull request #344 from yoh-there/master
9df1cc9 
Added 8 bits to HV kilometers (bringing it in line with car kms), attempted fix issue #337, fixed #342
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@fesch @yoh-there @HarmOtten