BUG: fix the wind velocity factors usage and better visualization of uniform distributions in Stochastic Classes (#783)

* wind factor bug corrected

the wind factor wasn't applied to the env.wind_velocity properties

* BUG: StochasticModel visualize attributes of a uniform distribution

It showed the nominal and the standard deviation values and it doesn't make sense in a uniform distribution. In a np.random.uniform the 'nominal value' is the lower bound of the distribution, and the 'standard deviation' value is the upper bound. Now, a new condition has been added for the uniform distributions where the mean and semi range are calculated and showed. This way the visualize_attribute function will show the whole range where the random values are uniformly taken in

* variable names corrections

* Corrections requested by the pylint test

* ENH: more intuitive uniform distribution display in StochasticModel

Co-authored-by: MateusStano <[email protected]>

---------

Co-authored-by: MateusStano <[email protected]>

Author: kevin-alcaniz

rocketpy/stochastic/stochastic_environment.py

@@ -184,5 +184,6 @@ def create_object(self):
                     # get original attribute value and multiply by factor
                     attribute_name = f"_{key.replace('_factor', '')}"
                     value = getattr(self, attribute_name) * value
+                    key = f"{key.replace('_factor', '')}"
                 setattr(self.obj, key, value)
         return self.obj

rocketpy/stochastic/stochastic_model.py

@@ -514,11 +514,19 @@ def format_attribute(attr, value):
                 )
             elif isinstance(value, tuple):
                 nominal_value, std_dev, dist_func = value
-                return (
-                    f"\t{attr.ljust(max_str_length)} "
-                    f"{nominal_value:.5f} ± "
-                    f"{std_dev:.5f} ({dist_func.__name__})"
-                )
+                if callable(dist_func) and dist_func.__name__ == "uniform":
+                    lower_bound = nominal_value
+                    upper_bound = std_dev
+                    return (
+                        f"\t{attr.ljust(max_str_length)} "
+                        f"{lower_bound:.5f}, {upper_bound:.5f} ({dist_func.__name__})"
+                    )
+                else:
+                    return (
+                        f"\t{attr.ljust(max_str_length)} "
+                        f"{nominal_value:.5f} ± "
+                        f"{std_dev:.5f} ({dist_func.__name__})"
+                    )
             return None
 
         attributes = {k: v for k, v in self.__dict__.items() if not k.startswith("_")}