Real support for multiple concurrent sensors..

[Linus Torvalds]

On Jul 27, 2017 00:44, "Robert Helling" <helling at atdotde.de> wrote:


As I said, I haven’t looked at the code. But from what I would expect to
see as the user would be that the TankItem for a given moment in time
should report the gas whose pressure has most recently changed


That doesn't work.

Not only is it complicated due to just issues with cooling, sensor
precision, inflation etc, is actually impossible for a very simple reason:
the cylinder you're breathing from may not even have any pressures at all
to begin with.

Yes, we do the nice depth-aware pressure graph, but only if the cylinders
had beginning and ending pressures. Which may not be set at all.

The old code set the sensor number even if there were no pressures just
based on the gas changes.

The new code doesn't have any sensor numbers at all. It has the gas
changes, but the silly TankItem code has thrown all of that away..

      Linus
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20170727/b60b0950/attachment.html>