add validator check

Author: HydrogenSulfate

deploy/python_infer/base.py

@@ -101,8 +101,8 @@ def _create_paddle_predictor(
         self,
     ) -> Tuple[paddle_inference.Predictor, paddle_inference.Config]:
         if misc.check_flag_enabled("FLAGS_enable_pir_api"):
-            # PIR mode
-            self.pdmodel_path = self.pdmodel_path.replace(".pdmodel", ".json")
+            # NOTE: Using 'json' as suffix instead of 'pdmodel' in PIR mode
+            self.pdmodel_path = self.pdmodel_path.replace(".pdmodel", ".json", 1)
 
         if not osp.exists(self.pdmodel_path):
             raise FileNotFoundError(
@@ -114,8 +114,8 @@ def _create_paddle_predictor(
                 f"Given 'pdiparams_path': {self.pdiparams_path} does not exist. "
                 "Please check if cfg.INFER.pdiparams_path is correct."
             )
-        config = paddle_inference.Config(self.pdmodel_path, self.pdiparams_path)
 
+        config = paddle_inference.Config(self.pdmodel_path, self.pdiparams_path)
         if self.device == "gpu":
             config.enable_use_gpu(self.gpu_mem, self.gpu_id)
             if self.engine == "tensorrt":

ppsci/solver/solver.py

@@ -277,6 +277,11 @@ def __init__(
                         )
                     _count[metric_name] = 1
             del _count
+        elif self.eval_during_train:
+            raise ValueError(
+                "eval_during_train is enabled but no validator is provided, "
+                "please provide a validator to enable evaluation during training."
+            )
 
         # whether set `stop_gradient=True` for every Tensor if no differentiation involved during evaluation
         if not cfg:
@@ -911,7 +916,6 @@ def export(
             jit.save(static_model, export_path, skip_prune_program=skip_prune_program)
         except Exception as e:
             raise e
-
         logger.message(
             f"Inference model has been exported to: {export_path}, including "
             + (
@@ -920,7 +924,6 @@ def export(
                 else "*.pdmodel, *.pdiparams and *.pdiparams.info files."
             )
         )
-
         jit.enable_to_static(False)
 
         if with_onnx:

ppsci/utils/misc.py

@@ -635,13 +635,13 @@ def plot_curve(
 
 
 def check_flag_enabled(flag_name: str) -> bool:
-    """Split given array into single channel array at axis -1 in order of given keys.
+    """Check whether the flag is enabled.
 
     Args:
         flag_name(str): Flag name to be checked whether enabled or disabled.
 
     Returns:
-        bool: Whether the flag is enabled.
+        bool: Whether given flag name is enabled in environment.
     """
     value = os.getenv(flag_name, False)
     if isinstance(value, str):