jiggle changes

Author: dooly123

Basis/Packages/com.gator-dragon-games.jigglephysics/Scripts/JiggleJobPrepareRender.cs

@@ -40,6 +40,14 @@ public void Execute() {
 
         for (int i = 0; i < personalColliderCount; i++) {
             var collider = personalColliders[i];
+            // Retired slots keep their last matrix, and freshly added ones are counted before the
+            // collider TransformAccessArray flips — so until the read job has visited them their
+            // matrix is whatever it was built with (or all zeros). Drawing either one pins a gizmo
+            // wherever the avatar was when it spawned. worldRadius is only ever set by Read, so it
+            // doubles as "this slot has been sampled at least once".
+            if (!collider.enabled || collider.worldRadius <= 0f) {
+                continue;
+            }
             var position = collider.localToWorldMatrix.c3.xyz;
             switch (collider.type) {
                 case JiggleCollider.JiggleColliderType.Sphere: {
@@ -67,6 +75,9 @@ public void Execute() {
 
         for (int i = 0; i < sceneColliderCount; i++) {
             var collider = sceneColliders[i];
+            if (!collider.enabled || collider.worldRadius <= 0f) {
+                continue;
+            }
             var position = collider.localToWorldMatrix.c3.xyz;
             switch (collider.type) {
                 case JiggleCollider.JiggleColliderType.Sphere: {

Basis/Packages/com.gator-dragon-games.jigglephysics/Scripts/JigglePhysics.cs

@@ -129,9 +129,16 @@ public static void Dispose() {
         jiggleRootLookup = new Dictionary<Transform, JiggleTreeSegment>();
         _globalDirty = true;
         jobs = null;
+        // JiggleRenderer.Dispose released the instancers and chunk buffers, so leaving this latched
+        // would skip the OnEnable that rebuilds them and silently stop drawing gizmos for the rest
+        // of the session (Initialize only runs again on a domain reload).
+        initializedRendering = false;
     }
 
     public static void ScheduleRender() {
+        if (jobs == null) {
+            return;
+        }
         if (!initializedRendering) {
             JiggleRenderer.OnEnable(jobs);
             initializedRendering = true;
@@ -140,6 +147,9 @@ public static void ScheduleRender() {
     }
 
     public static void CompleteRender(Material proceduralMaterial, Mesh sphere, Mesh capsule) {
+        if (jobs == null) {
+            return;
+        }
         if (!initializedRendering) {
             JiggleRenderer.OnEnable(jobs);
             initializedRendering = true;

Basis/Packages/com.gator-dragon-games.jigglephysics/Scripts/JiggleRenderInstancer.cs

@@ -30,13 +30,16 @@ private void GenerateChunks(NativeArray<GPUChunk> chunks, int count) {
             bufferCapacity = desiredChunkCount;
         }
 
-        bufferCount = count;
-        chunkBuffer.SetData(chunks, 0, 0, count);
+        bufferCount = math.clamp(count, 0, desiredChunkCount);
+        if (bufferCount == 0) {
+            return;
+        }
+        chunkBuffer.SetData(chunks, 0, 0, bufferCount);
     }
 
     public void Render(Bounds bounds, Mesh mesh, Material material, NativeArray<GPUChunk> chunks, int count) {
         GenerateChunks(chunks, count);
-        if (chunkBuffer == null) {
+        if (chunkBuffer == null || bufferCount == 0) {
             return;
         }
 
@@ -51,6 +54,9 @@ public void Render(Bounds bounds, Mesh mesh, Material material, NativeArray<GPUC
 
     public void Dispose() {
         chunkBuffer?.Release();
+        chunkBuffer = null;
+        bufferCapacity = 0;
+        bufferCount = 0;
     }
 }
 

Basis/Packages/com.gator-dragon-games.jigglephysics/Scripts/JiggleRenderer.cs

@@ -43,7 +43,7 @@ private static float4 ColorToFloat4(Color color) {
 
     private static void FlipData(JiggleJobs job, double simulatedTime) {
         var sceneColliderCapacity = job.GetSceneColliderCapacity();
-        var personalColliderCapacity = job.GetSceneColliderCapacity();
+        var personalColliderCapacity = job.GetPersonalColliderCapacity();
         var transformCapacity = job.GetTransformCapcity();
 
         int desiredSphereCount = math.max(sceneColliderCapacity + personalColliderCapacity + transformCapacity, 1);
@@ -80,18 +80,21 @@ public static void PrepareRender(JiggleJobs job) {
         jobPrepareRender.sceneColliders = job.GetSceneColliders(out jobPrepareRender.sceneColliderCount);
         jobPrepareRender.outputPoses = job.GetInterpolatedOutputPoses(out jobPrepareRender.transformCount);
         jobPrepareRender.trees = job.GetTrees(out jobPrepareRender.treeCount);
+        // Latch only when the job actually got scheduled. Flagging unconditionally meant a frame
+        // where the sim dependencies weren't ready still completed and re-rendered the previous
+        // frame's chunk data, pinning gizmos at stale positions.
         if (job.TryGetRenderDependencies(out var handle)) {
             handleRender = jobPrepareRender.Schedule(handle);
+            hasHandleRender = true;
         }
-
-        hasHandleRender = true;
     }
 
     public static void FinishRender(Material gpuInstanceMaterial, Mesh sphere, Mesh capsule) {
         if (!hasHandleRender) {
             return;
         }
         handleRender.Complete();
+        hasHandleRender = false;
         if (sphereChunks.IsCreated && sphere != null) {
             sphereInstancer.Render(jobPrepareRender.sphereBounds.Value, sphere, gpuInstanceMaterial, sphereChunks, jobPrepareRender.sphereCount.Value);
         }
@@ -101,6 +104,10 @@ public static void FinishRender(Material gpuInstanceMaterial, Mesh sphere, Mesh
     }
 
     public static void Dispose() {
+        if (hasHandleRender) {
+            handleRender.Complete();
+            hasHandleRender = false;
+        }
         sphereInstancer?.Dispose();
         sphereInstancer = null;
         capsuleInstancer?.Dispose();