<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi,<div class=""><br class=""></div><div class="">now, I have done some reading, both in the source and in trac (I should have done this earlier):</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 06 Feb 2015, at 22:54, Dirk Hohndel <<a href="mailto:dirk@hohndel.org" class="">dirk@hohndel.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">But a bug report from an OC diver forced me to dig into this and what I</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">think I'm beginning to understand doesn't make me happy.</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""></div></blockquote></div><br class=""></div><div class="">Double use of the event is not nice, but it does not seem to be the problem here: When the interpretation is „set-point change“, the name of the event (the fourth argument of add_event) is always „SP change“ and whenever we read this event, we not only compare the event type but also the name. So there cannot be any confusion from that point. The double use should be resolved, but this is not the problem in #826.</div><div class=""><br class=""></div><div class="">The problem for the user arises (as you comment in trac correctly) as the predator reports pO2 even in OC mode (probably a value that it computed) and historically (from the time before we had a dive-mode or even dctype) we determined a CCR from the fact that there was pO2 information. That should not be there. I would say, the bug is here.</div><div class=""><br class=""></div><div class="">So, how shall we proceed? In the future, I would say, we should not save those values.</div><div class=""><br class=""></div><div class="">What about old logs? We could delete all those pO2 values if the dive computer is a Predator and the user selects OC as dive mode?</div><div class=""><br class=""></div><div class="">Independent of that, the SP change at t=0 should not be displayed to the user as that is actually the dive mode at the start of the dive which is selected in the combo box. But that is a display problem (that can easily be solved).</div><div class=""><br class=""></div><div class="">Best</div><div class="">Robert</div></body></html>