MNT: lint corrections

Author: aZira371

rocketpy/rocket/parachute.py

@@ -46,11 +46,11 @@ class Parachute:
 
         - The string "apogee" which triggers the parachute at apogee, i.e.,
           when the rocket reaches its highest point and starts descending.
-        
+
         - The string "launch + X" where X is a number in seconds. The parachute
           will be ejected X seconds after launch (t=0). This is useful for
           simulating delay charges that activate at a fixed time from launch.
-        
+
         - The string "burnout + X" where X is a number in seconds. The parachute
           will be ejected X seconds after motor burnout. This is useful for
           simulating delay charges in motors with delay elements.
@@ -59,8 +59,8 @@ class Parachute:
         Trigger function created from the trigger used to evaluate the trigger
         condition for the parachute ejection system. It is a callable function
         that takes six arguments: Freestream pressure in Pa, Height above
-        ground level in meters, the state vector of the simulation, sensors 
-        list, current time t, and rocket object. It returns ``True`` if the 
+        ground level in meters, the state vector of the simulation, sensors
+        list, current time t, and rocket object. It returns ``True`` if the
         parachute ejection system should be triggered and ``False`` otherwise.
 
         .. note:
@@ -252,6 +252,7 @@ def triggerfunc(p, h, y, sensors, t=None, rocket=None):
 
                 def triggerfunc(p, h, y, sensors, t=None, rocket=None):
                     return trigger(p, h, y, sensors)
+
             self.triggerfunc = triggerfunc
 
         elif isinstance(trigger, (int, float)):
@@ -266,7 +267,7 @@ def triggerfunc(p, h, y, sensors, t=None, rocket=None):  # pylint: disable=unuse
 
         elif isinstance(trigger, str):
             trigger_lower = trigger.lower().strip()
-            
+
             if trigger_lower == "apogee":
                 # The parachute is deployed at apogee
                 def triggerfunc(p, h, y, sensors, t=None, rocket=None):  # pylint: disable=unused-argument
@@ -276,7 +277,7 @@ def triggerfunc(p, h, y, sensors, t=None, rocket=None):  # pylint: disable=unuse
                     return y[5] < 0
 
                 self.triggerfunc = triggerfunc
-            
+
             elif "+" in trigger_lower:
                 # Time-based trigger: "launch + X" or "burnout + X"
                 parts = trigger_lower.split("+")
@@ -286,7 +287,7 @@ def triggerfunc(p, h, y, sensors, t=None, rocket=None):  # pylint: disable=unuse
                         + "Expected format: 'launch + delay' or 'burnout + delay' "
                         + "where delay is a number in seconds."
                     )
-                
+
                 event = parts[0].strip()
                 try:
                     delay = float(parts[1].strip())
@@ -295,32 +296,32 @@ def triggerfunc(p, h, y, sensors, t=None, rocket=None):  # pylint: disable=unuse
                         f"Invalid delay value in trigger '{trigger}' for parachute '{self.name}'. "
                         + "Delay must be a number in seconds."
                     )
-                
+
                 if event == "launch":
                     # Deploy at launch time + delay
                     def triggerfunc(p, h, y, sensors, t=None, rocket=None):  # pylint: disable=unused-argument
                         if t is None:
                             return False
                         return t >= delay
-                    
+
                     self.triggerfunc = triggerfunc
-                
+
                 elif event == "burnout":
                     # Deploy at motor burnout time + delay
                     def triggerfunc(p, h, y, sensors, t=None, rocket=None):  # pylint: disable=unused-argument
                         if t is None or rocket is None:
                             return False
                         burnout_time = rocket.motor.burn_out_time
                         return t >= burnout_time + delay
-                    
+
                     self.triggerfunc = triggerfunc
-                
+
                 else:
                     raise ValueError(
                         f"Invalid time-based trigger event '{event}' for parachute '{self.name}'. "
                         + "Supported events are 'launch' and 'burnout'."
                     )
