From 4bd56cb14a52cb468d26394a07fea5fbee66ef55 Mon Sep 17 00:00:00 2001 From: josh-deb Date: Tue, 12 Sep 2023 16:01:35 -0700 Subject: [PATCH] fix(set_gem_timing): updated logic to reflect application layer concers: do not exit the program if PV and IOC software are out of sync --- scripts/set_gem_timing | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/scripts/set_gem_timing b/scripts/set_gem_timing index 26cae75d..feb8d59d 100755 --- a/scripts/set_gem_timing +++ b/scripts/set_gem_timing @@ -21,7 +21,7 @@ else fi if [[ -n $2 ]]; then - echo "Warning. Must pass strictly one argument, additional arg: $2 is ignored" + echo "Error. Must pass strictly one argument, at least one additional arg: $2, please run again with exclusively [NC or SC] as argument" exit 1 fi @@ -36,8 +36,7 @@ SET_MODE_CMD="caput KFE:CAM:TPR:02:MODE $1" echo "Current OP_MODE is: $OP_MODE" if [[ $OP_MODE = "$1" ]]; then - echo "Already in requested mode: $OP_MODE exitting with NOP" - exit 1 + echo "Warning. Already in requested mode: $OP_MODE" #Note: (josh) we are opting to consider this not error worthy else echo "Actuating requested timing change, caputing $1 to KFE:CAM:TPR:02:MODE" $SET_MODE_CMD