<html>#2491: VolumeIntegrals_vacuum inherits directly from ML_BSSN
<table style='border-spacing: 1ex 0pt; '>
<tr><td style='text-align:right'> Reporter:</td><td>Roland Haas</td></tr>
<tr><td style='text-align:right'>   Status:</td><td>new</td></tr>
<tr><td style='text-align:right'>Milestone:</td><td></td></tr>
<tr><td style='text-align:right'>  Version:</td><td>development version</td></tr>
<tr><td style='text-align:right'>     Type:</td><td>bug</td></tr>
<tr><td style='text-align:right'> Priority:</td><td>major</td></tr>
<tr><td style='text-align:right'>Component:</td><td></td></tr>
</table>

<p>Comment (by Zach Etienne):</p>
<p>Hey <span class="ap-mention" data-atlassian-id="5bae587b96242d2e2b6110a4">@Leonardo Werneck</span> . I chatted with Roland about this. He says</p>
<blockquote>
<p>The code itself is actually nice to read, just the method is not going to be quite what we want (and I *like* the fact that he reports with VError that he received unexpected input!).</p>
</blockquote>
<p>The above code has a couple problems. First, if the gridfunction name changes in any of the above thorns, then we’ll end up getting errors. Second, the gridfunction names themselves must be “fully qualified”, meaning “ham” → “LeanBSSNMoL::ham”, etc. </p>
<p>Roland suggests that we instead require the users to pass the names of gridfunctions through the parameter interface; e.g.,</p>
<div class="codehilite"><pre><span></span>H_gf   = (CCTK_REAL*)(CCTK_VarDataPtr(cctkGH,timelevel, HamiltonianVarString)); // HamiltonianVarString is defined in param.ccl
if(!H_gf) {
  CCTK_VError (__LINE__, __FILE__, CCTK_THORNSTRING, &quot;Couldn&#39;t get data pointer of input array variable &#39;%s&#39; &quot;, HamiltonianVarString);
}
</pre></div>


<p>… and similarly for momentum constraints. I’d put in ML_BSSN variable strings as default in the <code>param.ccl</code>; e.g., HamiltonianVarString = “ML_BSSN::H” as default</p>
<p>--<br/>
Ticket URL: <a href='https://bitbucket.org/einsteintoolkit/tickets/issues/2491/volumeintegrals_vacuum-inherits-directly'>https://bitbucket.org/einsteintoolkit/tickets/issues/2491/volumeintegrals_vacuum-inherits-directly</a></p>
</html>