From 586dc2c43820ad1a0cba3ccced729cb228d691e6 Mon Sep 17 00:00:00 2001 From: Lamont Granquist Date: Wed, 8 Nov 2023 12:59:14 -0800 Subject: [PATCH] Use propellantStability directly This lets the ThrustController and NodeExecutor use the 0.996 to 1.0 range as where to apply RCS to get back up to 1.0 Signed-off-by: Lamont Granquist --- MechJeb2/MechJebModuleNodeExecutor.cs | 2 +- MechJeb2/MechJebModuleThrustController.cs | 6 +- MechJeb2/PartExtensions.cs | 2 +- MechJeb2/VesselState.cs | 137 +--------------------- 4 files changed, 10 insertions(+), 137 deletions(-) diff --git a/MechJeb2/MechJebModuleNodeExecutor.cs b/MechJeb2/MechJebModuleNodeExecutor.cs index d7cace64..48498184 100644 --- a/MechJeb2/MechJebModuleNodeExecutor.cs +++ b/MechJeb2/MechJebModuleNodeExecutor.cs @@ -134,7 +134,7 @@ private void DoRCS(FlightCtrlState s) if (State != States.LEAD || RCSOnly) return; - if (!Core.Thrust.AutoRCSUllaging || VesselState.lowestUllage == VesselState.UllageState.VeryStable) + if (!Core.Thrust.AutoRCSUllaging || VesselState.lowestUllage >= 1.0) return; if (!Vessel.hasEnabledRCSModules()) diff --git a/MechJeb2/MechJebModuleThrustController.cs b/MechJeb2/MechJebModuleThrustController.cs index c4762e01..c73a118b 100644 --- a/MechJeb2/MechJebModuleThrustController.cs +++ b/MechJeb2/MechJebModuleThrustController.cs @@ -531,7 +531,7 @@ public override void Drive(FlightCtrlState s) /* prevent unstable ignitions */ if (LimitToPreventUnstableIgnition && s.mainThrottle > 0.0F && ThrottleLimit > 0.0F) { - if (VesselState.lowestUllage < VesselState.UllageState.Stable) + if (VesselState.lowestUllage < 0.95) // needs to match 'unstable' value in RF { ScreenMessages.PostScreenMessage(_preventingUnstableIgnitionsMessage); Debug.Log("MechJeb Unstable Ignitions: preventing ignition in state: " + VesselState.lowestUllage); @@ -886,10 +886,10 @@ private void ProcessUllage(FlightCtrlState s) if (!Vessel.hasEnabledRCSModules()) return; - if (VesselState.lowestUllage == VesselState.UllageState.VeryStable && VesselState.time > _ullageUntil) + if (VesselState.lowestUllage >= 1.0 && VesselState.time > _ullageUntil) return; - if (VesselState.lowestUllage != VesselState.UllageState.VeryStable) + if (VesselState.lowestUllage < 1.0) _ullageUntil = VesselState.time + MIN_RCS_TIME; // limit the throttle only if we aren't already burning (don't waste ignitions) diff --git a/MechJeb2/PartExtensions.cs b/MechJeb2/PartExtensions.cs index b1479957..6b617c98 100644 --- a/MechJeb2/PartExtensions.cs +++ b/MechJeb2/PartExtensions.cs @@ -57,7 +57,7 @@ public static bool UnstableUllage(this Part p) return false; if (VesselState.RFullageSetField.GetValue(eng) is object ullageSet) if (VesselState.RFGetUllageStabilityMethod.Invoke(ullageSet, Array.Empty()) is double propellantStability) - if (propellantStability < VesselState.RFveryStableValue) + if (propellantStability < 0.996) return true; } catch (ArgumentException) diff --git a/MechJeb2/VesselState.cs b/MechJeb2/VesselState.cs index b3f662f2..2766f916 100644 --- a/MechJeb2/VesselState.cs +++ b/MechJeb2/VesselState.cs @@ -30,25 +30,8 @@ public class VesselState // RealFuels.Ullage.UllageSet GetUllageStability method to call via reflection public static MethodInfo RFGetUllageStabilityMethod; - // RealFuels.Ullage.UllageSimulator fields to determine ullage status - public static double RFveryStableValue; - private static double RFstableValue; - private static double RFriskyValue; - private static double RFveryRiskyValue; - private static double RFunstableValue; - - public enum UllageState - { - VeryUnstable, - Unstable, - VeryRisky, - Risky, - Stable, - VeryStable // "Nominal" also winds up here - } - // lowestUllage is always VeryStable without RealFuels installed - public UllageState lowestUllage => einfo.lowestUllage; + public double lowestUllage => einfo.lowestUllage; public static bool isLoadedFAR; @@ -418,103 +401,6 @@ public void InitReflection() isLoadedRealFuels = false; } - FieldInfo RFveryStableField = ReflectionUtils.GetFieldByReflection("RealFuels", "RealFuels.Ullage.UllageSimulator", "veryStable", - BindingFlags.NonPublic | BindingFlags.Static); - if (RFveryStableField == null) - { - Debug.Log("MechJeb BUG: RealFuels loaded, but RealFuels.Ullage.UllageSimulator has no veryStable field, disabling RF"); - isLoadedRealFuels = false; - } - - try - { - RFveryStableValue = (double)RFveryStableField.GetValue(null); - } - catch (Exception e1) - { - Debug.Log( - "MechJeb BUG Exception thrown while getting veryStable value from RealFuels, ullage integration disabled: " + e1.Message); - isLoadedRealFuels = false; - return; - } - - FieldInfo RFstableField = ReflectionUtils.GetFieldByReflection("RealFuels", "RealFuels.Ullage.UllageSimulator", "stable", - BindingFlags.NonPublic | BindingFlags.Static); - if (RFstableField == null) - { - Debug.Log("MechJeb BUG: RealFuels loaded, but RealFuels.Ullage.UllageSimulator has no stable field, disabling RF"); - isLoadedRealFuels = false; - } - - try - { - RFstableValue = (double)RFstableField.GetValue(null); - } - catch (Exception e2) - { - Debug.Log("MechJeb BUG Exception thrown while getting stable value from RealFuels, ullage integration disabled: " + e2.Message); - isLoadedRealFuels = false; - return; - } - - FieldInfo RFriskyField = ReflectionUtils.GetFieldByReflection("RealFuels", "RealFuels.Ullage.UllageSimulator", "risky", - BindingFlags.NonPublic | BindingFlags.Static); - if (RFriskyField == null) - { - Debug.Log("MechJeb BUG: RealFuels loaded, but RealFuels.Ullage.UllageSimulator has no risky field, disabling RF"); - isLoadedRealFuels = false; - } - - try - { - RFriskyValue = (double)RFriskyField.GetValue(null); - } - catch (Exception e3) - { - Debug.Log("MechJeb BUG Exception thrown while getting risky value from RealFuels, ullage integration disabled: " + e3.Message); - isLoadedRealFuels = false; - return; - } - - FieldInfo RFveryRiskyField = ReflectionUtils.GetFieldByReflection("RealFuels", "RealFuels.Ullage.UllageSimulator", "veryRisky", - BindingFlags.NonPublic | BindingFlags.Static); - if (RFveryRiskyField == null) - { - Debug.Log("MechJeb BUG: RealFuels loaded, but RealFuels.Ullage.UllageSimulator has no veryRisky field, disabling RF"); - isLoadedRealFuels = false; - } - - try - { - RFveryRiskyValue = (double)RFveryRiskyField.GetValue(null); - } - catch (Exception e4) - { - Debug.Log("MechJeb BUG Exception thrown while getting veryRisky value from RealFuels, ullage integration disabled: " + - e4.Message); - isLoadedRealFuels = false; - return; - } - - FieldInfo RFunstableField = ReflectionUtils.GetFieldByReflection("RealFuels", "RealFuels.Ullage.UllageSimulator", "unstable", - BindingFlags.NonPublic | BindingFlags.Static); - if (RFunstableField == null) - { - Debug.Log("MechJeb BUG: RealFuels loaded, but RealFuels.Ullage.UllageSimulator has no unstable field, disabling RF"); - isLoadedRealFuels = false; - } - - try - { - RFunstableValue = (double)RFunstableField.GetValue(null); - } - catch (Exception e5) - { - Debug.Log("MechJeb BUG Exception thrown while getting unstable value from RealFuels, ullage integration disabled: " + e5.Message); - isLoadedRealFuels = false; - return; - } - if (isLoadedRealFuels) { Debug.Log("MechJeb: RealFuels Assembly is wired up properly"); @@ -1497,7 +1383,7 @@ public class EngineInfo public readonly Vector6 torqueDiffThrottle = new Vector6(); // lowestUllage is always VeryStable without RealFuels installed - public UllageState lowestUllage = UllageState.VeryStable; + public double lowestUllage; public struct FuelRequirement { @@ -1523,7 +1409,7 @@ public void Update(Vector3d c, Vessel vessel) resourceRequired.Clear(); - lowestUllage = UllageState.VeryStable; + lowestUllage = 1.0; CoM = c; @@ -1623,22 +1509,9 @@ public void CheckUllageStatus(ModuleEngines e) return; } - UllageState propellantState; - - if (propellantStability >= RFveryStableValue) - propellantState = UllageState.VeryStable; - else if (propellantStability >= RFstableValue) - propellantState = UllageState.Stable; - else if (propellantStability >= RFriskyValue) - propellantState = UllageState.Risky; - else if (propellantStability >= RFveryRiskyValue) - propellantState = UllageState.VeryRisky; - else - propellantState = UllageState.Unstable; - - if (propellantState < lowestUllage) + if (propellantStability < lowestUllage) { - lowestUllage = propellantState; + lowestUllage = propellantStability; } }