<div dir="ltr">On Fri, Jul 24, 2015 at 12:04 PM, 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 17:44, 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">On Fri, Jul 24, 2015 at 11:23 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><div>On 24 Jul 2015, at 17:04, 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 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><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></span><div>Hi Erik,</div><div><br></div><div>Something is wrong with the merge: the merge commit should have two parents, but it only has one.  </div></div><div><br></div><span style="white-space:pre-wrap">        </span><a href="https://bitbucket.org/einsteintoolkit/mclachlan/commits/all" target="_blank">https://bitbucket.org/einsteintoolkit/mclachlan/commits/all</a><div><br></div><div>It looks like the &quot;merge&quot; is just a set of changes on the master branch.</div></div></blockquote><div><br></div><div>I merged the master branch into the rewrite branch, and the result of this then back into the master. The former was non-trivial because of other changes that happened to the master in the mean time, including regenerating code, so I introduced a temporary branch &quot;integrate-rewrite&quot; for this. I didn&#39;t look at the parent structures of the commits.</div><div><br></div><div>I now see that integrate-rewrite doesn&#39;t have rewrite as parent. Very strange. I only used SourceTree, and only used standard mechanisms (merging, branching, committing).</div></div></div></div></blockquote><div><br></div></span><div>(summarising IM conversation with Erik)</div><div><br></div><div>It looks like some functionality from master was lost in the &quot;merge&quot;.  e.g. the ability to run the tests on any number of processes.  This suggests to me that other changes may also be missing.</div></div></div></blockquote><div><br></div><div>Leaving out this ability was on purpose to simplify the merge. In particular, this means that the commit that renamed &quot;phi&quot; to &quot;phiW&quot; could simultaneously rename the respective output files. Completely changing the output format requires re-generating all test results, making it impossible to track whether the code has an error. It is easy to do so after the fact -- I&#39;ll do that right now.</div><div><br></div><div>If you find other functionality that is missing, let me know. I&#39;m not aware of others, except as listed on the wiki.</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>