<html>#2627: Tmunu inherits from ADMBase and StaticConformal without using them
<table style='border-spacing: 1ex 0pt; '>
<tr><td style='text-align:right'> Reporter:</td><td>Gabriele Bozzola</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></td></tr>
<tr><td style='text-align:right'> Type:</td><td>bug</td></tr>
<tr><td style='text-align:right'> Priority:</td><td>trivial</td></tr>
<tr><td style='text-align:right'>Component:</td><td></td></tr>
</table>
<p>Comment (by Roland Haas):</p>
<p>Historic leftover and backwards compatible thing I would say.</p>
<p>The old Tmunu mechanism (using the FRIEND mechanism) had a routine <code>TmunuBase_SetTmunu</code> that used <code>ADMCoupling</code>'s <code>STRESSENERGY_guts.h</code> macros, which may have conceivably used <code>ADMBase</code> variables (at least that is my only guess). Or it may have been superfluous when the thorn was first created in 2009. The old Tmunu code was removed in git hash <a data-is-external-link="true" href="https://bitbucket.org/einsteintoolkit/einsteinbase/commits/5a412c04f4799f3add72194332f3b3d8236487bf" rel="nofollow">5a412c0</a> "TmunuBase: remove deprecated support for old CalcTmunu mechanism" of <a data-is-external-link="true" href="https://bitbucket.org/einsteintoolkit/einsteinbase" rel="nofollow">einsteinbase</a>.</p>
<p>Right now removing it may be dicey since inheritance is transitive, that is thorns inheriting from <code>TmunuBase</code> get access to all the grid functions that <code>TmunuBase</code> inherited. There is bound to be a thorn out there that <em>only</em> inherits from <code>TmunuBase</code> but still accesses <code>ADMBase</code> variables (valid code in fact).</p>
<p>This is probably a good candidate for a deprecated features announcement. While I doubt there are Tmunu using thorns that do not also use <code>ADMBase</code> there is a case to be made for no longer requiring <code>StaticConformal</code> in particular since support for conformaly related metrics is essentially non-existent in current codes.</p>
<p>--<br/>
Ticket URL: <a href='https://bitbucket.org/einsteintoolkit/tickets/issues/2627/tmunu-inherits-from-admbase-and'>https://bitbucket.org/einsteintoolkit/tickets/issues/2627/tmunu-inherits-from-admbase-and</a></p>
</html>