It's with my "old" KS16B - it worked before the last update (imho before updating to 3.0). Here i just correct it by multiplying with 100.
With the new KS16C is now a sign introduced for current. Here i use "if (current>3,2768) then current=6,5536-current)100 else current=current100"
You have now in KingSongAdapter.swift:
var current = Double(Int(buffer[11]) * 256 + Int(buffer[10])) / 100.0
; here you divide by 100 the first time -> and the comparison with 32768.0 would is by this wrong
; in the next line. You would have to compare against 327.68 and subtract 655.36

            if current >= 32768.0 {
                current = current - 65536.0
            }
            
            current = fabs(current / 100.0) // I have problems with sign in some wheels

; here you divide the second time by 100.0
; leaving the sign would be nice, too ;) - if it is possible...

ps.: hope you are fine and have still fun driving! Did not see you around the unicycle forum for quite some time.
pps.: The power values are wrong as in every wheel app (and by this the energy values, too). The problem is, that the wheel send battery voltage and motor current. But since the motor voltage (normally) does not equal the battery voltage one cannot just multiply battery voltage with motor current...;( ( http://forum.electricunicycle.org/topic/7549-current-demand-versus-battery-voltage/?do=findComment&comment=100472 )

Current divided once too much by 100 for KS in V3.0 #10

Description

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions