[ET Trac] [Einstein Toolkit] #447: simfactory 1.0 always uses -L 3

[Einstein Toolkit]

#447: simfactory 1.0 always uses -L 3
-----------------------------------------------+----------------------------
  Reporter:  alexander.beck-ratzka@…           |       Owner:  eschnett
      Type:  defect                            |      Status:  new     
  Priority:  major                             |   Milestone:          
 Component:  SimFactory                        |     Version:          
Resolution:                                    |    Keywords:          
-----------------------------------------------+----------------------------

Comment (by hinder):

 According to the documentation
 (http://cactuscode.org/documentation/referencemanual/ReferenceManualch2.html#x4-203000A2),
 the warning levels are:

 #define CCTK_WARN_ABORT    0    /* abort the Cactus run */
 #define CCTK_WARN_ALERT    1    /* the results of this run will probably
 */
                                 /* be wrong, but this isn’t quite certain,
 */
                                 /* so we’re not going to abort the run */
 #define CCTK_WARN_COMPLAIN 2    /* the user should know about this, but */
                                 /* the results of this run are probably ok
 */
 #define CCTK_WARN_PICKY    3    /* this is for small problems that can */
                                 /* probably be ignored, but that careful
 */
                                 /* people may want to know about */
 #define CCTK_WARN_DEBUG    4    /* these messages are probably useful */
                                 /* only for debugging purposes */

 so level 3 is actually "PICKY", not "DEBUG".  This means that unless
 something is actually wrong, or might be wrong, the warning should not be
 issued (in contrast to DEBUG).  Usually warnings are there for a reason,
 and hiding them could lead to hard-to-diagnose problems.   If all the
 warning calls in the code respected these definitions (unlikely), then
 possible sensible defaults for the log level would be 2 or 3.  Since
 scientists are supposed to be "careful people", we should all be using 3,
 which is the current default in SimFactory :)

 The interpretation of the command line arguments is given here:

 http://cactuscode.org/documentation/usersguide/UsersGuidech7.html#x10-36000B3.1

 Actually, I would be in favour of changing the Cactus default from 0 to 3.
 Then we could remove the "-L 3" in simfactory as the Cactus default would
 be suitable.  What do people think of this option?

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