<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">One possible convention for this is to:<div class=""><br class=""></div><div class=""><ol class="MailOutline"><li class="">Identify the car version early on (either automatically, or by configuration) and store it in a local member variable. Perhaps as an enumerated type.<br class=""><br class=""></li><li class="">For new experimental features, key them on either (a) specific car version known to work, or (b) a configuration option being enabled for that feature.</li></ol><div class=""><br class=""></div><div class="">The above would allow new features to be released for vehicle versions they have been tested on, maintain stability for other vehicle versions, but also allowing other types to set a config variable to be able to test the feature (and later enable it by default for that version should it be found to work).</div><div class=""><br class=""></div><div class="">Regards, Mark.</div><div><br class=""><blockquote type="cite" class=""><div class="">On 14 Dec 2020, at 1:22 PM, 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="">I envisage following a robust testing process outside of the main repo being a bit more difficult with the Leaf because of the quite substantial changes in the can bus framework/messages over the years. A change that works for more recent years but not for prior years is not always easy to spot, also improvements/additions of metrics may only work on a particular year unless another dev with a prior year implements and tests them. <div dir="auto" class=""><br class=""></div><div dir="auto" class="">Without a group of Leaf devs or testers across the various years that can be called upon to test prior to putting through a PR to main I expect future changes will stay in Leaf devs repos untested for other MYs, as there is no particular reason for others to jump in and test the code to confirm compatibility. </div><div dir="auto" class=""><br class=""></div><div dir="auto" class="">PR487 will be a good test case to see if the approach proposed can work in practice. <br class=""><div dir="auto" class=""><br class=""></div><div dir="auto" class=""><br class=""></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>