[PATCH] Correct bug: setpoint handling of Poseidon CCR dive logs

Dirk Hohndel dirk at hohndel.org
Wed Oct 29 12:22:09 PDT 2014


On Wed, Oct 29, 2014 at 09:05:10PM +0200, Willem Ferguson wrote:
> On 29/10/2014 20:36, Dirk Hohndel wrote:
> >On Wed, Oct 29, 2014 at 07:57:38PM +0200, Willem Ferguson wrote:
> >>@@ -1178,6 +1178,10 @@ static void fixup_dive_dc(struct dive *dive, struct divecomputer *dc)
> >>  				mintemp = temp;
> >>  		}
> >>+		// If there are consecutive identical O2 setpoint readings, throw away the redundant ones.
> >>+		if (sample->setpoint.mbar == lastsetpoint)
> >>+				sample->setpoint.mbar = 0;
> >I believe we came to the conclusion that this was the wrong thing to do
> >A setpoint of 0 indicates an OC dive...
> >
> >Can you explain why you need this?
> 
> An excellent question. In dive.c the criterion was whether a po2 partial
> pressure is specified or not. If not, an OC dive is assumed. For instance
> see function fill_pressures() in dive.c. Clearly there is some care required
> here when zeroing out all the duplicate values. The setpoint values are
> highly repetitive and, if not zeroed out, will really create a lot of
> additional space requirement.

Nonono. Of course they get zeroed out when WRITING them to XML. But they
should NOT need to be zeroed out in memory. I will spare you the obvious
jokes here...

> On the other hand the pressures.o2 values are
> not zeroed out since it is calculated much later in the process, after the
> plotdata structures have been initialised. On the other hand, the raw sensor
> values (from which pressures.o2 is calculated) are zeroed out. Hope this
> argument sounds half-logical??

No, it completely loses me, actually.

I think in memory we should NOT set anything to 0. When the setpoint
switches to 0 that means we are bailing out to OC...

/D


More information about the subsurface mailing list