[PATCH] Profile support for multiple concurrent pressure sensors

Linus Torvalds torvalds at linux-foundation.org
Sun Jul 30 13:16:07 PDT 2017


On Sun, Jul 30, 2017 at 12:54 PM, Gaetan Bisson <bisson at archlinux.org> wrote:
>
> The first two are the small O2 and air cylinders in my rebreather, the
> other two are EAN32 and EAN50 bailouts which I took with me on the dive
> but did not end up using. So it looks good to me, but is there some way
> I should be explicitly tagging my CCR cylinders and/or my bailout
> cylinders as such to clear up the confusion?

Subsurface doesn't really care.

And the only reason I was confused at all was that I didn't look
closely enough on the dive, and I expected it to be a regular OC dive
just because the other dives you had in that xml file were.

Yes, you *could* mark them explicitly as O2 and diluent cylinders for
subsurface, but now that we don't really consider the O2 pressures
special, it really doesn't even matter.

And in fact, when I tried, I noticed that we have some bug in the
"Use" editing, and we can only switch between "OC gas" and "unused",
and there's an error on stdout that says

    edit: editing failed

but since nobody uses that feature anyway (and the cylinder equipment
tab doesn't even *show* the use by default - you have to enable it by
right-clicking on the header), I am not going to bother even looking
into it.

                 Linus


More information about the subsurface mailing list