<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Rick,</div><br class=""><div class=""><blockquote type="cite" class=""><div class="">On 27 Aug 2015, at 00:39, Rick Walsh <<a href="mailto:rickmwalsh@gmail.com" class="">rickmwalsh@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: arial, helvetica, sans-serif;" class="">^^^This last line changes the calculated plan but I don't know why.  I have attached the results of the test run twice: firstly with the current master (80e7213582c203a128270a0e2e8fcc850eede063), the second with the line vpmb_start gradient() removed.  Removing it improves the test results (passes 8/10 compared to 6/10, and one of the two failing is failing by 3 seconds using the within-1%+1min-of-vpmb-comparison-result-is-good-enough logic).  The results are replicated when the plan is calculated by running Subsurface properly.<br class=""><br class=""></span></div><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: arial, helvetica, sans-serif;" class="">My problem is that I can't work out why this line should have any effect at all.  Otherwise I would have sent a patch.  Do you understand what it's changing?<br class=""></span></div></div></blockquote></div><br class=""><div class="">let me add to this confusion: In my branch, I had computed the first stop depth for each round of the CVA, as done in this patch:</div><div class=""><br class=""></div><div class=""></div></body></html>