interesting little bug...

[Dirk Hohndel]

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

> On Wed, Mar 27, 2013 at 7:10 PM, Linus Torvalds
> <torvalds at linux-foundation.org> wrote:
>>
>> I say "kind of" because we end up *showing* the result incorrectly. As
>> far as I can tell, we now merge those two dives correctly, including
>> renaming the sensor numbers on the Uemis, and all the sensor data is
>> there when we save it. But it doesn't *show* the two different sensors
>> correctly for the Uemis after the merge.
>
> Indeed. I just re-sent the patch along with a fix for the but that
> caused the gas mixes in the profile to be shown incorrectly.
>
> NOTE NOTE NOTE! With the three patches, we now show the Uemis pressure
> information correctly for the two different sensors, but the Petrel
> profile still looks bogus. The Petrel dive has no cylinder pressure
> data, but there's still some bug in the profile code that gets
> confused about the gas changes when it wants to do the *interpolation*
> of the known starting and ending pressures.

With all four of your patches applied (and pushed) things look pretty
good to me. Thanks for working on this!

/D