STY: solve linting and style remarks.

Author: phmbressan

rocketpy/control/controller.py

@@ -1,5 +1,6 @@
 from inspect import signature
 from typing import Iterable
+
 from rocketpy.tools import from_hex_decode, to_hex_encode
 
 from ..prints.controller_prints import _ControllerPrints

rocketpy/rocket/aero_surface/air_brakes.py

@@ -207,6 +207,7 @@ def all_info(self):
         self.info()
         self.plots.drag_coefficient_curve()
 
+    # pylint: disable=unused-argument
     def to_dict(self, **kwargs):
         return {
             "drag_coefficient_curve": self.drag_coefficient,

rocketpy/rocket/rocket.py

@@ -1,8 +1,8 @@
 import math
+import warnings
+from typing import Iterable
 
 import numpy as np
-from rocketpy.tools import find_obj_from_hash
-from typing import Iterable
 
 from rocketpy.control.controller import _Controller
 from rocketpy.mathutils.function import Function
@@ -23,7 +23,11 @@
 from rocketpy.rocket.aero_surface.generic_surface import GenericSurface
 from rocketpy.rocket.components import Components
 from rocketpy.rocket.parachute import Parachute
-from rocketpy.tools import deprecated, parallel_axis_theorem_from_com
+from rocketpy.tools import (
+    deprecated,
+    find_obj_from_hash,
+    parallel_axis_theorem_from_com,
+)
 
 
 # pylint: disable=too-many-instance-attributes, too-many-public-methods, too-many-instance-attributes

rocketpy/sensors/sensor.py

@@ -271,6 +271,7 @@ def _generic_export_measured_data(self, filename, file_format, data_labels):
                 print(f"Data saved to {filename}")
             return
 
+    # pylint: disable=unused-argument
     def to_dict(self, **kwargs):
         return {
             "sampling_rate": self.sampling_rate,

rocketpy/tools.py

@@ -24,7 +24,6 @@
 from cftime import num2pydate
 from matplotlib.patches import Ellipse
 from packaging import version as packaging_version
-from typing import Iterable
 
 # Mapping of module name and the name of the package that should be installed
 INSTALL_MAPPING = {"IPython": "ipython"}

tests/integration/test_encoding.py

@@ -1,16 +1,15 @@
 import json
 import os
+from unittest.mock import patch
 
 import numpy as np
 import pytest
-from unittest.mock import patch
-
 
 from rocketpy._encoders import RocketPyDecoder, RocketPyEncoder
-
 from rocketpy.tools import from_hex_decode
 
 
+# pylint: disable=unused-argument
 @patch("matplotlib.pyplot.show")
 @pytest.mark.parametrize(
     ["flight_name", "include_outputs"],

tests/integration/test_flight.py

@@ -415,23 +415,6 @@ def test_air_brakes_flight(mock_show, flight_calisto_air_brakes):  # pylint: dis
     test_flight = flight_calisto_air_brakes
     air_brakes = test_flight.rocket.air_brakes[0]
 
-    import json
-    from rocketpy._encoders import RocketPyEncoder, RocketPyDecoder
-
-    with open("test_decode_airbrake.json", "w") as f:
-        json.dump(
-            test_flight,
-            f,
-            cls=RocketPyEncoder,
-            indent=2,
-            discretize=True,
-            include_outputs=False,
-            allow_pickle=True,
-        )
-    with open("test_decode_airbrake.json", "r") as f:
-        test_flight = json.load(f, cls=RocketPyDecoder)
-    air_brakes = test_flight.rocket.air_brakes[0]
-
     assert air_brakes.plots.all() is None
     assert air_brakes.prints.all() is None