messing with the gas / tank handling

Anton Lundin glance at acc.umu.se
Tue Oct 28 15:38:16 PDT 2014


On 28 October, 2014 - Linus Torvalds wrote:

> On Tue, Oct 28, 2014 at 2:13 PM, Dirk Hohndel <dirk at hohndel.org> wrote:
> >
> > Bug #742 shows that we got things pretty badly wrong for the Cobalt if the
> > primary gas used on a dive (or in the case of his sample data, the only
> > gas used on a dive) was not tank 0. We showed that ugly gas change, we
> > showed that two gases were used during the dive, we associated the
> > pressure from the samples with the wrong tank, etc.
> >
> > I just pushed out a commit that pretends to fix this. And it makes sense
> > to me. But I'd really appreciate if you (or anyone else for that matter)
> > would take a look and double check that this is ok.
> 
> It looks fine, but the whole "<= 30s" and "first sample" thing looks a
> bit confused. Do you really need *both*?
> 
> Also, that whole
> 
>     /* if we have an explicit first cylinder */
>     if (sample->sensor == 0 && first_cylinder != 0)
>         sample->sensor = first_cylinder;
> 
> looks a bit iffy. I guess we're stuck with it, but my gut feel is that
> it's actually an Atomic Cobalt back-end bug, and that the Cobalt
> should just have reported the right sensor, rather than reporting
> sensor 0.
> 
> The reason it's iffy is that another dive computer that does this
> *right* might be impacted. But right now, I guess only the Cobalt has
> this whole "first_cylinder" issue, so maybe worry about that
> possibility later.. And quite frankly, we'll need to eventually fix
> the fact that we can only have one cylinder pressure per sample, so
> this code will end up needing changes in more fundamental ways
> eventually. So the whole "this might break in the future" is not a
> very strong objection. The code is pretty much *guaranteed* to break
> in the future for other reasons anyway.
> 

Multiple sensor support feels way better than the current separate
"diluent-sensor" code we currently carry for the Mk6.


//Anton


-- 
Anton Lundin	+46702-161604


More information about the subsurface mailing list