Suggestions from reviewer

Author: Nicola-Fonzi

SU2_PY/FSI_tools/FSI_config.py

@@ -124,23 +124,19 @@ def applyDefaults(self):
 
         if "MAPPING_MODES" not in self._ConfigContent:
             self._ConfigContent["MAPPING_MODES"] = "NO"
-            print("WARNING: MAPPING_MODES keyword was not found in the configuration file of the interface, setting to NO")
+            print("MAPPING_MODES keyword was not found in the configuration file of the interface, setting to NO")
 
         if "IMPOSED_MOTION" not in self._ConfigContent:
             self._ConfigContent["IMPOSED_MOTION"] = "NO"
-            print("WARNING: IMPOSED_MOTION keyword was not found in the configuration file of the interface, setting to NO")
+            print("IMPOSED_MOTION keyword was not found in the configuration file of the interface, setting to NO")
 
         if self._ConfigContent["IMPOSED_MOTION"] == "YES":
             if self._ConfigContent["AITKEN_RELAX"] != "STATIC" or self._ConfigContent["AITKEN_PARAM"] != 1.0:
-                print("WARNING: Setting the Aitken parameter as static and equal to 1 for IMPOSED solver")
-                self._ConfigContent["AITKEN_RELAX"] = "STATIC"
-                self._ConfigContent["AITKEN_PARAM"] = 1.0
+                raise Exception("When imposing motion, the Aitken parameter must be static and equal to 1")
 
         if self._ConfigContent["RESTART_SOL"] == "YES":
             if self._ConfigContent["TIME_TRESHOLD"] != -1:
-                print("WARNING: Setting the time threshold to -1 for immediate coupling when using restart")
-                self._ConfigContent["TIME_TRESHOLD"] = -1
+                raise Exception("When restarting a simulation, the time threshold must be -1 for immediate coupling")
 
         if self._ConfigContent["MAPPING_MODES"] == "YES" and self._ConfigContent["CSD_SOLVER"]!="NATIVE":
-            self._ConfigContent["CSD_SOLVER"]="NATIVE"
-            print("WARNING: It has been requested to map modes, but the native solver was not selected. It is being selected automatically")
+            raise Exception("Mapping modes only works with the native solver")

SU2_PY/fsi_computation.py

@@ -57,7 +57,7 @@ def main():
   (options, args)=parser.parse_args()
 
   if options.with_MPI:
-    from mpi4py import MPI  # MPI is initialized from now by python and can be continued in C++ !
+    from mpi4py import MPI  # MPI is initialized from now by python and can be continued in C++
     comm = MPI.COMM_WORLD
     myid = comm.Get_rank()
     numberPart = comm.Get_size()
@@ -112,7 +112,7 @@ def main():
   if have_MPI:
     comm.barrier()
 
-  # --- Initialize the solid solver --- # (!! for now we are using only serial solid solvers)
+  # --- Initialize the solid solver --- #
   # Serial solvers
   if CSD_Solver in ["NATIVE"]:
     if myid == rootProcess:
@@ -127,6 +127,7 @@ def main():
     else:
       SolidSolver = None
   # Parallel solvers
+  # For now we are only using serial solvers
   else:
     raise Exception('\n Invalid solid solver option')