<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Are we sure we have the right cables? Remember that on the Nissan Leaf specific cable, the two buses are switched. On the new OBDII generic cables, we have both buses available.<div class=""><br class=""></div><div class="">Regards, Mark</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 2 May 2018, at 7:55 PM, Tom Parker <<a href="mailto:tom@carrott.org" class="">tom@carrott.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
  
    <meta http-equiv="Content-Type" content="text/html;
      charset=windows-1252" class="">
  
  <div text="#000000" bgcolor="#FFFFFF" class="">
    On 02/05/18 01:08, ovms wrote: <br class="">
    <blockquote type="cite" cite="mid:c0421900-ad2c-43e8-bc03-74595acea9aa@DISKSTATION2" class="">
      <div class=""><div style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; color: rgb(39, 42, 216); background-color: rgb(255, 255, 255);" class=""><strong class=""><span style="color: #ba2da2;" class="">case</span> 0x50d</strong><span style="" class=""><strong class="">:</strong>   //Changed this
            from 5bc. 50d is the instrument SOC</span></div><div style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span class="Apple-converted-space">   </span> {</div><div style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span class="Apple-converted-space">     </span> uint16_t nl_gids
          = ((uint16_t) d[<span style="color: #272ad8;" class="">0</span>]
          << <span style="color: #272ad8;" class="">2</span>) | ((d[<span style="color: #272ad8;" class="">1</span>] & <span style="color:
            #272ad8;" class="">0xc0</span>) >> <span style="color:
            #272ad8;" class="">6</span>);</div><div style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span class="Apple-converted-space">     </span> <span style="color: #ba2da2;" class="">if</span> (nl_gids == <span style="color: #272ad8;" class="">1023</span>)</div><div style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span class="Apple-converted-space">       </span> {</div><p style="margin: 0.0px 0.0px 0.0px 0.0px; font: 11.0px Menlo;
          color: #008400; background-color: #ffffff;" class=""><span style="" class=""><span class="Apple-converted-space"> 
                    </span></span></p><div style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span class="Apple-converted-space">       </span> <span style="color: #ba2da2;" class="">break</span>;</div><div style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span class="Apple-converted-space">       </span> }</div><div style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span class="Apple-converted-space">     </span> uint16_t max_gids
          = MyConfig.GetParamValueInt(<span style="color: #d12f1b;" class="">"xnl"</span>,
          <span style="color: #d12f1b;" class="">"maxGids"</span>,
          GEN_1_NEW_CAR_GIDS);</div><div style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span class="Apple-converted-space">     </span> <span style="color: #ba2da2;" class="">float</span> km_per_kwh =
          MyConfig.GetParamValueFloat(<span style="color: #d12f1b;" class="">"xnl"</span>,
          <span style="color: #d12f1b;" class="">"kmPerKWh"</span>,
          GEN_1_KM_PER_KWH);</div><div style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span class="Apple-converted-space">     </span> <span style="color: #ba2da2;" class="">float</span> wh_per_gid =
          MyConfig.GetParamValueFloat(<span style="color: #d12f1b;" class="">"xnl"</span>,
          <span style="color: #d12f1b;" class="">"whPerGid"</span>,
          GEN_1_WH_PER_GID);</div><p style="margin: 0.0px 0.0px 0.0px 0.0px; font: 12.0px
          Helvetica; background-color: #ffffff; min-height: 14.0px;" class=""> <br class="webkit-block-placeholder"></p><div style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span class="Apple-converted-space">     </span>
          m_gids->SetValue(nl_gids);</div><div style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span class="Apple-converted-space">     </span>
          StandardMetrics.ms_v_bat_soc->SetValue(<strong class="">nl_gids / <span style="color: #272ad8;" class="">4</span></strong>);  //Changed
          this so the value is divided by 4.</div><p style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""> </p><p style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""> </p><div style="margin: 0px; font-style: normal; font-variant-caps: normal; font-weight: normal; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class="">SOC now shows
          exactly the same in the app as the instrument-panel does, and
          is not flapping. (I know the instrument-panel is different
          from the true SOC)</div>
      </div>
    </blockquote>
    <br class="">
    I wasn't able to reproduce this. On my car and in the CAN bus
    captures I've got, there is no 0x50d frame on the EV bus. Now that
    I'm back in the house, I see that 0x50d is present on the Car CAN
    bus, but I don't have any data on what's there.<br class="">
    <br class="">
    My notes indicate that 0x55b on the EV bus contains the state of
    charge, and I've implemented this at
<a class="moz-txt-link-freetext" href="https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/compare/master...carrott:nissan-leaf-soc-instrument?expand=1">https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/compare/master...carrott:nissan-leaf-soc-instrument?expand=1</a>
    along with a configuration option to choose how to show the SOC.<br class="">
    <br class="">
    However 0x55b doesn't work how I expected on my 2016 24kWh car, I
    see <br class="">
    <br class="">
    xnl.v.bat.gids                           140<br class="">
    xnl.v.bat.soc.instrument                 56.3%<br class="">
    xnl.v.bat.soc.newcar                     49.8221%<br class="">
    <br class="">
    but the instrument cluster displays 52%. Could it be displaying 56.3
    - 5 = 51.3 rounded up to 52?<br class="">
    <br class="">
    Should I be looking at 0x50d on the car bus to faithfully display
    what the instrument cluster displays? I'll check tomorrow.<br class="">
    <br class="">
    Assuming we get this working, should we default to using the
    instrument cluster state of charge, or the "new car" state of
    charge? I'm guessing we should go with the instrument cluster value,
    which is a change from previous behavior on the leaf but is in
    keeping with Mark's "re-implement the broken calculations by the
    car".<br class="">
  </div>

_______________________________________________<br class="">OvmsDev mailing list<br class=""><a href="mailto:OvmsDev@lists.openvehicles.com" class="">OvmsDev@lists.openvehicles.com</a><br class="">http://lists.openvehicles.com/mailman/listinfo/ovmsdev<br class=""></div></blockquote></div><br class=""></div></body></html>