Odd calculated deco "ripples" (was Re: RFC: color change for calculated deco)

[Dirk Hohndel]

Linus Torvalds <torvalds at linux-foundation.org> writes:

> On Mon, Jan 14, 2013 at 8:13 AM, Dirk Hohndel <dirk at hohndel.org> wrote:
>> I haven't looked at the patch (on my phone) but that seems like the wrong
>> fix. Add-segment assumes constant depth. So now you do an unnecessary
>> staircase approximation, right?
>
> Yes. However, for the *profile* that's fine, since we know the profile
> samples are at most ten seconds apart.

That still changes the calculation by quite a bit if you are ascending
at typical 'travel speed' between deco stops - you'll move by 5 feet in
those ten seconds, right?

> For the planning, probably not so much. Which is why I put quotes
> around "fixes".
>
> The other alternative is to keep the depth interpolation, but get rid
> of the min_pressure thing, and just use the tissue pressure for the
> sample.
>
> Like this patch.

So this uses the LAST instead of the lowest. I'm a bit unclear how that
fixes the ripples - have you tried and verified that this is the case?

/D