<div dir="ltr"><br><br>On Sun, Jul 30, 2017 at 10:16 AM, Linus Torvalds <<a href="mailto:torvalds@linux-foundation.org">torvalds@linux-foundation.org</a>> wrote:<br>><br>> What happens is that we actually do populate the gas changes, but we<br>> do it by populating the end pressure to the time of the last sample.<br>><br>> And with the OSTC3, the last sample is actually way beyond the end of<br>> the dive, because the OSTC3 keeps doing samples for a long time after<br>> you've already surfaced.<br>><br>> So when we try to populate the fend pressure data for your dive, we<br>> don't see any plot entries for that time, and we fail.<br>><br>> Easily fixed, I think. Let me try.<br><br>Yup, that was it. The attached patch fixes it.<br><br>The patch *does* show something a bit odd, though. Your second dive in that XML file you sent actually has gas data for *four* cylinders:<br><br><div>  'Steel 2L' start='190.0 bar' end='110.0 bar'<br>  'Steel 2L' o2='100.0%' start='180.0 bar' end='90.0 bar'<br>  'Steel 6L' o2='32.0%' start='160.0 bar' end='160.0 bar'<br>  'Steel 6L' o2='50.0%' start='180.0 bar' end='180.0 bar' <br><br>but you don't actually have any gas change events for that dive. Oddly, the "equipment" window only shows two of the cylinders, I think there's some broken equipment logic that hides the two other ones. Probably related to the fact that they have no gas use associated with them (beginning and ending pressures are the same).<br><br>But with this patch, we now we show pressure graphs all four of them, and because none of them has any time associated with them, we show them for the whole dive:</div><div><br></div><div><img src="cid:ii_j5r01azj1_15d948a4bb20f840" width="562" height="301"><br><br></div><div>which is probably not what you really expected. </div><div><br></div><div>Anyway, this patch is correct, although I think I'll add the logic to not show the pressure graph for a cylinder that doesn't have a pressure drop (unlike the equipment tab, it actually *does* make sense to hide the cylinders that aren't used on the dive).</div><div><br></div><div>So the patch I'll send to Dirk will look slightly different.</div><div><br></div><div>           Linus<br><br></div></div>