Improve performance

- Improved performance of ModuleAutoCutDrogue by harmony patching events to fire when the chutes deploy instead of polling every frame.

Author: SofieBrink

GameData/KSPCommunityPartModules/KSPCommunityPartModules.ckan

@@ -9,6 +9,8 @@ description: >-
 
   This mod is meant as community project, so feel free to propose additional module ideas by opening an issue, or contribute with a pull request on GitHub.
 ksp_version_min: '1.12.3'
+depends:
+  - name: Harmony2
 supports:
   - name: BoringCrewServices
   - name: BluedogDB

Source/KSPCommunityPartModules.csproj

@@ -31,7 +31,7 @@
     <WarningLevel>4</WarningLevel>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="KSPBuildTools" Version="0.0.2-alpha.5" />
+    <PackageReference Include="KSPBuildTools" Version="0.0.2-alpha.7" />
   </ItemGroup>
   <ItemGroup>
     <Compile Include="Modules\ModuleAutoCutDrogue.cs" />
@@ -41,6 +41,11 @@
   <ItemGroup>
     <Content Include="HOW_TO_BUILD.txt" />
   </ItemGroup>
-  <ItemGroup />
+  <ItemGroup>
+    <Reference Include="$(KSPRoot)/GameData/000_Harmony/0Harmony.dll">
+      <Private>False</Private>
+      <CKANIdentifier>Harmony2</CKANIdentifier>
+    </Reference>
+  </ItemGroup>
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
 </Project>

Source/Modules/ModuleAutoCutDrogue.cs

@@ -4,6 +4,8 @@
     Originally By:  Jsolson
     Originally For: Bluedog Design Bureau
 */
+using HarmonyLib;
+using System;
 using System.Linq;
 using UnityEngine;
 
@@ -22,6 +24,10 @@ class ModuleAutoCutDrogue : PartModule
         public bool triggered = false;
 
         private ModuleParachute chute = null;
+
+        // Technically this creates an edge case where if two loaded vessels deploy a main parachute at the same time only one of the vessels will cut its drogue chutes,
+        // but I feel like its unlikely enough to not warrent the effort required to fix it.
+        private static int lastFrame;
         private bool IsChuteDeployed(ModuleParachute pParachute) => pParachute.deploymentState == ModuleParachute.deploymentStates.DEPLOYED
                                                                  || pParachute.deploymentState == ModuleParachute.deploymentStates.SEMIDEPLOYED;
         public override void OnStart(StartState state)
@@ -32,33 +38,68 @@ public override void OnStart(StartState state)
 
             Fields[nameof(autoCutDrogue)].guiActive = !isDrogueChute;
             Fields[nameof(autoCutDrogue)].guiActiveEditor = !isDrogueChute;
+
+            if (!isDrogueChute) ModuleParachuteEvents.OnDeployed += OnParachuteDeployed; 
+            if (!isDrogueChute) ModuleParachuteEvents.OnRepacked += OnParachuteRepacked;
         }
 
-        public void FixedUpdate()
+        public void OnDestroy()
         {
-            if (isDrogueChute || chute == null)
-            {
-                this.isEnabled = false;
-                this.enabled = false;
-                return;
-            }
+            ModuleParachuteEvents.OnDeployed -= OnParachuteDeployed;
+            ModuleParachuteEvents.OnRepacked -= OnParachuteRepacked;
+        }
 
-            if (IsChuteDeployed(chute))
+        private void OnParachuteDeployed(ModuleParachute pChute)
+        {
+            if (pChute == chute && !triggered && lastFrame != Time.frameCount)
             {
-                if (!triggered)
+                var drogues = vessel.FindPartModulesImplementing<ModuleAutoCutDrogue>().Where(d => d.isDrogueChute && d.chute != null);
+                foreach (ModuleAutoCutDrogue d in drogues)
                 {
-                    var drogues = vessel.FindPartModulesImplementing<ModuleAutoCutDrogue>().Where(d => d.isDrogueChute && d.chute != null);
-                    foreach (ModuleAutoCutDrogue d in drogues)
-                    {
-                        if (IsChuteDeployed(d.chute)) d.chute.CutParachute();
-                    }
-                    triggered = true;
+                    if (IsChuteDeployed(d.chute)) d.chute.CutParachute();
                 }
+                lastFrame = Time.frameCount;
+                triggered = true;
             }
-            else if (chute.deploymentState == ModuleParachute.deploymentStates.STOWED)
-            {
-                triggered = false;
-            }
+        }
+
+        private void OnParachuteRepacked(ModuleParachute pChute)
+        {
+            if (pChute == chute && triggered) triggered = false;
+        }
+    }
+
+    [HarmonyPatch(typeof(ModuleParachute))]
+    internal class ModuleParachuteEvents
+    {
+        public static void ModuleManagerPostLoad()
+        {
+            Harmony harmony = new Harmony("KSPCommunityPartModules");
+            harmony.PatchAll();
+        }
+
+        public static event Action<ModuleParachute> OnDeployed;
+        public static event Action<ModuleParachute> OnRepacked;
+
+        [HarmonyPostfix]
+        [HarmonyPatch("OnParachuteSemiDeployed")]
+        static void RaiseEventOnSemiDeployed(ModuleParachute __instance)
+        {
+            OnDeployed?.Invoke(__instance);
+        }
+
+        [HarmonyPostfix]
+        [HarmonyPatch("OnParachuteFullyDeployed")]
+        static void RaiseEventOnFullyDeployed(ModuleParachute __instance)
+        {
+            OnDeployed?.Invoke(__instance);
+        }
+
+        [HarmonyPostfix]
+        [HarmonyPatch("Repack")]
+        static void RaiseEventOnRepack(ModuleParachute __instance)
+        {
+            OnRepacked?.Invoke(__instance);
         }
     }
 }