<div dir="ltr">Comments:<div><br></div><div>1. I would change CMB-applications/Baseline and SMTK/baselines to</div><div>  baselines/cmbapplications</div><div>  baselines/smtk</div><div><br></div><div>2. Agreeing with Rob, it will be good to also create directories for files (ADHShallowWater.sbt, simple.stl, etc) under root level</div><div><br></div><div>3. SimBuilderGUI folder has FunctionExpressions.py and FunctionExpressions.sbt file, which is used by a test that's added in V3 with this commit</div><div>a726d2c31d679bb2da62979e2e1dd52be66521df</div><div><br></div><div>and disabled in cmb V4 with commit  due to missing test data directory</div><div>959cf16ee5927f8ece7b9563e8476c0ab95d7f77</div><div><br></div><div>I will take a look at it to see if this test is still relevant.</div><div><br></div><div>Yumin</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 29, 2016 at 9:18 AM, Ben Boeckel <span dir="ltr"><<a href="mailto:ben.boeckel@kitware.com" target="_blank">ben.boeckel@kitware.com</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, Jun 29, 2016 at 09:08:39 -0400, Robert Maynard wrote:<br>
> - Can we have consistent capitalization and plurality between the<br>
> baseline folders ( CMB-Applications/Baseline && SMTK/baselines)<br>
<br>
</span>FWIW, I'd prefer all-lowercase.<br>
<span class="HOEnZb"><font color="#888888"><br>
--Ben<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Smtk-developers mailing list<br>
<a href="mailto:Smtk-developers@smtk.org">Smtk-developers@smtk.org</a><br>
<a href="http://public.kitware.com/mailman/listinfo/smtk-developers" rel="noreferrer" target="_blank">http://public.kitware.com/mailman/listinfo/smtk-developers</a><br>
</div></div></blockquote></div><br></div>