[PATCH] Ignore events for gaz change if there is no change.

Patrick Valsecchi patrick at thus.ch
Fri Nov 22 00:14:14 UTC 2013


Hi,

The trick of ignoring gaz change if time<30 is not working for
merged dives. We should instead remember the previous gaz (initialized
to air) and ignore the event if it is the same.

While on the subject, why do we get the next plot_data event and not the 
one that have the same time? I saw that the cylinder change appears 
actually in the next plot_data, but that should not be. Isn't that 
slightly messing with the deco computation? IMHO, we should change the 
code that sets the cylinder in plot_data to do it one item before (on 
the one that have the same time as the event) or fix the event time, I 
don't know which one is correct. But the current way of doing things 
gives us the wrong y coordinate and is a bit weird.
What do you think?

Thanks.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Ignore-events-for-gaz-change-if-there-is-no-change.patch
Type: text/x-patch
Size: 4289 bytes
Desc: not available
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20131122/eb40273e/attachment.bin>


More information about the subsurface mailing list