<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 31 August 2015 at 02:16, 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Could I get a more detailed commit message for this?<br>
<br></blockquote><div><br></div><div>It appeared to be an obvious change once I worked out that there was a hidden bug, but it's quite well hidden, so not actually obvious at all.  Yes, it does deserve a commit message.<br><br>If adjacent samples at times t0 and t1 are greater than time_stepsize apart,<br>this for loop steps through the time between samples to calculate tissue<br>tolerance incrementally at each intermediate time, j, (and interpolated depth).<br><br>If the difference between t1 and t0 is not a multiple of time_stepsize,<br>immediately before the final increment time_stepsize needs to be reduced to<br>t1 - j so that j = t1 in the final increment.<br><br>This is necessary when:<br>    t1 - j < time_stepsize    (i.e. we are about to start the final<br>                increment, and time_stepsize needs to be<br>                reduced), and<br>    j < t1            (i.e. we didn't just do the final increment -<br>                without this condition time_stepsize would be<br>                set to zero, leading to an infinite loop)<br><br>Previously, the check was (j - t0 < time_stepsize), which always returns false.<br><br><br></div><div>Cheers,<br><br></div><div>Rick<br></div><div><br></div></div></div></div>