<html>
<head>
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>On 21/04/18 04:00, Robin O'Leary wrote:<br>
</p>
<blockquote type="cite" cite="mid:20180420160059.GA20288@mail.ro.nu">
<blockquote type="cite">
<pre wrap="">I'm already polling the SOH from the BMS with more precision than is
available on 0x5b3, but that calculation needs to know the "new car" Ah
battery size to convert from the current Ah to the SOH. Can you store the
0x5b3 SOH in a different metric (or a class variable?) to avoid clobbering
the existing SOH value.
</pre>
</blockquote>
<pre wrap="">
Ah, is that in a separate branch? I didn't see anything in the master
repository that set ms_v_bat_soh.</pre>
</blockquote>
<br>
It's in master on line 250:
<a class="moz-txt-link-freetext" href="https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/blob/master/vehicle/OVMS.V3/components/vehicle_nissanleaf/src/vehicle_nissanleaf.cpp#L250">https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/blob/master/vehicle/OVMS.V3/components/vehicle_nissanleaf/src/vehicle_nissanleaf.cpp#L250</a><br>
<br>
<blockquote type="cite" cite="mid:20180420160059.GA20288@mail.ro.nu">
<blockquote type="cite">
<pre wrap="">Did you mean to remove the odometer code? Can you put it back and see if it
works with an odometer in miles, I've only been able to test with my km
odometers.
</pre>
</blockquote>
<pre wrap="">
I can confirm that the value on the bus is always km regardless of the
displayed units preference, which I removed from the ms_v_pos_odometer
setting (the git diff got a bit confused, and shows the change as an
addition and the deletion separated by a big chunk of new code to do
with doors and drive state).
</pre>
</blockquote>
<br>
Ah, I see now. I'm glad to hear the units are always in km, I copied
the approach I took from somewhere, happy leaf I think. If it's not
necessary, rip it out. If we discover a car that isn't like this we
can deal with it then. Can you remove the <span class="pl-c1">0x355
based unit check and the </span><span class="pl-c1"><span
class="blob-code-inner">m_odometer_units variable in the class
definition.<br>
</span></span>
</body>
</html>