From 40af3d68bf4f04bc0eb4621325638e8a6cb842e7 Mon Sep 17 00:00:00 2001 From: Xevion Date: Wed, 6 May 2020 13:35:44 -0500 Subject: [PATCH] remove unused onValidate, rename regenerate point method --- procedural-placement/Assets/PointRendering.cs | 16 +--------------- procedural-placement/Assets/SamplingEditor.cs | 2 +- 2 files changed, 2 insertions(+), 16 deletions(-) diff --git a/procedural-placement/Assets/PointRendering.cs b/procedural-placement/Assets/PointRendering.cs index a7538df..d1eeecd 100644 --- a/procedural-placement/Assets/PointRendering.cs +++ b/procedural-placement/Assets/PointRendering.cs @@ -33,22 +33,8 @@ public class PointRendering : MonoBehaviour { private int _retryAttempts; public List points; - - public void OnValidate() { - Debug.Log("Firing onValidate!"); - // Check to see if point position related values changed. SphereSize is rendering only, so we ignore it. - if (numPoints != _prevNumPoints || _prevRegionSize != regionSize - || samplingMethod != _samplingMethod || radius != _radius || retryAttempts != _retryAttempts) { - // Update the property tracking values - _prevNumPoints = numPoints; - _prevRegionSize = regionSize; - _samplingMethod = samplingMethod; - _radius = radius; - _retryAttempts = retryAttempts; - } - } - public void regeneratePoints() { + public void RegeneratePoints() { switch (samplingMethod) { case SamplingTypes.Random: points = PointGeneration.random_sampling(numPoints, regionSize); diff --git a/procedural-placement/Assets/SamplingEditor.cs b/procedural-placement/Assets/SamplingEditor.cs index d3b63e9..3a3d65d 100644 --- a/procedural-placement/Assets/SamplingEditor.cs +++ b/procedural-placement/Assets/SamplingEditor.cs @@ -59,7 +59,7 @@ public class SamplingEditor : Editor { // Handling for different types of point sampling methods if (regenerate) { float t1 = Time.realtimeSinceStartup; - points.regeneratePoints(); + points.RegeneratePoints(); float t2 = Time.realtimeSinceStartup; this._message = $"{points.points.Count:n0} points in {t2 - t1:0.0000}s"; }