[ET Trac] [Einstein Toolkit] #681: LSUThorns/Vectors: Update Kranc-specific code

Einstein Toolkit trac-noreply at einsteintoolkit.org
Tue Jan 17 03:27:16 CST 2012


#681: LSUThorns/Vectors: Update Kranc-specific code
------------------------------------+---------------------------------------
  Reporter:  eschnett               |       Owner:        
      Type:  defect                 |      Status:  review
  Priority:  major                  |   Milestone:        
 Component:  EinsteinToolkit thorn  |     Version:        
Resolution:                         |    Keywords:        
------------------------------------+---------------------------------------

Comment (by hinder):

 OK I see.  Would it make sense to change the conditional to #ifdef KRANC_C
 so that you can be sure you are only affecting Kranc thorns?  It might be
 that other header files also define E for their own purposes.  For Kranc-
 generated thorns, we can be fairly sure that this is not the case.

 Now that I think about it some more, why is this replacement handled here
 by macros?  Wouldn't it make more sense to have E -> ToReal[E] in Kranc
 when vectorisation is enabled (and similarly for Pi)?  Then Vectors would
 not need to know about a Kranc-specific symbol, and Kranc would be
 treating these constants just like it treats parameters (for example).

-- 
Ticket URL: <https://trac.einsteintoolkit.org/ticket/681#comment:4>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit


More information about the Trac mailing list