ProfileOptions: add some basic integrity checking

pypesto/profile/options.py

@@ -91,3 +91,24 @@ def create_instance(
             return maybe_options
         options = ProfileOptions(**maybe_options)
         return options
+
+    def validate(self):
+        """Check if options are valid.
+
+        Raise ``ValueError`` if not.
+        """
+        if self.min_step_size <= 0:
+            raise ValueError("min_step_size must be > 0.")
+        if self.max_step_size <= 0:
+            raise ValueError("max_step_size must be > 0.")
+        if self.min_step_size > self.max_step_size:
+            raise ValueError("min_step_size must be <= max_step_size.")
+        if self.default_step_size <= 0:
+            raise ValueError("default_step_size must be > 0.")
+        if self.default_step_size > self.max_step_size:
+            raise ValueError("default_step_size must be <= max_step_size.")
+        if self.default_step_size < self.min_step_size:
+            raise ValueError("default_step_size must be >= min_step_size.")
+
+        if self.magic_factor_obj_value < 0 or self.magic_factor_obj_value >= 1:
+            raise ValueError("magic_factor_obj_value must be >= 0 and < 1.")

pypesto/profile/profile.py

@@ -89,6 +89,7 @@ def parameter_profile(
     if profile_options is None:
         profile_options = ProfileOptions()
     profile_options = ProfileOptions.create_instance(profile_options)
+    profile_options.validate()
 
     # create a function handle that will be called later to get the next point
     if isinstance(next_guess_method, str):