Comment cleanup

This reverts commit e128f605193b1c3d19bd2c617c9f64fb15959180.

src/engine/db/handlers/dbBuildingHandler.java

@@ -15,6 +15,7 @@ import engine.Enum.ProtectionState;
 import engine.Enum.TaxType;
 import engine.gameManager.BuildingManager;
 import engine.gameManager.DbManager;
+import engine.math.Vector2f;
 import engine.math.Vector3fImmutable;
 import engine.objects.*;
 import org.joda.time.DateTime;
@@ -548,6 +549,63 @@ public class dbBuildingHandler extends dbHandlerBase {
         return false;
     }
 
+    public void LOAD_CONVEX_HULLS() {
+
+        int recordsRead = 0;
+
+        try (Connection connection = DbManager.getConnection();
+             PreparedStatement preparedStatement = connection.prepareStatement("SELECT * FROM static_convex_hulls")) {
+
+            ResultSet rs = preparedStatement.executeQuery();
+
+            while (rs.next()) {
+
+                recordsRead++;
+
+                int meshID = rs.getInt("meshID");
+                String[] hullString = rs.getString("vertices").split(";");
+
+                // Filter things that couldn't be wrapped
+
+                if (hullString.length < 3) {
+                    Logger.error("Mesh : " + meshID + " has less than 3 vertices.");
+                    continue;
+                }
+
+                ArrayList<Vector2f> convexHull = new ArrayList<>();
+                ArrayList<Float> floats = new ArrayList<>();
+
+                for (String read : hullString) {
+
+                    floats.add(Float.parseFloat(read));
+
+                    if (floats.size() == 2) {
+                        convexHull.add(new Vector2f(floats.get(0), floats.get(1)));
+                        floats.clear();
+                    }
+
+                }
+
+                ArrayList<ArrayList<Vector2f>> hullList;
+
+                if (BuildingManager._hull_data.get(meshID) == null) {
+                    hullList = new ArrayList<>();
+                    hullList.add(convexHull);
+                    BuildingManager._hull_data.put(meshID, hullList);
+
+                } else {
+                    hullList = BuildingManager._hull_data.get(meshID);
+                    hullList.add(convexHull);
+                }
+            }
+
+        } catch (SQLException e) {
+            Logger.error(e);
+        }
+
+        Logger.info("read: " + recordsRead + " cached: " + BuildingManager._hull_data.size());
+    }
+
     public HashMap<Integer, ArrayList<BuildingRegions>> LOAD_BUILDING_REGIONS() {
 
         HashMap<Integer, ArrayList<BuildingRegions>> regionList = new HashMap<>();

src/engine/devcmd/cmds/RegionCmd.java

@@ -12,10 +12,22 @@ package engine.devcmd.cmds;
 import engine.Enum;
 import engine.devcmd.AbstractDevCmd;
 import engine.gameManager.BuildingManager;
+import engine.gameManager.NavigationManager;
+import engine.gameManager.ZoneManager;
+import engine.math.Vector3f;
+import engine.math.Vector3fImmutable;
 import engine.objects.*;
 
+import java.awt.*;
+import java.awt.geom.AffineTransform;
+import java.awt.geom.Area;
+import java.awt.geom.Path2D;
+import java.awt.geom.Rectangle2D;
 import java.lang.reflect.Field;
 
+import static java.awt.geom.Path2D.WIND_EVEN_ODD;
+import static java.awt.geom.Path2D.WIND_NON_ZERO;
+
 public class RegionCmd extends AbstractDevCmd {
 
     public RegionCmd() {
@@ -35,20 +47,36 @@ public class RegionCmd extends AbstractDevCmd {
             this.throwbackInfo(pc, "No Building At This Location.") ;
         }
         Regions region = ((AbstractCharacter)target).region;
+        output += "In Floor: " + ((AbstractCharacter) target).getInFloorID() + newline;
+        output += "In Building: " + ((AbstractCharacter) target).getInBuilding() + newline;
         if (region == null) {
-            this.throwbackInfo(pc, "No Region Found.");
-            return;
-        }
-
-        if(region != null) {
+            output += "No Region Found." + newline;
+        }else{
             output += "Player Info: " + ((AbstractCharacter) target).getName() + newline;
             output +=  "Region Building: " + building.getName() + newline;
-            output +=  "Region Height: " + region.lerpY((AbstractCharacter)target) + newline;
+            output +=  "Region Height: " + region.lerpY(((AbstractCharacter)target).loc) + newline;
             output +=  "is Stairs: " + region.isStairs() + newline;
-            output += "is Outside: " + region.isOutside();
+            output += "is Outside: " + region.isOutside() + newline;
+            output += "is Entrance: " + region.isExit() + newline;
+            for(Vector3f point : region.regionPoints)
+                output += point + newline;
+            output += "NavMesh Data" + newline;
             this.throwbackInfo(pc, output);
         }
+        //Zone zone = ZoneManager.findSmallestZone(((AbstractCharacter) target).loc);
+        //if(zone != null) {
+        //    output += "zone: " + zone.zoneName + newline;
+        //    output += "on navmesh: " + zone.navMesh.contains(((AbstractCharacter) target).loc.x,((AbstractCharacter) target).loc.z) + newline;
+        //}else {
+        //    output += "zone: null" + newline;
+        //}
 
+            output += "pointBlocked: " + NavigationManager.pointIsBlocked(((AbstractCharacter)target).loc);
+
+
+
+
+        this.throwbackInfo(pc, output);
     }
 
     @Override

src/engine/devcmd/cmds/aiInfoCmd.java

@@ -13,7 +13,9 @@ import engine.Enum;
 import engine.Enum.GameObjectType;
 import engine.devcmd.AbstractDevCmd;
 import engine.gameManager.PowersManager;
+import engine.math.Vector3fImmutable;
 import engine.mobileAI.MobAI;
+import engine.mobileAI.utilities.PathingUtilities;
 import engine.objects.AbstractGameObject;
 import engine.objects.Mob;
 import engine.objects.PlayerCharacter;
@@ -150,7 +152,10 @@ public class aiInfoCmd extends AbstractDevCmd {
             for (Integer outlawUUID : outlaws)
                 output += outlawUUID + newline;
         }
-
+        output += "Walking: " + ((Mob) target).isMoving() + newline;
+        output += "Destination: " + ((Mob) target).destination + newline;
+        output += "is Pathing: " + mob.isPathing + newline;
+        ((Mob) target).isPathing = false;
         throwbackInfo(playerCharacter, output);
     }
 

src/engine/gameManager/BuildingManager.java

@@ -18,7 +18,9 @@ import engine.job.JobContainer;
 import engine.job.JobScheduler;
 import engine.jobs.UpgradeBuildingJob;
 import engine.math.Bounds;
+import engine.math.Vector2f;
 import engine.math.Vector3fImmutable;
+import engine.mobileAI.utilities.PathingUtilities;
 import engine.net.client.ClientConnection;
 import engine.net.client.msg.ErrorPopupMsg;
 import engine.net.client.msg.ManageCityAssetsMsg;
@@ -27,6 +29,7 @@ import engine.objects.*;
 import engine.server.MBServerStatics;
 import org.pmw.tinylog.Logger;
 
+import java.awt.geom.Path2D;
 import java.time.LocalDateTime;
 import java.time.ZoneId;
 import java.util.ArrayList;
@@ -37,7 +40,7 @@ import java.util.concurrent.ThreadLocalRandom;
 public enum BuildingManager {
 
     BUILDINGMANAGER;
-
+    public static HashMap<Integer, ArrayList<ArrayList<Vector2f>>> _hull_data = new HashMap<>();
     public static HashMap<Integer, ArrayList<BuildingLocation>> _stuckLocations = new HashMap<>();
     public static HashMap<Integer, ArrayList<BuildingLocation>> _slotLocations = new HashMap<>();
 
@@ -960,4 +963,63 @@ public enum BuildingManager {
         return null;
     }
 
+    public static void bakeNavMesh(Building building) {
+        building.meshes = new ArrayList<>();
+        if (building.parentZone == null) {
+            Logger.error("Attempt to bake navmesh with no parent: " + building.getObjectUUID());
+            return;
+        }
+
+        // Build up navmesh by stencil of the
+        // convex hull meshes that comprise the prop.
+
+        ArrayList<ArrayList<Vector2f>> convexHullList;
+        convexHullList = _hull_data.get(building.meshUUID);
+
+        if (convexHullList == null) {
+            Logger.error("Attempt to bake navmesh with no meshes: " + building.getObjectUUID());
+            return;
+        }
+
+        for (ArrayList<Vector2f> meshEntry : convexHullList) {
+            Path2D.Float meshBound = new Path2D.Float();
+            Vector3fImmutable offsetVect = new Vector3fImmutable(meshEntry.get(0).x + building.loc.x, building.loc.y,meshEntry.get(0).y + building.loc.z);
+            Vector3fImmutable rotatedStart =  Vector3fImmutable.rotateAroundPoint(building.loc,offsetVect,building.getRot().y);
+            meshBound.moveTo(rotatedStart.x,rotatedStart.z);
+            for (Vector2f vect : meshEntry) {
+                if(meshEntry.indexOf(vect) == 0){
+                    continue;
+                }
+                Vector3fImmutable pos = new Vector3fImmutable(vect.x + building.loc.x, building.loc.y,vect.y + building.loc.z);
+                Vector3fImmutable rotatedPos =  Vector3fImmutable.rotateAroundPoint(building.loc,pos,building.getRot().getRotation());
+                meshBound.lineTo(rotatedPos.x,rotatedPos.z);
+            }
+            meshBound.lineTo(rotatedStart.x,rotatedStart.z);
+            meshBound.closePath();
+            building.meshes.add(meshBound);
+            building.parentZone.navObstacles.add(meshBound);
+        }
+        //add navNodes to parent zone list
+        float X = building.getBounds().getHalfExtents().x;
+        float Y = building.getBounds().getHalfExtents().y;
+        ArrayList<Vector2f> cornersAndFaces = new ArrayList<>();
+        cornersAndFaces.add(new Vector2f(building.loc.x - X,building.loc.z - Y));
+        cornersAndFaces.add(new Vector2f(building.loc.x + X,building.loc.z + Y));
+        cornersAndFaces.add(new Vector2f(building.loc.x + X,building.loc.z - Y));
+        cornersAndFaces.add(new Vector2f(building.loc.x - X,building.loc.z + Y));
+        cornersAndFaces.add(new Vector2f(building.loc.x - X,building.loc.z));
+        cornersAndFaces.add(new Vector2f(building.loc.x + X,building.loc.z));
+        cornersAndFaces.add(new Vector2f(building.loc.x,building.loc.z - Y));
+        cornersAndFaces.add(new Vector2f(building.loc.x,building.loc.z + Y));
+        for(Vector2f point : cornersAndFaces){
+            if(!NavigationManager.pointIsBlocked(new Vector3fImmutable(point.x,building.loc.y,point.y))){
+                building.parentZone.navNodes.add(new PathingUtilities.Node(point,null,building));
+            }
+        }
+
+        //add region centers to the zones navNodes list
+        for(Regions region : building.getBounds().getRegions()){
+            building.parentZone.navNodes.add(new PathingUtilities.Node(new Vector2f(region.center.x,region.center.z),region,building));
+        }
+    }
 }

src/engine/gameManager/NavigationManager.java

@@ -0,0 +1,137 @@
+package engine.gameManager;
+
+import engine.Enum;
+import engine.math.Vector3fImmutable;
+import engine.mobileAI.MobAI;
+import engine.net.client.msg.MoveToPointMsg;
+import engine.objects.*;
+import java.awt.geom.Path2D;
+import java.util.ArrayList;
+
+
+public class NavigationManager {
+
+    private static final int cellGap = 1;
+    private static final int stepHeight = 2;
+
+    public static void pathfind(AbstractCharacter character, Vector3fImmutable goal) {
+        if(!pathBlocked(character.loc,goal)){
+            character.destination = goal;
+            MobAI.directMove((Mob)character,character.combatTarget != null);
+        }
+        try {
+            ArrayList<Vector3fImmutable> path = getPath(character.loc, goal);//getOptimizedPath(getPath(character.loc, goal), getPath(goal, character.loc));
+            if (path.isEmpty()) {
+                MobAI.directMove((Mob)character,character.combatTarget != null);
+                return; //no points to walk to
+            }
+            //character.navPath = path;
+
+        } catch (Exception e) {
+            //something failed
+        }
+    }
+
+    public static ArrayList<Vector3fImmutable> getOptimizedPath(ArrayList<Vector3fImmutable> startToGoal, ArrayList<Vector3fImmutable> goalToStart) {
+        ArrayList<Vector3fImmutable> optimalPath = new ArrayList<>();
+        optimalPath.add(startToGoal.get(0));
+        for (Vector3fImmutable point : startToGoal) {
+            if (goalToStart.contains(point) && !optimalPath.contains(point)) {
+                optimalPath.add(point);
+            }
+
+        }
+
+        //optimize the path to its smallest possible amount of points
+
+
+        return optimalPath;
+    }
+
+    private static ArrayList<Vector3fImmutable> getPath(Vector3fImmutable start, Vector3fImmutable goal) {
+        ArrayList<Vector3fImmutable> path = new ArrayList<>();
+        path.add(start);
+        Vector3fImmutable current = start;
+        boolean obstructed = false;
+        int count = 0;
+        while (current.distanceSquared(goal) > 9 && count < 250) {
+            //gather the 8 cells around the player
+            ArrayList<Vector3fImmutable> surroundingCells = new ArrayList<>();
+            surroundingCells.add(current.add(new Vector3fImmutable(cellGap, 0, 0)));
+            surroundingCells.add(current.add(new Vector3fImmutable(cellGap, 0, cellGap)));
+            surroundingCells.add(current.add(new Vector3fImmutable(0, 0, cellGap)));
+            surroundingCells.add(current.add(new Vector3fImmutable(-cellGap, 0, 0)));
+            surroundingCells.add(current.add(new Vector3fImmutable(-cellGap, 0, -cellGap)));
+            surroundingCells.add(current.add(new Vector3fImmutable(0, 0, -cellGap)));
+            surroundingCells.add(current.add(new Vector3fImmutable(-cellGap, 0, cellGap)));
+            surroundingCells.add(current.add(new Vector3fImmutable(cellGap, 0, -cellGap)));
+            Vector3fImmutable cheapest = new Vector3fImmutable(Vector3fImmutable.ZERO);
+            for (Vector3fImmutable point : surroundingCells) {
+                count++;
+
+                if (path.contains(point))
+                    continue;
+
+                if (pointIsBlocked(point)) {
+                    obstructed = true;
+                    continue;
+                }
+
+                if (getCost(cheapest, current, goal) > getCost(point, current, goal))
+                    cheapest = point;
+            }
+            current = cheapest;
+            path.add(cheapest);
+        }
+        if (obstructed) {
+            return path;
+        } else {
+            ArrayList<Vector3fImmutable> goalPath = new ArrayList<>();
+            goalPath.add(start);
+            goalPath.add(start.moveTowards(goal,32));
+            goalPath.add(goal);
+            return goalPath; //if the path isn't obstructed we can walk directly from start to the goal
+
+        }
+    }
+
+    public static float getCost(Vector3fImmutable point, Vector3fImmutable start, Vector3fImmutable goal) {
+        float gCost = start.distanceSquared(point);
+        float hCost = goal.distanceSquared(point);
+        return gCost + hCost;
+    }
+
+    public static boolean pathBlocked(Vector3fImmutable start, Vector3fImmutable end){
+        Zone zone = ZoneManager.findSmallestZone(start);
+        if(zone != null) {
+            for (Path2D.Float obstacle : zone.navObstacles)
+                if(obstacle.intersects(start.x,start.z,end.x,end.z))
+                    return true;
+        }
+        return false;
+    }
+    public static boolean pointIsBlocked(Vector3fImmutable point) {
+    Zone zone = ZoneManager.findSmallestZone(point);
+    if(zone != null){
+        for(Path2D.Float obstacle : zone.navObstacles)
+            if (obstacle.contains(point.x,point.z)) {
+                return true;
+            }
+    }
+
+        //Building building = BuildingManager.getBuildingAtLocation(point);
+        //if(building != null) {
+            //for (Path2D.Float mesh : building.meshes) {
+                //if (mesh.contains(point.x,point.z)) {
+                    //return true;
+                //}
+            //}
+            //for (Regions region : building.getBounds().getRegions()) {
+                //if (region.isPointInPolygon(point))
+                    //if (Math.abs(region.lerpY(point) - point.y) > stepHeight) // get the height distance between current height and target location height
+                        //return true;
+            //}
+        //}
+        return false;
+    }
+}

src/engine/gameManager/ZoneManager.java

@@ -14,6 +14,7 @@ import engine.math.Bounds;
 import engine.math.Vector2f;
 import engine.math.Vector3f;
 import engine.math.Vector3fImmutable;
+import engine.mobileAI.utilities.PathingUtilities;
 import engine.objects.Building;
 import engine.objects.City;
 import engine.objects.Zone;
@@ -21,6 +22,7 @@ import engine.objects.ZoneTemplate;
 import engine.server.MBServerStatics;
 import org.pmw.tinylog.Logger;
 
+import java.awt.geom.Path2D;
 import java.time.Instant;
 import java.time.LocalDateTime;
 import java.time.ZoneId;

src/engine/math/Bounds.java

@@ -41,7 +41,7 @@ public class Bounds {
     private boolean flipExtents;
 
     private ArrayList<Regions> regions = new ArrayList<>();
-    private ArrayList<Colliders> colliders = new ArrayList<>();
+    public ArrayList<Colliders> colliders = new ArrayList<>();
 
     // Default constructor
 

src/engine/mobileAI/MobAI.java

@@ -11,14 +11,16 @@ package engine.mobileAI;
 import engine.Enum;
 import engine.Enum.DispatchChannel;
 import engine.InterestManagement.WorldGrid;
+import engine.exception.MsgSendException;
 import engine.gameManager.*;
 import engine.math.Vector3f;
 import engine.math.Vector3fImmutable;
 import engine.mobileAI.Threads.MobAIThread;
 import engine.mobileAI.Threads.Respawner;
 import engine.mobileAI.utilities.CombatUtilities;
-import engine.mobileAI.utilities.MovementUtilities;
+import engine.mobileAI.utilities.PathingUtilities;
 import engine.net.DispatchMessage;
+import engine.net.client.msg.MoveToPointMsg;
 import engine.net.client.msg.PerformActionMsg;
 import engine.net.client.msg.PowerProjectileMsg;
 import engine.objects.*;
@@ -103,7 +105,7 @@ public class MobAI {
             if (mob.behaviourType.callsForHelp)
                 MobCallForHelp(mob);
 
-            if (!MovementUtilities.inRangeDropAggro(mob, target)) {
+            if (inRangeDropAggro(mob, target)) {
                 mob.setCombatTarget(null);
                 return;
             }
@@ -303,7 +305,7 @@ public class MobAI {
                         MobAI.Patrol(minion);
                 }
 
-            MovementUtilities.aiMove(mob, mob.destination, true);
+            aiMove(mob, true,0);
 
         } catch (Exception e) {
             Logger.info(mob.getObjectUUID() + " " + mob.getName() + " Failed At: AttackTarget" + " " + e.getMessage());
@@ -584,7 +586,7 @@ public class MobAI {
             if(mob.isPlayerGuard() || mob.isSiege()) {
                 bypassLoadedPlayerCheck = true;
                 if(mob.combatTarget != null && mob.combatTarget.getObjectType().equals(Enum.GameObjectType.PlayerCharacter))
-                    if(mob.combatTarget.loc.distanceSquared(mob.loc) > 10000)
+                    if(mob.combatTarget.loc.distanceSquared(mob.loc) > 62500)
                         mob.setCombatTarget(null);
             }
 
@@ -689,7 +691,7 @@ public class MobAI {
                 if (aiAgent.enemy.size() > 0 && aiAgent.enemy.contains(loadedPlayer.getRace().getRaceType().getMonsterType()) == false)
                     continue;
 
-                if (MovementUtilities.inRangeToAggro(aiAgent, loadedPlayer)) {
+                if (inRangeToAggro(aiAgent, loadedPlayer)) {
                     aiAgent.setCombatTarget(loadedPlayer);
                     return;
                 }
@@ -722,7 +724,7 @@ public class MobAI {
 
         try {
 
-            if (!MovementUtilities.canMove(mob))
+            if (!canMove(mob))
                 return;
 
             mob.updateLocation();
@@ -749,7 +751,7 @@ public class MobAI {
                             return;
 
                         mob.destination = mob.guardCaptain.getLoc();
-                        MovementUtilities.moveToLocation(mob, mob.destination, 5, false);
+                        aiMove(mob,  false,5);
                     } else
                         chaseTarget(mob);
                     break;
@@ -834,7 +836,7 @@ public class MobAI {
             if (mob.getCombatTarget() == null)
                 return;
 
-            if (mob.getCombatTarget().getObjectType().equals(Enum.GameObjectType.PlayerCharacter) && MovementUtilities.inRangeDropAggro(mob, (PlayerCharacter) mob.getCombatTarget()) == false &&
+            if (mob.getCombatTarget().getObjectType().equals(Enum.GameObjectType.PlayerCharacter) && inRangeDropAggro(mob, (PlayerCharacter) mob.getCombatTarget()) &&
                     mob.agentType.equals(Enum.AIAgentType.PET) == false) {
 
                 mob.setCombatTarget(null);
@@ -877,7 +879,7 @@ public class MobAI {
                         }
                     }
                 }
-            } else if (MovementUtilities.inRangeOfBindLocation(mob) == false) {
+            } else if (inRangeOfBindLocation(mob) == false) {
 
                 PowersBase recall = PowersManager.getPowerByToken(-1994153779);
                 PowersManager.useMobPower(mob, mob, recall, 40);
@@ -905,7 +907,7 @@ public class MobAI {
             if (CombatUtilities.inRange2D(mob, mob.getCombatTarget(), mob.getRange()) == false) {
                 if (mob.getRange() > 15) {
                     mob.destination = mob.getCombatTarget().getLoc();
-                    MovementUtilities.moveToLocation(mob, mob.destination, 0, false);
+                    aiMove(mob, false,0);
                 } else {
 
                     //check if building
@@ -913,12 +915,12 @@ public class MobAI {
                     switch (mob.getCombatTarget().getObjectType()) {
                         case PlayerCharacter:
                         case Mob:
-                            mob.destination = MovementUtilities.GetDestinationToCharacter(mob, (AbstractCharacter) mob.getCombatTarget());
-                            MovementUtilities.moveToLocation(mob, mob.destination, mob.getRange() + 1, false);
+                            mob.destination = mob.combatTarget.loc;//GetDestinationToCharacter(mob, (AbstractCharacter) mob.getCombatTarget());
+                            aiMove(mob, false,mob.getRange() + 1);
                             break;
                         case Building:
                             mob.destination = mob.getCombatTarget().getLoc();
-                            MovementUtilities.moveToLocation(mob, mob.getCombatTarget().getLoc(), 0, false);
+                            aiMove(mob, false,0);
                             break;
                     }
                 }
@@ -1000,7 +1002,7 @@ public class MobAI {
                 if (ZoneManager.seaFloor.zoneMobSet.contains(mob))
                     mob.killCharacter("no owner");
 
-            if (MovementUtilities.canMove(mob) && mob.behaviourType.canRoam)
+            if (canMove(mob) && mob.behaviourType.canRoam)
                 CheckMobMovement(mob);
 
             CheckForAttack(mob);
@@ -1130,7 +1132,7 @@ public class MobAI {
                 if (GuardCanAggro(mob, loadedPlayer) == false)
                     continue;
 
-                if (MovementUtilities.inRangeToAggro(mob, loadedPlayer) && mob.getCombatTarget() == null) {
+                if (inRangeToAggro(mob, loadedPlayer) && mob.getCombatTarget() == null) {
                     mob.setCombatTarget(loadedPlayer);
                     return;
                 }
@@ -1240,9 +1242,9 @@ public class MobAI {
             float xPoint = ThreadLocalRandom.current().nextInt(400) - 200;
             float zPoint = ThreadLocalRandom.current().nextInt(400) - 200;
             Vector3fImmutable TreePos = mob.getGuild().getOwnedCity().getLoc();
-            mob.destination = new Vector3fImmutable(TreePos.x + xPoint, TreePos.y, TreePos.z + zPoint);
+            mob.destination =  new Vector3fImmutable(TreePos.x + xPoint, TreePos.y, TreePos.z + zPoint);
 
-            MovementUtilities.aiMove(mob, mob.destination, true);
+            aiMove(mob, true,0);
 
             if (mob.agentType.equals(Enum.AIAgentType.GUARDCAPTAIN)) {
                 for (Integer minionUUID : mob.minions) {
@@ -1252,11 +1254,12 @@ public class MobAI {
                     //make sure mob is out of combat stance
 
                     if (minion.despawned == false) {
-                        if (MovementUtilities.canMove(minion)) {
+                        if (canMove(minion)) {
                             Vector3f minionOffset = Formation.getOffset(2, mob.minions.indexOf(minionUUID) + 3);
                             minion.updateLocation();
                             Vector3fImmutable formationPatrolPoint = new Vector3fImmutable(mob.destination.x + minionOffset.x, mob.destination.y, mob.destination.z + minionOffset.z);
-                            MovementUtilities.aiMove(minion, formationPatrolPoint, true);
+                            minion.destination = formationPatrolPoint;
+                            aiMove(minion, true,0);
                         }
                     }
                 }
@@ -1284,7 +1287,7 @@ public class MobAI {
                 if (potentialTarget.equals(mob.getCombatTarget()))
                     continue;
 
-                if (potentialTarget != null && mob.playerAgroMap.get(potentialTarget.getObjectUUID()).floatValue() > CurrentHateValue && MovementUtilities.inRangeToAggro(mob, potentialTarget)) {
+                if (potentialTarget != null && mob.playerAgroMap.get(potentialTarget.getObjectUUID()).floatValue() > CurrentHateValue && inRangeToAggro(mob, potentialTarget)) {
                     CurrentHateValue = mob.playerAgroMap.get(potentialTarget.getObjectUUID()).floatValue();
                     mostHatedTarget = potentialTarget;
                 }
@@ -1296,4 +1299,100 @@ public class MobAI {
         }
         return null;
     }
+
+    public static boolean inRangeDropAggro(Mob agent, AbstractCharacter target) {
+        float rangeSquared = MobAIThread.AI_BASE_AGGRO_RANGE * MobAIThread.AI_BASE_AGGRO_RANGE;
+        if(agent.isPlayerGuard())
+            rangeSquared = 62500;
+        if(agent.isSiege())
+            rangeSquared = agent.getRange() * agent.getRange();
+
+        return agent.loc.distanceSquared(target.loc) > rangeSquared;
+
+    }
+
+    public static boolean inRangeToAggro(Mob agent, PlayerCharacter target) {
+        return MobAIThread.AI_BASE_AGGRO_RANGE * MobAIThread.AI_BASE_AGGRO_RANGE > agent.loc.distanceSquared(target.loc) ;
+    }
+
+    public static boolean inRangeOfBindLocation(Mob agent) {
+        if(agent.isPlayerGuard() && agent.guardedCity.isLocationOnCityZone(agent.loc))
+            return true;
+         float bindRangeSquared = (agent.parentZone.getBounds().getHalfExtents().x * agent.parentZone.getBounds().getHalfExtents().x) + (MobAIThread.AI_DROP_AGGRO_RANGE * MobAIThread.AI_DROP_AGGRO_RANGE);
+        return agent.loc.distanceSquared(agent.bindLoc) < bindRangeSquared;
+    }
+
+    public static boolean canMove(Mob agent) {
+        if (!agent.behaviourType.canRoam)
+            return false;
+
+        return (agent.isAlive() && !agent.getBonuses().getBool(Enum.ModType.Stunned, Enum.SourceType.None) && !agent.getBonuses().getBool(Enum.ModType.CannotMove, Enum.SourceType.None));
+    }
+
+    public static Vector3fImmutable GetDestinationToCharacter(Mob aiAgent, AbstractCharacter character) {
+
+        if (!character.isMoving())
+            return character.getLoc();
+
+
+        float agentDistanceEndLoc = aiAgent.getLoc().distanceSquared2D(character.getEndLoc());
+        float characterDistanceEndLoc = character.getLoc().distanceSquared2D(character.getEndLoc());
+
+        if (agentDistanceEndLoc > characterDistanceEndLoc)
+            return character.getEndLoc();
+
+        return character.getLoc();
+    }
+
+    public static void aiMove(Mob mob, boolean isWalking, float offset) {
+
+        if(mob.isMoving()) {
+            return;
+        }
+        if(!mob.isPathing){
+            ArrayList<PathingUtilities.Node> path = PathingUtilities.getPath(mob, mob.destination);
+            if(path != null && path.size() > 0)
+                PathingUtilities.followPath(mob,path);
+        }
+    }
+
+    public static void directMove(AbstractCharacter mob,boolean isWalking){
+        //update our walk/run state.
+        if (isWalking && !mob.isWalk()) {
+            mob.setWalkMode(true);
+            MovementManager.sendRWSSMsg(mob);
+        } else if (!isWalking && mob.isWalk()) {
+            mob.setWalkMode(false);
+            MovementManager.sendRWSSMsg(mob);
+        }
+        mob.endLoc = mob.destination;
+
+        MoveToPointMsg msg = new MoveToPointMsg();
+
+        msg.setSourceType(Enum.GameObjectType.Mob.ordinal());
+        msg.setSourceID(mob.getObjectUUID());
+        msg.setStartCoord(mob.loc);
+        msg.setEndCoord(mob.destination);
+        Regions region = Regions.getRegionAtLocation(mob.destination);
+        if(region != null){
+            msg.setInBuildingFloor(region.room);
+            msg.setInBuilding(region.level);
+            msg.setStartLocType(0);
+            msg.setInBuildingUUID(region.parentBuildingID);
+        } else{
+            msg.setInBuildingFloor(-1);
+            msg.setInBuilding(-1);
+            msg.setStartLocType(0);
+            msg.setInBuildingUUID(0);
+        }
+
+
+
+        try {
+            MovementManager.movement(msg, mob);
+        } catch (MsgSendException e) {
+            // TODO Figure out how we want to handle the msg send exception
+            e.printStackTrace();
+        }
+    }
 }

src/engine/mobileAI/utilities/MovementUtilities.java

@@ -1,295 +0,0 @@
-// • ▌ ▄ ·.  ▄▄▄·  ▄▄ • ▪   ▄▄· ▄▄▄▄·  ▄▄▄·  ▐▄▄▄  ▄▄▄ .
-// ·██ ▐███▪▐█ ▀█ ▐█ ▀ ▪██ ▐█ ▌▪▐█ ▀█▪▐█ ▀█ •█▌ ▐█▐▌·
-// ▐█ ▌▐▌▐█·▄█▀▀█ ▄█ ▀█▄▐█·██ ▄▄▐█▀▀█▄▄█▀▀█ ▐█▐ ▐▌▐▀▀▀
-// ██ ██▌▐█▌▐█ ▪▐▌▐█▄▪▐█▐█▌▐███▌██▄▪▐█▐█ ▪▐▌██▐ █▌▐█▄▄▌
-// ▀▀  █▪▀▀▀ ▀  ▀ ·▀▀▀▀ ▀▀▀·▀▀▀ ·▀▀▀▀  ▀  ▀ ▀▀  █▪ ▀▀▀
-//      Magicbane Emulator Project © 2013 - 2022
-//                www.magicbane.com
-
-
-package engine.mobileAI.utilities;
-
-import engine.Enum;
-import engine.Enum.GameObjectType;
-import engine.Enum.ModType;
-import engine.Enum.SourceType;
-import engine.exception.MsgSendException;
-import engine.gameManager.MovementManager;
-import engine.math.Vector3fImmutable;
-import engine.mobileAI.Threads.MobAIThread;
-import engine.net.client.msg.MoveToPointMsg;
-import engine.objects.*;
-import org.pmw.tinylog.Logger;
-
-import java.util.concurrent.ThreadLocalRandom;
-
-import static engine.math.FastMath.sqr;
-import static engine.math.FastMath.sqrt;
-
-public class MovementUtilities {
-
-
-    public static boolean inRangeOfBindLocation(Mob agent) {
-
-
-        if (agent.isPlayerGuard()) {
-
-            Mob guardCaptain = null;
-            if (agent.getContract() != null)
-                guardCaptain = agent;
-            else
-                guardCaptain = (Mob) agent.guardCaptain;
-
-            if (guardCaptain != null) {
-                Building barracks = guardCaptain.building;
-
-                if (barracks != null) {
-                    City city = barracks.getCity();
-
-                    if (city != null) {
-                        Building tol = city.getTOL();
-
-                        //Guards recall distance = 814.
-                        if (tol != null) {
-                            return !(agent.getLoc().distanceSquared2D(tol.getLoc()) > sqr(Enum.CityBoundsType.ZONE.halfExtents));
-                        }
-
-                    }
-                }
-            }
-
-            return true;
-        }
-
-        Vector3fImmutable sl = new Vector3fImmutable(agent.getLoc().getX(), 0, agent.getLoc().getZ());
-        Vector3fImmutable tl = new Vector3fImmutable(agent.getTrueBindLoc().x, 0, agent.getTrueBindLoc().z);
-
-        float distanceSquaredToTarget = sl.distanceSquared2D(tl); //distance to center of target
-        float zoneRange = 250;
-
-        if (agent.getParentZone() != null) {
-            if (agent.getParentZone().bounds != null)
-                zoneRange = agent.getParentZone().bounds.getHalfExtents().x * 2;
-        }
-
-        if (zoneRange > 300)
-            zoneRange = 300;
-
-        if (agent.getSpawnRadius() > zoneRange)
-            zoneRange = agent.getSpawnRadius();
-
-
-        return distanceSquaredToTarget < sqr(MobAIThread.AI_DROP_AGGRO_RANGE + zoneRange);
-
-    }
-
-    public static boolean inRangeToAggro(Mob agent, PlayerCharacter target) {
-
-        Vector3fImmutable sl = agent.getLoc();
-        Vector3fImmutable tl = target.getLoc();
-
-        float distanceSquaredToTarget = sl.distanceSquared2D(tl) - sqr(agent.calcHitBox() + target.calcHitBox()); //distance to center of target
-        float range = MobAIThread.AI_BASE_AGGRO_RANGE;
-
-        if (agent.isPlayerGuard())
-            range = 150;
-
-        return distanceSquaredToTarget < sqr(range);
-
-    }
-
-    public static boolean inRangeDropAggro(Mob agent, AbstractCharacter target) {
-
-        Vector3fImmutable sl = agent.getLoc();
-        Vector3fImmutable tl = target.getLoc();
-
-        float distanceSquaredToTarget = sl.distanceSquared2D(tl) - sqr(agent.calcHitBox() + target.calcHitBox()); //distance to center of target
-
-        float range = agent.getRange() + 150;
-
-        if (range > 200)
-            range = 200;
-
-
-        return distanceSquaredToTarget < sqr(range);
-
-    }
-
-    public static Vector3fImmutable GetMoveLocation(Mob aiAgent, AbstractCharacter aggroTarget) {
-
-        // Player isnt moving and neither is mob.  Just return
-        // the mobile's current location.  Ain't goin nowhere!
-        // *** Refactor: Check to ensure methods calling us
-        // all don't sent move messages when not moving.
-
-        if ((aggroTarget.isMoving() == false))
-            return aggroTarget.getLoc();
-
-        if (aggroTarget.getEndLoc().x != 0) {
-
-            float aggroTargetDistanceSquared = aggroTarget.getLoc().distanceSquared2D(aggroTarget.getEndLoc());
-            float aiAgentDistanceSquared = aiAgent.getLoc().distanceSquared2D(aggroTarget.getEndLoc());
-
-            if (aiAgentDistanceSquared >= aggroTargetDistanceSquared)
-                return aggroTarget.getEndLoc();
-            else {
-                float distanceToMove = sqrt(aggroTargetDistanceSquared + aiAgentDistanceSquared) * .5f;
-
-                return aggroTarget.getFaceDir().scaleAdd(distanceToMove, aggroTarget.getLoc());
-
-            }
-        }
-
-        // One of us is moving so let's calculate our destination loc for this
-        // simulation frame.  We will simply project our position onto the
-        // character's movement vector and return the closest point.
-
-        return aiAgent.getLoc().ClosestPointOnLine(aggroTarget.getLoc(), aggroTarget.getEndLoc());
-    }
-
-    public static void moveToLocation(Mob agent, Vector3fImmutable newLocation, float offset, boolean isWalking) {
-        try {
-
-            //don't move farther than 30 units from player.
-            if (offset > 30)
-                offset = 30;
-            Vector3fImmutable newLoc = Vector3fImmutable.getRandomPointInCircle(newLocation, offset);
-
-
-            agent.setFaceDir(newLoc.subtract2D(agent.getLoc()).normalize());
-
-            aiMove(agent, newLoc, isWalking);
-        } catch (Exception e) {
-            Logger.error(e.toString());
-        }
-    }
-
-
-    public static boolean canMove(Mob agent) {
-        if (agent.getMobBase() != null && Enum.MobFlagType.SENTINEL.elementOf(agent.getMobBase().getFlags()))
-            return false;
-
-        return (agent.isAlive() && !agent.getBonuses().getBool(ModType.Stunned, SourceType.None) && !agent.getBonuses().getBool(ModType.CannotMove, SourceType.None));
-    }
-
-    public static Vector3fImmutable randomPatrolLocation(Mob agent, Vector3fImmutable center, float radius) {
-
-        //Determing where I want to move.
-        return new Vector3fImmutable((center.x - radius) + ((ThreadLocalRandom.current().nextFloat() + .1f * 2) * radius),
-                center.y,
-                (center.z - radius) + ((ThreadLocalRandom.current().nextFloat() + .1f * 2) * radius));
-    }
-
-    public static Long estimateMovementTime(Mob agent) {
-        if (agent.getEndLoc().x == 0 && agent.getEndLoc().y == 0)
-            return 0L;
-
-        return (long) ((agent.getLoc().distance2D(agent.getEndLoc()) * 1000) / agent.getSpeed());
-    }
-
-    public static void aiMove(Mob agent, Vector3fImmutable vect, boolean isWalking) {
-
-        //update our walk/run state.
-        if (isWalking && !agent.isWalk()) {
-            agent.setWalkMode(true);
-            MovementManager.sendRWSSMsg(agent);
-        } else if (!isWalking && agent.isWalk()) {
-            agent.setWalkMode(false);
-            MovementManager.sendRWSSMsg(agent);
-        }
-
-        MoveToPointMsg msg = new MoveToPointMsg();
-
-
-//		Regions currentRegion = Mob.InsideBuildingRegion(agent);
-//
-//		if (currentRegion != null){
-//
-//
-//			if (currentRegion.isGroundLevel()){
-//				agent.setInBuilding(0);
-//				agent.setInFloorID(-1);
-//			}else{
-//				agent.setInBuilding(currentRegion.getLevel());
-//				agent.setInFloorID(currentRegion.getRoom());
-//			}
-//		}else{
-//			agent.setInBuilding(-1);
-//			agent.setInFloorID(-1);
-//			agent.setInBuildingID(0);
-//		}
-//		agent.setLastRegion(currentRegion);
-
-
-        Vector3fImmutable startLoc = null;
-        Vector3fImmutable endLoc = null;
-
-//		if (agent.getLastRegion() != null){
-//			Building inBuilding = Building.getBuildingFromCache(agent.getInBuildingID());
-//			if (inBuilding != null){
-//				startLoc = ZoneManager.convertWorldToLocal(inBuilding, agent.getLoc());
-//				endLoc = ZoneManager.convertWorldToLocal(inBuilding, vect);
-//			}
-//		}else{
-//			agent.setBuildingID(0);
-//			agent.setInBuildingID(0);
-//			startLoc = agent.getLoc();
-//			endLoc = vect;
-//		}
-
-        startLoc = agent.getLoc();
-        endLoc = vect;
-
-        msg.setSourceType(GameObjectType.Mob.ordinal());
-        msg.setSourceID(agent.getObjectUUID());
-        msg.setStartCoord(startLoc);
-        msg.setEndCoord(endLoc);
-        msg.setInBuildingFloor(-1);
-        msg.setInBuilding(-1);
-        msg.setStartLocType(0);
-        msg.setInBuildingUUID(0);
-
-
-        try {
-            MovementManager.movement(msg, agent);
-        } catch (MsgSendException e) {
-            // TODO Figure out how we want to handle the msg send exception
-            e.printStackTrace();
-        }
-    }
-
-    public static Vector3fImmutable GetDestinationToCharacter(Mob aiAgent, AbstractCharacter character) {
-
-        if (!character.isMoving())
-            return character.getLoc();
-
-
-        float agentDistanceEndLoc = aiAgent.getLoc().distanceSquared2D(character.getEndLoc());
-        float characterDistanceEndLoc = character.getLoc().distanceSquared2D(character.getEndLoc());
-
-        if (agentDistanceEndLoc > characterDistanceEndLoc)
-            return character.getEndLoc();
-
-        return character.getLoc();
-    }
-
-    public static boolean updateMovementToCharacter(Mob aiAgent, AbstractCharacter aggroTarget) {
-
-        if (aiAgent.destination.equals(Vector3fImmutable.ZERO))
-            return true;
-
-        if (!aiAgent.isMoving())
-            return true;
-
-
-        if (aggroTarget.isMoving()) {
-            return !aiAgent.destination.equals(aggroTarget.getEndLoc()) && !aiAgent.destination.equals(aggroTarget.getLoc());
-        } else {
-            if (aiAgent.destination.equals(aggroTarget.getLoc()))
-                return false;
-        }
-
-        return false;
-    }
-
-}

src/engine/mobileAI/utilities/PathingUtilities.java

@@ -0,0 +1,160 @@
+package engine.mobileAI.utilities;
+import engine.Enum;
+import engine.exception.MsgSendException;
+import engine.gameManager.BuildingManager;
+import engine.gameManager.MovementManager;
+import engine.gameManager.ZoneManager;
+import engine.math.Vector2f;
+import engine.math.Vector3fImmutable;
+import engine.mobileAI.MobAI;
+import engine.net.client.msg.MoveToPointMsg;
+import engine.objects.AbstractCharacter;
+import engine.objects.Building;
+import engine.objects.Regions;
+import engine.objects.Zone;
+import org.pmw.tinylog.Logger;
+
+import java.awt.geom.Path2D;
+import java.util.ArrayList;
+
+public class PathingUtilities {
+    public static class Node {
+        public Vector2f location;
+        public ArrayList<Node> neighbors;
+        public Regions region;
+        public Building parentBuilding;
+
+        public Node(Vector2f loc, Regions reg, Building parent){
+            this.location = loc;
+            this.region = reg;
+            this.parentBuilding = parent;
+            this.neighbors = new ArrayList<>();
+        }
+        public Node(Node clone){
+            this.location = clone.location;
+            this.region = clone.region;
+            this.parentBuilding = clone.parentBuilding;
+            this.neighbors = (ArrayList<Node>) clone.neighbors.clone();
+        }
+        public ArrayList<Node> getNeighbors(){
+            if(neighbors.size() == 0){
+                Zone zone = ZoneManager.findSmallestZone(new Vector3fImmutable(this.location.x,0,this.location.y));
+                if(zone == null)
+                    return null;
+
+                for(Node potentialNeighbor : zone.navNodes){
+                        for (Path2D.Float obstacle : zone.navObstacles) {
+                            if (!this.equals(potentialNeighbor) && !obstacle.intersects(this.location.x, this.location.y, potentialNeighbor.location.x, potentialNeighbor.location.y) && this.location.distance(potentialNeighbor.location) < 65) {
+                                this.neighbors.add(potentialNeighbor);
+                            }
+                        }
+                }
+            }
+            return this.neighbors;
+        }
+    }
+
+    public static Node getClosestNode(Vector3fImmutable loc){
+        Zone zone = ZoneManager.findSmallestZone(loc);
+        Regions region = Regions.getRegionAtLocation(loc);
+        if(region != null){
+            for(Node node : zone.navNodes){
+                if(node.location == new Vector2f(region.center.x,region.center.z))
+                    return node;
+            }
+        }
+        float disSq = 10000000;
+        Node closest = null;
+        for(Node node : zone.navNodes){
+            if(node.location.distanceSquared(new Vector2f(loc.x,loc.z)) < disSq)
+                closest = node;
+        }
+        return closest;
+    }
+    public static ArrayList<Node> getPath(AbstractCharacter mover, Vector3fImmutable goal){
+        ArrayList<Node> path = new ArrayList<>();
+        Node startNode = getClosestNode(mover.loc);
+        Node goalNode = getClosestNode(goal);
+        if(goalNode == null)
+            move(mover,goal);
+        Node currentNode = new Node(startNode);
+        path.add(startNode);
+        int attempts = 0;
+        while(!currentNode.equals(goalNode) && attempts < 250){
+            attempts ++;
+            currentNode = getCheapestNeighbor(currentNode,new Vector2f(goal.x,goal.z));
+            path.add(currentNode);
+        }
+        return path;
+    }
+    public static Node getCheapestNeighbor(Node node, Vector2f goal){
+        Node cheapest = null;
+        for(Node neighbor : node.getNeighbors()){
+            if(cheapest == null) {
+                cheapest = neighbor;
+                continue;
+            }
+            if(getCost(cheapest.location,node.location,goal) > getCost(neighbor.location,node.location,goal)){
+                cheapest = neighbor;
+            }
+        }
+        return cheapest;
+    }
+    public static float getCost(Vector2f point, Vector2f start, Vector2f goal) {
+        float gCost = start.distanceSquared(point);
+        float hCost = goal.distanceSquared(point);
+        return gCost + hCost;
+    }
+    public static void followPath(AbstractCharacter character, ArrayList<Node> path){
+        character.isPathing = true;
+        while(new Vector2f(character.loc.x,character.loc.z).distanceSquared(path.get(path.size() - 1).location) > 9 && path.size() > 0){
+            if( character.isMoving())
+                continue;
+            if(character.combatTarget != null){
+                //need ot adjust new path for a moving combat target
+                if(character.combatTarget.loc.distanceSquared2D(new Vector3fImmutable(path.get(path.size() - 1).location.x,0,path.get(path.size() - 1).location.y)) > 625){
+                    character.isPathing = false;
+                    break;
+                }
+            }
+            move(character,new Vector3fImmutable(path.get(0).location.x,0,path.get(0).location.y));
+            path.remove(0);
+        }
+        character.isPathing = false;
+    }
+    public static void move(AbstractCharacter mob, Vector3fImmutable goal) {
+        mob.destination = goal;
+        if (mob.getObjectType().equals(Enum.GameObjectType.Mob)) {
+            mob.setWalkMode(!(mob.combatTarget == null));
+            MovementManager.sendRWSSMsg(mob);
+        }
+        mob.endLoc = goal;
+
+        MoveToPointMsg msg = new MoveToPointMsg();
+
+        msg.setSourceType(Enum.GameObjectType.Mob.ordinal());
+        msg.setSourceID(mob.getObjectUUID());
+        msg.setStartCoord(mob.loc);
+        msg.setEndCoord(goal);
+        Regions region = Regions.getRegionAtLocation(goal);
+        if (region != null) {
+            msg.setInBuildingFloor(region.room);
+            msg.setInBuilding(region.level);
+            msg.setStartLocType(0);
+            msg.setInBuildingUUID(region.parentBuildingID);
+        } else {
+            msg.setInBuildingFloor(-1);
+            msg.setInBuilding(-1);
+            msg.setStartLocType(0);
+            msg.setInBuildingUUID(0);
+        }
+
+
+        try {
+            MovementManager.movement(msg, mob);
+        } catch (MsgSendException e) {
+            // TODO Figure out how we want to handle the msg send exception
+            e.printStackTrace();
+        }
+    }
+}

src/engine/net/client/handlers/ChangeAltitudeHandler.java

@@ -79,7 +79,7 @@ public class ChangeAltitudeHandler extends AbstractClientMsgHandler {
                     float startAlt = 0;
                     Building regionBuilding = Regions.GetBuildingForRegion(upRegion);
                     if (upRegion != null)
-                        startAlt = upRegion.lerpY(pc) - regionBuilding.getLoc().y;
+                        startAlt = upRegion.lerpY(pc.loc) - regionBuilding.getLoc().y;
                     float rounded = startAlt * .10f;
 
                     rounded = ((int) rounded) * 10;
@@ -143,7 +143,7 @@ public class ChangeAltitudeHandler extends AbstractClientMsgHandler {
                 float landingAltitude = 0;
                 Building building = Regions.GetBuildingForRegion(region);
                 if (building != null)
-                    landingAltitude = region.lerpY(pc) - building.getLoc().y;
+                    landingAltitude = region.lerpY(pc.loc) - building.getLoc().y;
 
                 if (landingAltitude >= targetAlt) {
                     pc.landingRegion = region;

src/engine/objects/AbstractCharacter.java

@@ -25,6 +25,7 @@ import engine.jobs.TrackJob;
 import engine.math.AtomicFloat;
 import engine.math.Bounds;
 import engine.math.Vector3fImmutable;
+import engine.mobileAI.utilities.PathingUtilities;
 import engine.net.ByteBufferWriter;
 import engine.net.DispatchMessage;
 import engine.net.client.msg.ErrorPopupMsg;
@@ -120,11 +121,16 @@ public abstract class AbstractCharacter extends AbstractWorldObject {
     private long lastHateUpdate = 0;
     private byte aoecntr = 0;
 
+    public Vector3fImmutable destination = Vector3fImmutable.ZERO;
+
     public int hidden = 0; // current rank of hide/sneak/invis
     public CopyOnWriteArrayList<Integer> minions = new CopyOnWriteArrayList();
 
     public ArrayList<CharacterRune> runes;
 
+    public ArrayList<PathingUtilities.Node> navPath = new ArrayList<>();
+    public boolean isPathing = false;
+
     public AbstractCharacter() {
         super();
         this.firstName = "";
@@ -834,6 +840,10 @@ public abstract class AbstractCharacter extends AbstractWorldObject {
         if (this.isCasting && this.getObjectType().equals(GameObjectType.PlayerCharacter))
             return false;
 
+        if(this.getObjectType().equals(GameObjectType.Mob)){
+            if(this.destination.equals(Vector3fImmutable.ZERO))
+                return false;
+        }
         return true;
     }
 
@@ -987,22 +997,11 @@ public abstract class AbstractCharacter extends AbstractWorldObject {
     @Override
     public final void setLoc(final Vector3fImmutable value) {
 
-        Building building = BuildingManager.getBuildingAtLocation(this.loc);
-        Regions region = null;
-        if(building != null) {
-            //look for region in the building we are in
-            for (Regions regionCycle : building.getBounds().getRegions()) {
-                float regionHeight = regionCycle.highLerp.y - regionCycle.lowLerp.y;
-                if(regionHeight < 10)
-                    regionHeight = 10;
-                if (regionCycle.isPointInPolygon(value) && Math.abs(regionCycle.highLerp.y - value.y) < regionHeight)
-                    region = regionCycle;
-            }
-        }
+        Regions region = Regions.getRegionAtLocation(value);
         float regionHeightOffset = 0;
         if(region != null){
             this.region = region;
-            regionHeightOffset = region.lerpY(this);
+            regionHeightOffset = region.lerpY(this.loc);
             this.inBuilding = region.level; // -1 not in building 0 on ground floor, 1 on first floor etc
             this.inBuildingID = region.parentBuildingID;
             this.inFloorID = region.room;

src/engine/objects/AbstractWorldObject.java

@@ -179,7 +179,7 @@ public abstract class AbstractWorldObject extends AbstractGameObject {
             if (region.center.y == region.highLerp.y)
                 worldObject.loc = worldObject.loc.setY(region.center.y + worldObject.getAltitude());
             else
-                worldObject.loc = worldObject.loc.setY(region.lerpY(worldObject) + worldObject.getAltitude());
+                worldObject.loc = worldObject.loc.setY(region.lerpY(worldObject.loc) + worldObject.getAltitude());
 
         return region;
     }
@@ -504,7 +504,7 @@ public abstract class AbstractWorldObject extends AbstractGameObject {
             this.loc = loc;
 
             if(this instanceof AbstractCharacter && this.region != null){
-                this.loc = this.loc.setY(this.region.lerpY(this) + this.getAltitude());
+                this.loc = this.loc.setY(this.region.lerpY(this.loc) + this.getAltitude());
             } else{
                 this.loc = this.loc.setY(Terrain.getWorldHeight(this.getLoc()) + this.getAltitude());
             }

src/engine/objects/Building.java

@@ -34,6 +34,7 @@ import engine.net.client.msg.UpdateObjectMsg;
 import engine.server.MBServerStatics;
 import org.pmw.tinylog.Logger;
 
+import java.awt.geom.Path2D;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.time.LocalDateTime;
@@ -99,6 +100,8 @@ public class Building extends AbstractWorldObject {
     private ConcurrentHashMap<Integer, Condemned> condemned;
     private ArrayList<Building> children = null;
 
+    public ArrayList<Path2D.Float> meshes;
+
     /**
      * ResultSet Constructor
      */
@@ -910,7 +913,7 @@ public class Building extends AbstractWorldObject {
 
         // Note: We handle R8 tree edge case for mesh and health
         // after city is loaded to avoid recursive result set call
-        // in City resulting in a stack ovreflow.
+        // in City resulting in a stack overflow.
 
         if (blueprint != null) {
 
@@ -1003,6 +1006,8 @@ public class Building extends AbstractWorldObject {
 
         if (this.upgradeDateTime != null)
             BuildingManager.submitUpgradeJob(this);
+
+        BuildingManager.bakeNavMesh(this); // update the navmesh of the parent zone
     }
 
     public synchronized boolean setOwner(AbstractCharacter newOwner) {
@@ -1530,4 +1535,5 @@ public class Building extends AbstractWorldObject {
     public void RemoveFromBarracksList() {
 
     }
+
 }

src/engine/objects/MeshBounds.java

@@ -12,6 +12,8 @@ package engine.objects;
 import engine.gameManager.DbManager;
 import engine.math.Bounds;
 
+import java.awt.*;
+import java.awt.geom.Area;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 
@@ -45,5 +47,10 @@ public class MeshBounds {
         Bounds.meshBoundsCache = DbManager.BuildingQueries.LOAD_MESH_BOUNDS();
     }
 
-
+    public Area getArea(float x, float z){
+        Polygon area = new Polygon();
+        area.addPoint((int)(x + this.minX), (int) (z + this.minZ));
+        area.addPoint((int)(x + this.maxX), (int)(z + this.maxZ));
+        return  new Area(area);
+    }
 }

src/engine/objects/Mob.java

@@ -53,7 +53,7 @@ public class Mob extends AbstractIntelligenceAgent implements Delayed {
     public long nextCallForHelp = 0;
     public ReentrantReadWriteLock minionLock = new ReentrantReadWriteLock();
     public boolean despawned = false;
-    public Vector3fImmutable destination = Vector3fImmutable.ZERO;
+
     public MobBase mobBase;
     public int spawnDelay;
     public Zone parentZone;
@@ -1899,4 +1899,5 @@ public class Mob extends AbstractIntelligenceAgent implements Delayed {
     public int compareTo(@NotNull Delayed o) {
         return toIntExact(this.respawnTime - ((Mob) o).respawnTime);
     }
+
 }

src/engine/objects/PlayerCharacter.java

@@ -4828,7 +4828,7 @@ public class PlayerCharacter extends AbstractCharacter {
             if (this.landingRegion != null) {
                 this.altitude = 0;
                 this.region = this.landingRegion;
-                this.loc = this.loc.setY(this.landingRegion.lerpY(this));
+                this.loc = this.loc.setY(this.landingRegion.lerpY(this.loc));
             } else
                 this.altitude = this.getDesiredAltitude();
 

src/engine/objects/Regions.java

@@ -9,14 +9,18 @@
 
 package engine.objects;
 
+import engine.InterestManagement.Terrain;
 import engine.InterestManagement.WorldGrid;
 import engine.gameManager.BuildingManager;
 import engine.math.Bounds;
 import engine.math.FastMath;
 import engine.math.Vector3f;
 import engine.math.Vector3fImmutable;
+import engine.mobileAI.utilities.PathingUtilities;
 import engine.server.MBServerStatics;
 
+import java.awt.*;
+import java.awt.geom.Area;
 import java.util.ArrayList;
 import java.util.HashMap;
 
@@ -178,7 +182,7 @@ public class Regions {
             boolean movingUp = false;
 
             boolean movingDown = false;
-            float yLerp = worldObject.region.lerpY(worldObject);
+            float yLerp = worldObject.region.lerpY(worldObject.loc);
 
             if (yLerp == (worldObject.region.highLerp.y))
                 movingUp = true;
@@ -270,32 +274,18 @@ public class Regions {
         return BuildingManager.getBuildingFromCache(region.parentBuildingID);
     }
 
-    public static Regions GetRegionForTeleport(Vector3fImmutable location) {
+    public static Regions getRegionAtLocation(Vector3fImmutable location) {
         Regions region = null;
-
-
-        //Find building
-        for (AbstractWorldObject awo : WorldGrid.getObjectsInRangePartial(location, 128, MBServerStatics.MASK_BUILDING)) {
-            Building building = (Building) awo;
-            if (!Bounds.collide(location, building.getBounds()))
-                continue;
-            if(building != null) {
-                region = BuildingManager.GetRegion(building, location.x, location.y, location.z);
+        Building building = BuildingManager.getBuildingAtLocation(location);
+        if(building != null) {
+            //look for region in the building we are in
+            for (Regions regionCycle : building.getBounds().getRegions()) {
+                float regionHeight = regionCycle.highLerp.y - regionCycle.lowLerp.y;
+                if(regionHeight < 10)
+                    regionHeight = 10;
+                if (regionCycle.isPointInPolygon(location) && Math.abs(regionCycle.highLerp.y - location.y) < regionHeight)
+                    region = regionCycle;
             }
-            //find regions that intersect x and z, check if object can enter.
-            //for (Regions toEnter : building.getBounds().getRegions()) {
-            //    if (toEnter.isPointInPolygon(location)) {
-
-            //        if (region == null)
-            //            region = toEnter;
-            //        else // we're using a low level to high level tree structure, database not always in order low to high.
-                        //check for highest level index.
-            //            if (region != null && toEnter.highLerp.y > region.highLerp.y)
-            //                region = toEnter;
-
-
-            //    }
-           // }
         }
         return region;
     }
@@ -350,10 +340,10 @@ public class Regions {
         return inside;
     }
 
-    public float lerpY(AbstractWorldObject lerper) {
+    public float lerpY(Vector3fImmutable lerper) {
 
         Vector3fImmutable lengthVector = this.highLerp.subtract2D(this.lowLerp);
-        Vector3fImmutable characterVector = lerper.getLoc().subtract2D(this.lowLerp);
+        Vector3fImmutable characterVector = lerper.subtract2D(this.lowLerp);
         float lengthVectorMagnitude = lengthVector.magnitude();
         float characterVectorMagnitude = characterVector.magnitude();
         float percentDistance = characterVectorMagnitude / lengthVectorMagnitude;
@@ -374,4 +364,12 @@ public class Regions {
     public boolean isExit() {
         return exit;
     }
+
+    public Area getArea(){
+        Polygon area = new Polygon();
+        for( Vector3f point : this.regionPoints){
+            area.addPoint((int) point.x, (int) point.z);
+        }
+        return  new Area(area);
+    }
 }

src/engine/objects/Zone.java

@@ -9,6 +9,7 @@
 
 package engine.objects;
 
+import java.awt.geom.Area;
 import engine.Enum;
 import engine.InterestManagement.Terrain;
 import engine.db.archive.DataWarehouse;
@@ -16,10 +17,12 @@ import engine.gameManager.ZoneManager;
 import engine.math.Bounds;
 import engine.math.Vector2f;
 import engine.math.Vector3fImmutable;
+import engine.mobileAI.utilities.PathingUtilities;
 import engine.net.ByteBufferWriter;
 import engine.server.MBServerStatics;
 import org.pmw.tinylog.Logger;
-
+import java.awt.*;
+import java.awt.geom.Path2D;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.util.ArrayList;
@@ -64,6 +67,8 @@ public class Zone extends AbstractWorldObject {
     public float sea_level;
 
     public Terrain terrain = null;
+    public ArrayList<Path2D.Float> navObstacles = new ArrayList<>();
+    public ArrayList<PathingUtilities.Node> navNodes = new ArrayList<>();
 
     /**
      * ResultSet Constructor
@@ -180,9 +185,7 @@ public class Zone extends AbstractWorldObject {
         }
 
         ZoneManager.populateZoneCollections(this);
-
     }
-
     /* Method sets a default value for player cities
      * otherwise using values derived from the loadnum
      * field in the obj_zone database table.

src/engine/powers/poweractions/TeleportPowerAction.java

@@ -96,7 +96,7 @@ public class TeleportPowerAction extends AbstractPowerAction {
 
         //TODO verify target loc is valid loc
 
-        Regions region = Regions.GetRegionForTeleport(targetLoc);
+        Regions region = Regions.getRegionAtLocation(targetLoc);
 
         if (region != null && !region.isOutside())
             return;

src/engine/server/world/WorldServer.java

@@ -403,6 +403,9 @@ public class WorldServer {
 		Logger.info("Loading building slot/stuck location data.");
 		BuildingLocation.loadBuildingLocations();
 
+		Logger.info("Loading mesh hulls.");
+		DbManager.BuildingQueries.LOAD_CONVEX_HULLS();
+
 		// Starting before loading of structures/guilds/characters
 		// so the database connections are available to write
 		// historical data.