[ET Trac] [Einstein Toolkit] #683: LSUThorns/Vectors: Remove pos, add sin/cos/tan functions

Einstein Toolkit trac-noreply at einsteintoolkit.org
Mon Nov 28 09:56:12 CST 2011


#683: LSUThorns/Vectors: Remove pos, add sin/cos/tan functions
------------------------------------+---------------------------------------
  Reporter:  eschnett               |       Owner:        
      Type:  enhancement            |      Status:  review
  Priority:  major                  |   Milestone:        
 Component:  EinsteinToolkit thorn  |     Version:        
Resolution:                         |    Keywords:        
------------------------------------+---------------------------------------

Comment (by knarf):

 I agree with both of you. Yes, changes which are not ready should not be
 committed. In most cases anyway. In this case you made a good point of why
 they should anyway Erik. What do you think about protecting the integer
 code with a single #define, which by default disables the code, and which
 has a comment attached explaining that this is not yet fully implemented?
 This way you would not have uncommitted code, and users would know why
 this code isn't used yet.

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


More information about the Trac mailing list