<div dir="ltr">Dirk, Jan,<br><div><div class="gmail_extra"><br><div class="gmail_quote">On 5 July 2015 at 23:56, 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"><span class="">On Sat, Jul 04, 2015 at 12:27:23AM +0200, Jan Darowski wrote:<br>
> Hi,<br>
> Here is another pull request. I hope now it's better. Everything was<br>
> reorganized from scratch, the final code is almost the same.<br>
<br>
</span>I like the patches much better.<br>
I agree with Robert that you could have squashed a couple together, but in<br>
the end I felt it was better to just pull what you had.<br>
<br></blockquote><div> </div><div>I've reattached the patch to use the same VPM configuration parameters used in other programs.  This is necessary to match Eric Baker's 'benchmark' dive profile.  The patch is the same as before, but hopefully with a better commit message and line wrapping.<br><br></div><div>I assume this was overlooked rather than deliberately omitted before, but do say if there's a reason why it shouldn't be applied now.<br></div><div><br></div><div>The second patch adds units to the structure comments (but not in the variable names).  I'm confused by the units for surface_tension_gamma and skin_compression_gammaC.  According to <a href="http://www.deepocean.net/deepocean/index.php?science04.php">http://www.deepocean.net/deepocean/index.php?science04.php</a><br>surface_tension_gamma = 17.9 dyn/cm = 0.179 N/m, but 17.9 dyn/cm = 0.0179 N/m<br>surface_tension_gamma = 257 dyn/cm = 2.57 N/m, but 257 dyn/cm = 0.257 N/m<br></div><div><br></div><div>My previous version of the patch gave the unit as mN/cm, but I'm not sure that's right either.<br></div><div><br></div><div>As Jan noted before, it isn't the first error on that page.  However, the calculated profile matches the benchmark, so something must be right (or unit errors are cancelling each other out).<br></div><div><br></div><div>Cheers,<br><br></div><div>Rick<br></div><div><br></div></div></div></div></div>