<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 <<a href="mailto:baiotti@ile.osaka-u.ac.jp">baiotti@ile.osaka-u.ac.jp</a>> 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 <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. </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. This will likely affect the majority of tests in not only the ET but also in groups' private repositories. I think that changing something as fundamental as this might be unreasonable. The test suites should output these points anyway to catch regressions. 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>-- </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>