<div dir="auto"><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr"><span style="font-family:arial,sans-serif">Hi,</span></div><div dir="ltr"><div><font face="arial, sans-serif"><br></font></div><div><font face="arial, sans-serif">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)"><font face="arial, sans-serif"><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">07f25e3).</code><span style="font-weight:normal">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><font face="arial, sans-serif"><br></font></div><div><font face="arial, sans-serif">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)">41ac2f1.</span></div><div><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)"><br></span></div><div>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""><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">32db785.</code></h4><div><br></div><div>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></div><div><br></div><div>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><br></div><div>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><br></div><div>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><br></div><div>Kind regards</div><div>Derek</div></div>
</div></div>