<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Nov 15, 2014 at 7:19 PM, Linus Torvalds <span dir="ltr"><<a href="mailto:torvalds@linux-foundation.org" target="_blank">torvalds@linux-foundation.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><p dir="ltr"><br>
On Nov 15, 2014 2:20 AM, "Miika Turkia" <<a href="mailto:miika.turkia@gmail.com" target="_blank">miika.turkia@gmail.com</a>> wrote:<br>
><br>
> If start and end cylinder pressure is not explicitly set for secondary<br>
> cylinders, we must store the start and end pressures from the<br>
> sample_{start,end} attribute. Faulty behavior is obvious at least when<br>
> importing Poseidon logs.</p>
</span><p dir="ltr">This is not how is supposed to work.</p>
<p dir="ltr">If we have cylinder sample data, we don't save start/end pressures per cylinder, we get them from the samples.</p>
<p dir="ltr">The saved cylinder pressures are supposed to happen only if the user did some manual overriding (maybe there is sample data, but it was incomplete and the transmitter flaked out in the middle of the dive), or if there is no sample data and the user entered it manually.</p>
<p dir="ltr">Generally the save format tries to avoid saving redundant data very much on purpose.</p><span class="HOEnZb"><font color="#888888">

</font></span></blockquote></div>You are correct. Adding the diluent pressures (that were missing) to be saved to the XML seems to have fixed the start and end pressures without this hack.<br><br>miika<br></div></div>