Timing information in the regression tests

[junwang-noaa]

Description

In the regression test log files, it would be good if the timing information can be added. This information will allow developers to track when code updates cause significant slowdown

Solution

In the run output files, there is a total wall clock time "The total amount of wall time", we can grab that line to the each test log information.

Related to

Directly reference any issues or PRs in this or other repositories that this is related to, and describe how they are related.

[DusanJovic-NOAA]

$ git diff rt_utils.sh
diff --git a/tests/rt_utils.sh b/tests/rt_utils.sh
index 52f9dd8..a6e982a 100755
--- a/tests/rt_utils.sh
+++ b/tests/rt_utils.sh
@@ -327,6 +327,10 @@ check_results() {
 
   fi
 
+  echo                                               >> ${REGRESSIONTEST_LOG}
+  grep "The total amount of wall time" ${RUNDIR}/out >> ${REGRESSIONTEST_LOG}
+  echo                                               >> ${REGRESSIONTEST_LOG}
+
   echo "Test ${TEST_NR} ${TEST_NAME} ${test_status}" >> ${REGRESSIONTEST_LOG}
   echo                                               >> ${REGRESSIONTEST_LOG}
   echo "Test ${TEST_NR} ${TEST_NAME} ${test_status}"

[junwang-noaa]

Thanks for making this change.

…

On Wed, Mar 10, 2021 at 9:34 AM Dusan Jovic ***@***.***> wrote: $ git diff rt_utils.sh diff --git a/tests/rt_utils.sh b/tests/rt_utils.sh index 52f9dd8..a6e982a 100755 --- a/tests/rt_utils.sh +++ b/tests/rt_utils.sh @@ -327,6 +327,10 @@ check_results() { fi + echo >> ${REGRESSIONTEST_LOG} + grep "The total amount of wall time" ${RUNDIR}/out >> ${REGRESSIONTEST_LOG} + echo >> ${REGRESSIONTEST_LOG} + echo "Test ${TEST_NR} ${TEST_NAME} ${test_status}" >> ${REGRESSIONTEST_LOG} echo >> ${REGRESSIONTEST_LOG} echo "Test ${TEST_NR} ${TEST_NAME} ${test_status}" — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#459 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AI7D6TK5UATOR76N7W5TDNDTC5YMPANCNFSM4Y4JE7RA> .

[DeniseWorthen]

We could combine this with the change for Issue #228?

[junwang-noaa]

Do we have anything to change for issue #228?

…

On Wed, Mar 10, 2021 at 9:59 AM Denise Worthen ***@***.***> wrote: We could combine this with the change for Issue #228 <#228>? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#459 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AI7D6TL4NF5C6NYS7K6EBFTTC53NRANCNFSM4Y4JE7RA> .