-            
+
             else:
                 raise ValueError(
                     f"Unable to set the trigger function for parachute '{self.name}'. "

tests/integration/test_parachute_time_trig.py

@@ -10,14 +10,14 @@ def test_flight_with_launch_plus_delay_trigger(
     example_spaceport_env, calisto_motorless, cesaroni_m1670
 ):
     """Test a complete flight simulation with a 'launch + X' parachute trigger.
-    
+
     This simulates a rocket with a delay charge that deploys the parachute at a
     fixed time after launch, similar to model rockets without avionics.
     """
     # Use existing rocket and add motor
     rocket = calisto_motorless
     rocket.add_motor(cesaroni_m1670, position=-1.373)
-    
+
     # Add a parachute with "launch + 5" trigger (5 seconds after launch)
     rocket.add_parachute(
         name="Main",
@@ -27,7 +27,7 @@ def test_flight_with_launch_plus_delay_trigger(
         lag=1.5,
         noise=(0, 8.3, 0.5),
     )
-    
+
     # Run flight simulation
     flight = Flight(
         environment=example_spaceport_env,
@@ -37,15 +37,15 @@ def test_flight_with_launch_plus_delay_trigger(
         heading=0,
         terminate_on_apogee=False,
     )
-    
+
     # Verify that the parachute was deployed at approximately the right time
     # The parachute should deploy at t=5s + lag=1.5s = 6.5s (fully deployed)
     assert flight.t is not None
-    
+
     # Check that parachute deployment happened (should have parachute_cd_s set)
     # This attribute is set when parachute is deployed
     assert hasattr(flight, "parachute_cd_s")
-    
+
     # Verify simulation completed successfully
     assert flight.apogee_time > 0
 
@@ -55,17 +55,17 @@ def test_flight_with_burnout_plus_delay_trigger(
     example_spaceport_env, calisto_motorless, cesaroni_m1670
 ):
     """Test a complete flight simulation with a 'burnout + X' parachute trigger.
-    
+
     This simulates a rocket with a motor delay charge that deploys the parachute
     at a fixed time after motor burnout, typical of model rocket motors.
     """
     # Use existing rocket and add motor
     rocket = calisto_motorless
     rocket.add_motor(cesaroni_m1670, position=-1.373)
-    
+
     # Get motor burnout time
     motor_burnout = rocket.motor.burn_out_time
-    
+
     # Add a parachute with "burnout + 3" trigger (3 seconds after burnout)
     rocket.add_parachute(
         name="Main",
@@ -75,7 +75,7 @@ def test_flight_with_burnout_plus_delay_trigger(
         lag=1.5,
         noise=(0, 8.3, 0.5),
     )
-    
+
     # Run flight simulation
     flight = Flight(
         environment=example_spaceport_env,
@@ -85,17 +85,17 @@ def test_flight_with_burnout_plus_delay_trigger(
         heading=0,
         terminate_on_apogee=False,
     )
-    
+
     # Verify that the parachute was deployed
     assert flight.t is not None
-    
+
     # Check that parachute deployment happened
     assert hasattr(flight, "parachute_cd_s")
-    
+
     # The parachute should deploy after motor burnout + delay
     # Expected deployment at burnout_time + 3s + lag
     expected_deploy_time = motor_burnout + 3.0 + 1.5
-    
+
     # Verify simulation completed successfully and parachute deployed after burnout
     assert flight.apogee_time > 0
     # The simulation should run past the expected deployment time
@@ -107,13 +107,13 @@ def test_flight_with_multiple_time_based_parachutes(
     example_spaceport_env, calisto_motorless, cesaroni_m1670
 ):
     """Test a flight with multiple time-based parachutes (drogue and main).
-    
+
     This simulates a dual-deployment system using time-based triggers.
     """
     # Use existing rocket and add motor
     rocket = calisto_motorless
     rocket.add_motor(cesaroni_m1670, position=-1.373)
-    
+
     # Add drogue parachute - deploys at burnout + 2 seconds
     rocket.add_parachute(
         name="Drogue",
@@ -123,7 +123,7 @@ def test_flight_with_multiple_time_based_parachutes(
         lag=0.5,
         noise=(0, 8.3, 0.5),
     )
-    
+
     # Add main parachute - deploys at burnout + 10 seconds
     rocket.add_parachute(
         name="Main",
@@ -133,7 +133,7 @@ def test_flight_with_multiple_time_based_parachutes(
         lag=1.0,
         noise=(0, 8.3, 0.5),
     )
-    
+
     # Run flight simulation
     flight = Flight(
         environment=example_spaceport_env,
@@ -143,11 +143,11 @@ def test_flight_with_multiple_time_based_parachutes(
         heading=0,
         terminate_on_apogee=False,
     )
-    
+
     # Verify simulation completed successfully
     assert flight.t is not None
     assert flight.apogee_time > 0
-    
+
     # Check that parachute deployment happened
     assert hasattr(flight, "parachute_cd_s")
 
@@ -157,13 +157,13 @@ def test_flight_with_mixed_trigger_types(
     example_spaceport_env, calisto_motorless, cesaroni_m1670
 ):
     """Test a flight with both time-based and traditional parachute triggers.
-    
+
     This ensures backward compatibility when mixing trigger types.
     """
     # Use existing rocket and add motor
     rocket = calisto_motorless
     rocket.add_motor(cesaroni_m1670, position=-1.373)
-    
+
     # Add drogue parachute with traditional apogee trigger
     rocket.add_parachute(
         name="Drogue",
@@ -173,7 +173,7 @@ def test_flight_with_mixed_trigger_types(
         lag=0.5,
         noise=(0, 8.3, 0.5),
     )
-    
+
     # Add main parachute with altitude trigger
     rocket.add_parachute(
         name="Main",
@@ -183,7 +183,7 @@ def test_flight_with_mixed_trigger_types(
         lag=1.0,
         noise=(0, 8.3, 0.5),
     )
-    
+
     # Run flight simulation
     flight = Flight(
         environment=example_spaceport_env,
@@ -193,10 +193,10 @@ def test_flight_with_mixed_trigger_types(
         heading=0,
         terminate_on_apogee=False,
     )
-    
+
     # Verify simulation completed successfully
     assert flight.t is not None
     assert flight.apogee_time > 0
-    
+
     # Check that parachute deployment happened
-    assert hasattr(flight, "parachute_cd_s")
+    assert hasattr(flight, "parachute_cd_s")