AvoidBehavior should not consider Vertices as nearby entities

oguzeroglu · Aug 8, 2020 · 6fe9573 · 6fe9573
1 parent 444b573
commit 6fe9573
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/js/steering/behavior/AvoidBehavior.js b/js/steering/behavior/AvoidBehavior.js
@@ -1,6 +1,7 @@
 import { SteeringBehavior } from "./SteeringBehavior";
 import { Box } from "../../core/Box";
 import { Vector3D } from "../../core/Vector3D";
+import { Vertex } from "../../core/Vertex";
 import { VectorPool } from "../../core/VectorPool";
 import { logger } from "../../debug/Logger";
 
@@ -34,6 +35,11 @@ AvoidBehavior.prototype.findMostThreateningObstacle = function(steerable){
   box.expandByPoint(steerable.box.max);
 
   steerable.executeForEachCloseEntity(function(entity){
+
+    if (entity instanceof Vertex){
+      return;
+    }
+
     if (box.intersectsBox(entity.box)){
       if (!mostThreatening){
         mostThreatening = entity;

diff --git a/test/steering/behavior/AvoidBehaviorTest.js b/test/steering/behavior/AvoidBehaviorTest.js
@@ -57,6 +57,13 @@ describe("AvoidBehavior", function(){
     var avoidBehavior = new Kompute.AvoidBehavior({ maxSeeAhead: 50000, maxAvoidForce: 100 });
 
     expect(avoidBehavior.findMostThreateningObstacle(steerable)).to.eql(null);
+
+    var graph = new Kompute.Graph();
+    graph.addVertex(new Kompute.Vector3D(0, 0, 0));
+
+    world.insertGraph(graph);
+
+    expect(avoidBehavior.findMostThreateningObstacle(steerable)).to.eql(null);
   });
 
   it("should find most mostThreatening if not going towards the obstacle [hits obstacle by size]", function(){