<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">I’ve tagged, built, and released to EAP, 3.2.015 - based on head of edge.<div class=""><br class=""></div><div class="">Not sure if this is generally releasable at this point (perhaps too rushed), but that should make it easier to access.</div><div class=""><br class=""></div><div class="">Regards, Mark.<br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On 2 Sep 2020, at 2:38 AM, Derek Caudwell <<a href="mailto:d.caudwell@gmail.com" class="">d.caudwell@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="auto" class=""><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr"><span style="font-family:arial,sans-serif" class="">Hi,</span></div><div dir="ltr" class=""><div class=""><font face="arial, sans-serif" class=""><br class=""></font></div><div class=""><font face="arial, sans-serif" class="">Looking at the differences between 3.2.0.14 and earlier it looks like the use of EV Can rather than CAR Can to enable battery monitoring on the Leaf was the main trigger for frequent event crashes (fixed 31/8 in commit </font></div><h4 style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;color:rgb(36,41,46)" class=""><font face="arial, sans-serif" class=""><code style="box-sizing:border-box;padding:0.2em 0.4em;font-weight:400;background-color:rgb(246,248,250);border:1px solid rgb(234,236,239);border-radius:0.2em" class="">07f25e3).</code><span style="font-weight:normal" class="">The polling being turned on (when vehicle ON was set) being more noticeable than at other times as both buses have a lot of traffic - but still not guaranteed to cause a crash as reported by GlynHudson in issue #409. Based on my own experience this issue could still occur randomly but less frequently when the vehicle was off/not polling. </span></font></h4><div class=""><font face="arial, sans-serif" class=""><br class=""></font></div><div class=""><font face="arial, sans-serif" class="">The issue would still occur even if enable canwrites were set to off as reported by GlynHudson in #409, as canwrite logic was not implemented in the Leaf code until 19/8 in commit </font><span style="background-color:rgb(246,248,250);font-family:SFMono-Regular,Consolas,"Liberation Mono",Menlo,monospace;font-size:14.4px;color:rgb(36,41,46)" class="">41ac2f1.</span></div><div class=""><span style="background-color:rgb(246,248,250);font-family:SFMono-Regular,Consolas,"Liberation Mono",Menlo,monospace;font-size:14.4px;color:rgb(36,41,46)" class=""><br class=""></span></div><div class="">A temporary workaround was also implemented in the Leaf code to switch buses prior to polling on 16/8 in commit </div><h4 style="box-sizing:border-box;margin-top:0px;margin-bottom:0px;font-size:16px;color:rgb(36,41,46);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI",Helvetica,Arial,sans-serif,"Apple Color Emoji","Segoe UI Emoji"" class=""><code style="box-sizing:border-box;font-family:SFMono-Regular,Consolas,"Liberation Mono",Menlo,monospace;font-size:14.4px;padding:0.2em 0.4em;font-weight:400;background-color:rgb(246,248,250);border:1px solid rgb(234,236,239);border-radius:0.2em" class="">32db785.</code></h4><div class=""><br class=""></div><div class="">The subsequent poller rework has enabled the Leaf to direct polls to the correct bus, allowing the PID polls originally defined (but not working depending on the car state) to function when the vehicle is on or off and charging. Also the IncomingPollReply code was tidied up and aligned with that used for the Twizy. The vehicle poller extra frame request delay was also originally fixed at 25ms uneccessarily slowing down large replies such as the battery cell voltage responses, now set to 0. These changes mean the polling for the Leaf is now working as it should.<br class=""></div><div class=""><br class=""></div><div class="">The current edge version appears to be stable and allows PID polling reliably to both buses without causing crashes. I have been running it for 24h with 6 start/stop events without issue. </div><div class=""><br class=""></div><div class="">If others agree could 3.2.0.15 be staged for EAP release to move Leaf users off 3.2.0.14? A note could be to 3.2.0.14 stating there is a known stability issue on the Leaf and to use 3.2.0.15 or similar.</div><div class=""><br class=""></div><div class="">GlynHudson and others on other Leaf models can test to ensure we are back to a stable release with the required polling functionality working. </div><div class=""><br class=""></div><div class="">Kind regards</div><div class="">Derek</div></div>
</div></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>