included walltime clock in CCTK_VWarn and CCTK_VInfo output

Comments (5)

1. 

   Erik Schnetter

   • removed comment

   The parameter setting cactus::info_format = "human-readable time stamp" should do this.

   • 2018-09-27T08:40:37+00:00
2. 

   Roland Haas reporter

   • changed status to resolved
   • removed comment

   Great! I had no idea that this existed. Thank you.

   • 2018-09-27T08:57:14+00:00
3. 

   Erik Schnetter

   • removed comment

   Should we change the default?

   • 2018-09-27T09:00:07+00:00
4. 

   Roland Haas reporter

   • changed status to open
   • removed comment

   Maybe.

   Changing the default probably break people parsing scripts eg that look for the mass from TwoPunctures (see: https://bitbucket.org/simulationtools/simulationtools/src/7d99c8ebdc26a7cc01db97ca4b0d82976e25eb2a/TwoPunctures.m?at=master#TwoPunctures.m-87 and eg StringMatchQ["a Foo", "Foo" (* Foo is pattern *)] -> False).

   Having said that though: yes, I would think that changing the default info (and warning format) might be nice but will take two releases. Namely changing the defaults to:

   • Cactus::highlight_warning_messages = no
   • Cactus::info_format = "human-readable time stamp"

   the latter is a bit verbose since it contains the full date in the form "Wed Jun 30 21:49:08 1993" which is overkill and somewhat hard to parse while we would likely be fine with the HH:MM:SS part.

   • 2018-09-27T09:42:20+00:00
5. 

   Roland Haas reporter

   On second thought, I think the only change to defaults I would like to push for myself would be to make

   Cactus::highlight_warning_messages = no
   

   the default since it removes control characters from log files. Typically highlight_warning_messagesonly makes sense if Cactus’ log goes directly to a terminal since (by default) less or more will not interpret the control characters.

   • 2019-06-29T19:58:24+00:00
6. Log in to comment