fix path contain space

Author: ooooo-create

tools/ci/check_validate.sh

@@ -54,30 +54,32 @@ function prepare_paddle_env() {
 
 function check_torch_validation() {
   LOG "[INFO] Start run validate for changed torch samples ..."
-  MODIFIED_MODEL_PATHS=()
-  for file in $(git diff --name-only develop | grep -E "\bsamples\b/(.*\.py|.*\.json)")
-  do
+  mapfile -t MODIFIED_FILES < <(git diff --name-only develop | grep -E "\bsamples\b/(.*\.py|.*\.json)")
+  declare -A unique_paths
+  for file in "${MODIFIED_FILES[@]}"; do
     LOG "[INFO] Found ${file} modified."
-    model_path=$(dirname ${file})
-    MODIFIED_MODEL_PATHS[${#MODIFIED_MODEL_PATHS[@]}]=$model_path
+    model_path=$(dirname "${file}")
+    unique_paths["$model_path"]=1
   done
-  MODIFIED_MODEL_PATHS=($(echo ${MODIFIED_MODEL_PATHS[@]} | tr ' ' '\n' | sort | uniq))
+  mapfile -t MODIFIED_MODEL_PATHS < <(printf '%s\n' "${!unique_paths[@]}" | sort)
   local total_models=${#MODIFIED_MODEL_PATHS[@]}
   if [ ${total_models} -eq 0 ]; then
     LOG "[INFO] No changed torch sample models detected. Skip torch validation."
     return 0
   fi
-  LOG "[INFO] Validation will run on ${total_models} torch model path(s): ${MODIFIED_MODEL_PATHS[@]}"
+  LOG "[INFO] Validation will run on ${total_models} torch model path(s):"
+  printf "  - %s\n" "${MODIFIED_MODEL_PATHS[@]}" >&2
   fail_name=()
-  for model_path in ${MODIFIED_MODEL_PATHS[@]}
+  for model_path in "${MODIFIED_MODEL_PATHS[@]}"
   do
-    python -m graph_net.torch.validate --model-path ${GRAPH_NET_EXTRACT_WORKSPACE}/${model_path} --graph-net-samples-path ${GRAPH_NET_EXTRACT_WORKSPACE}/samples >&2
+    python -m graph_net.torch.validate --model-path "${GRAPH_NET_EXTRACT_WORKSPACE}/${model_path}" --graph-net-samples-path "${GRAPH_NET_EXTRACT_WORKSPACE}/samples" >&2
     [ $? -ne 0 ] && fail_name[${#fail_name[@]}]="${model_path}"
   done
   local failed_cnt=${#fail_name[@]}
   if [ ${failed_cnt} -ne 0 ]; then
-    LOG "[FATAL] Failed tests (${failed_cnt}/${total_models}): ${fail_name[@]}"
-    echo ${fail_name[@]}
+    LOG "[FATAL] Failed tests (${failed_cnt}/${total_models}):"
+    printf "  - %s\n" "${fail_name[@]}" >&2
+    printf "%s\n" "${fail_name[@]}"
     exit -1
   else
     LOG "[INFO] All torch sample validations passed (0 failures / ${total_models} total)."
@@ -86,30 +88,32 @@ function check_torch_validation() {
 
 function check_paddle_validation() {
   LOG "[INFO] Start run validate for changed paddle samples ..."
-  MODIFIED_MODEL_PATHS=()
-  for file in $(git diff --name-only develop | grep -E "\bpaddle_samples\b/(.*\.py|.*\.json)")
-  do
+  mapfile -t MODIFIED_FILES < <(git diff --name-only develop | grep -E "\bsamples\b/(.*\.py|.*\.json)")
+  declare -A unique_paths
+  for file in "${MODIFIED_FILES[@]}"; do
     LOG "[INFO] Found ${file} modified."
-    model_path=$(dirname ${file})
-    MODIFIED_MODEL_PATHS[${#MODIFIED_MODEL_PATHS[@]}]=$model_path
+    model_path=$(dirname "${file}")
+    unique_paths["$model_path"]=1
   done
-  MODIFIED_MODEL_PATHS=($(echo ${MODIFIED_MODEL_PATHS[@]} | tr ' ' '\n' | sort | uniq))
+  mapfile -t MODIFIED_MODEL_PATHS < <(printf '%s\n' "${!unique_paths[@]}" | sort)
   local total_models=${#MODIFIED_MODEL_PATHS[@]}
   if [ ${total_models} -eq 0 ]; then
     LOG "[INFO] No changed paddle sample models detected. Skip paddle validation."
     return 0
   fi
-  LOG "[INFO] Validation will run on ${total_models} paddle model path(s): ${MODIFIED_MODEL_PATHS[@]}"
+  LOG "[INFO] Validation will run on ${total_models} paddle model path(s):"
+  printf "  - %s\n" "${MODIFIED_MODEL_PATHS[@]}" >&2
   fail_name=()
-  for model_path in ${MODIFIED_MODEL_PATHS[@]}
+  for model_path in "${MODIFIED_MODEL_PATHS[@]}"
   do
-    python -m graph_net.paddle.validate --model-path ${GRAPH_NET_EXTRACT_WORKSPACE}/${model_path} --graph-net-samples-path ${GRAPH_NET_EXTRACT_WORKSPACE}/samples >&2
+    python -m graph_net.paddle.validate --model-path "${GRAPH_NET_EXTRACT_WORKSPACE}/${model_path}" --graph-net-samples-path "${GRAPH_NET_EXTRACT_WORKSPACE}/samples" >&2
     [ $? -ne 0 ] && fail_name[${#fail_name[@]}]="${model_path}"
   done
   local failed_cnt=${#fail_name[@]}
   if [ ${failed_cnt} -ne 0 ]; then
-    LOG "[FATAL] Failed tests (${failed_cnt}/${total_models}): ${fail_name[@]}"
-    echo ${fail_name[@]}
+    LOG "[FATAL] Failed tests (${failed_cnt}/${total_models}):"
+    printf "  - %s\n" "${fail_name[@]}" >&2
+    printf "%s\n" "${fail_name[@]}"
     exit -1
   else
     LOG "[INFO] All paddle sample validations passed (0 failures / ${total_models} total)."