<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On 10 Oct 2011, at 18:54, Erik Schnetter wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>On Sun, Oct 9, 2011 at 7:39 PM, Luca Baiotti &lt;<a href="mailto:baiotti@ile.osaka-u.ac.jp">baiotti@ile.osaka-u.ac.jp</a>&gt; wrote:<br><blockquote type="cite">Hello,<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">I was looking into the param.ccl of CarpetIOASCII (hg version) and found<br></blockquote><blockquote type="cite">some deprecated parameters:<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">BOOLEAN out3D_ghosts "Output ghost zones (DEPRECATED)"<br></blockquote><blockquote type="cite">{<br></blockquote><blockquote type="cite">} "yes"<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">BOOLEAN out3D_outer_ghosts "Output outer boundary zones (assuming that<br></blockquote><blockquote type="cite">there are nghostzones boundary points (DEPRECATED)"<br></blockquote><blockquote type="cite">{<br></blockquote><blockquote type="cite">} "yes"<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">In what sense are they deprecated? I don't do 3D ASCII output, but I<br></blockquote><blockquote type="cite">think that setting the above parameters to "no" would be better (see<br></blockquote><blockquote type="cite">also below).<br></blockquote><br>These are deprecated because they do not calculate the number of ghost<br>zones correctly; they make some assumptions that hold in many cases,<br>but may either output ghost zones or not output real grid points in<br>some cases.<br><br><blockquote type="cite">Furthermore,<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">BOOLEAN out1D_d "Do output along the diagonal" STEERABLE = ALWAYS<br></blockquote><blockquote type="cite">{<br></blockquote><blockquote type="cite">} "yes"<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">should also default to no, I suggest.<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Finally, I would also make the default of:<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">BOOLEAN output_ghost_points "Output ghost points"<br></blockquote><blockquote type="cite">{<br></blockquote><blockquote type="cite">} "yes"<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">to "no". This would also make the script<br></blockquote><blockquote type="cite">CarpetIOASCII/src/util/Carpet2ygraph.pl<br></blockquote><blockquote type="cite">work again (see other message about <a href="http://Carpet2ygraph.pl">Carpet2ygraph.pl</a> in this mailing list).<br></blockquote><blockquote type="cite">Except for debugging, is there any advantage in outputting ghost zones?<br></blockquote><br>That is correct, one only wants to look at ghost zones for debugging.<br>Do you want to open a TRAC request, or suggest this again after the<br>release? Carpet is now certainly mature enough so that debugging<br>output does not need to be the default any more.<br></div></blockquote><div><br></div></div><div>The relevant TRAC ticket is&nbsp;<a href="https://trac.einsteintoolkit.org/ticket/677">https://trac.einsteintoolkit.org/ticket/677</a>.</div><div><br></div><div>Here are my 2 pence:</div><div><br></div><div>* Parameters which have been deprecated for a while should be removed now, and not changed, unless they are in fact routinely used, and the deprecation status should be changed;</div><div><br></div><div>* If we are going to change the default behaviour, we should also consider fixing the parameters to do a sensible thing;</div><div><br></div><div>* This output is useful not only for debugging Carpet, but also for debugging user thorns.&nbsp;</div><div><br></div><div>* If possible, I would avoid a change which needed the test suites to be regenerated, unless someone is willing to commit time to doing this carefully. &nbsp;This will likely affect the majority of tests in not only the ET but also in groups' private repositories. &nbsp;I think that changing something as fundamental as this might be unreasonable. &nbsp;The test suites should output these points anyway to catch regressions. &nbsp;The test suite parameter files could be modified to handle the change of defaults, and this is much preferable to regenerating the data.</div><div><br></div><div>
<span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>--&nbsp;</div><div>Ian Hinder</div><div><a href="http://numrel.aei.mpg.de/people/hinder">http://numrel.aei.mpg.de/people/hinder</a></div></div></span></div></span></div></span></span>
</div>
<br></body></html>