<div dir="ltr">On Fri, Jul 24, 2015 at 11:04 AM, Ian Hinder <span dir="ltr">&lt;<a href="mailto:ian.hinder@aei.mpg.de" target="_blank">ian.hinder@aei.mpg.de</a>&gt;</span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><span class=""><div>On 24 Jul 2015, at 16:59, Ian Hinder &lt;<a href="mailto:ian.hinder@aei.mpg.de" target="_blank">ian.hinder@aei.mpg.de</a>&gt; wrote:</div><br><blockquote type="cite"><div style="word-wrap:break-word"><br><div><div>On 24 Jul 2015, at 14:13, Erik Schnetter &lt;<a href="mailto:schnetter@cct.lsu.edu" target="_blank">schnetter@cct.lsu.edu</a>&gt; wrote:</div><br><blockquote type="cite"><div dir="ltr">I just pushed all the changes related to the McLachlan-rewrite merge. Apart from the McLachlan Kranc scripts and the respective generated code, there are a few thorns that needed to have test results updated.<div><br></div><div>I see zero test failures with these changes on Bethe, Blue Waters, Carver, Datura, Edison, Hopper, Mike, Nvidia (a Perimeter workstation), Philip, Queen Bee, Shelob, and Stampede.</div></div></blockquote><div><br></div><div>Hi Erik,</div><div><br></div><div>It looks like there are some issues with backward compatibility.  I get this error:</div><div><br></div><div>WARNING[L2,P0] (Cactus): ParameterSetKeyword: Unable to set keyword &#39;ML_BSSN::my_rhs_boundary_condition&#39;, new value &#39;static&#39; is not in any active range<br>WARNING[L1,P0] (Cactus): Major error in parameter file &#39;/lustre/datura/ianhin/simulations/einsteintoolkit/rewrite_ML_BSSN_bench_o4/output-0000/ML_BSSN_bench.par&#39; line 30: Range error setting parameter &#39;ML_BSSN::my_rhs_boundary_condition&#39; to &#39;static&#39;<br></div></div></div></blockquote><div><br></div></span><div>Also, any output parameters referring to the old variables are now invalid; e.g. phi has been renamed phiW.  Given this, it looks like old parameter files will very likely all have to be modified for the new version.</div></div></div></blockquote><div><br></div><div>Ian</div><div><br></div><div>Yes, this is correct. You suggested renaming &quot;phiW&quot; to &quot;cf&quot; in the ticket -- if so, now would be the time to do so to avoid two subsequent renamings.</div><div><br></div><div>-erik</div></div><div><br></div>-- <br><div class="gmail_signature">Erik Schnetter &lt;<a href="mailto:schnetter@cct.lsu.edu" target="_blank">schnetter@cct.lsu.edu</a>&gt;<br><a href="http://www.perimeterinstitute.ca/personal/eschnetter/" target="_blank">http://www.perimeterinstitute.ca/personal/eschnetter/</a></div>
</div></div>