Review request: mCCR diving with OSTC3 fixed setpoints

Jef Driesen jef at libdivecomputer.org
Fri Apr 28 12:50:45 PDT 2017


On 28-04-17 18:56, Jan Mulder wrote:
> The short remark of Jef changed basically everything. So I replaced my
> adapted libdivecomputer branch completely (but still under the same
> name, sorry to anybody who already pulled it). The concept of the change
> is now very simple.  Libdivecomputer just puts the initial setpoint in
> OSTC3 CCR mode in the first sample, and now Subsurface is now working
> straight from the box.
>
> So anybody who wants to take a look, the updated code is now at
> https://github.com/janmulder/libdc/tree/ostc3-upd, and there is no
> update in Subsurface required anymore.
>
> As a "bonus", also the battery percentage data that is recently added by
> HW to the logbook of the OSTC3, is now also visible in Subsurface.
>
> And finally a question for Jef: how do I submit commits for libdivecomputer?

You just did :-) I can easily pull the commit from github.

I think there is still something missing. You are now always emitting the 
setpoint sample, even for non-CCR dives. I think that's wrong and there should 
be an extra check.

Jef


More information about the subsurface mailing list