Caching bounds for primary colliders instead of regenerating every frame

martindevans · Jul 1, 2021 · 5781a5d · 5781a5d
1 parent 6ba5f79
commit 5781a5d
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/ShipCombatCore/ShipCombatCore.csproj b/ShipCombatCore/ShipCombatCore.csproj
@@ -6,7 +6,7 @@
     <TargetFramework>netstandard2.1</TargetFramework>
     <Authors>Martin Evans</Authors>
     <Product />
-    <Version>23.0.0</Version>
+    <Version>24.0.0</Version>
   </PropertyGroup>
 
   <ItemGroup>

diff --git a/ShipCombatCore/Simulation/Behaviours/SphereColliderPrimary.cs b/ShipCombatCore/Simulation/Behaviours/SphereColliderPrimary.cs
@@ -30,7 +30,7 @@ public class SphereColliderPrimary
 
         private readonly HashSet<SphereColliderPrimary> _queryResults = new();
 
-        public BoundingBox Bounds => new(new BoundingSphere(Position, Radius));
+        public BoundingBox Bounds { get; private set; }
 
         public override void Initialise(INamedDataProvider? initialisationData)
         {
@@ -40,6 +40,11 @@ public override void Initialise(INamedDataProvider? initialisationData)
             _secondaryManager = Owner.Scene!.GetManager<SphereColliderSecondary.Manager>();
         }
 
+        protected void InitialiseBounds()
+        {
+            Bounds = new(new BoundingSphere(Position, Radius));
+        }
+
         public override void CreateProperties(Entity.ConstructionContext context)
         {
             _radius = context.CreateProperty(PropertyNames.SphereRadius);
@@ -95,6 +100,7 @@ public override void Add(SphereColliderPrimary behaviour)
             {
                 base.Add(behaviour);
 
+                behaviour.InitialiseBounds();
                 var min = ToBucket(behaviour.Bounds.Min);
                 var max = ToBucket(behaviour.Bounds.Max);
 

diff --git a/SpaceShipCombatSimulator/SpaceShipCombatSimulator.csproj b/SpaceShipCombatSimulator/SpaceShipCombatSimulator.csproj
@@ -4,7 +4,7 @@
     <Nullable>enable</Nullable>
     <OutputType>Exe</OutputType>
     <TargetFramework>net5.0</TargetFramework>
-    <Version>23.0.0</Version>
+    <Version>24.0.0</Version>
   </PropertyGroup>
 
   <ItemGroup>