<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 17 September 2015 at 04:21, Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.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="">On Wed, Sep 16, 2015 at 07:57:24PM +1000, Rick Walsh wrote:<br>
> Other VPM-B implementations (V-Planner, MultiDeco, Fortran code) take the<br>
> default gradient of onset of impermeability as 8.2 atm.  We use bar as the<br>
> pressure unit, and 8.2 atm = 8.30865 bar.<br>
><br>
> This has negligible effect on all of the calculated plans I've tested, resulting<br>
> in a maximum decrease of 1 minute in total dive time for ~150 m deep dives, but<br>
> we should get unit conversions correct anyway.<br>
<br>
</span>I'm not arguing against that. But with this patch TestPlan fails.<br>
So either submit a patch that also adapts the runtime we expect in the<br>
test (and argue why that is the right thing to do), or...<br>
<br>
NAK as it is<br></blockquote><div><br></div><div>Oops - I should have checked that - it's a bit embarrassing.<br><br></div><div>While sitting in the naughty corner, I realized we were calculating the CVA next gradient based on the tissue loading at the end of the previous iteration, rather than the end of the user-entered portion of the dive.  The attached patch fixes that, and it also makes all the tests pass when applied on top of the previous patch.<br><br></div><div>Rick<br></div><div><br></div></div></div></div>