Scripting Helpers is winding down operations and is now read-only. More info→

So Path.Blocked doesnt work... Is there a way around it?

Asked by

Adv3rtizement 101

5 years ago

Edited 5 years ago

So I was working with the pathfinding service and everything was going fine... until I realized that Path.Blocked apparently doesn't work correctly! As a result, I've been trying to find a way around this. I came up with a solution of maybe making lines through the waypoints and checking for a touched event or something? I'm not sure how I'd do that though. If you have any ideas I'm all ears!

Here is my code - it needs to create a new path if a part blocks the path....

001--Pahfinding service
002local pathfinding = game:GetService("PathfindingService")
003--Settings for the path
004local obHorSep = 3
005local obVerSep = 5
006--Variables for storing the waypoints and the current waypoint index
007local waypoints
008local curWayIndex
009--Variables for npc, npc humanoid, the object, and the destination.
010local npc
011local npcHum
012local part
013local destination
014--Stores if the npc is moving
015local moving = false
016--Creates Path
017function createPath()
018    --Creates the path type
019    local path = pathfinding:CreatePath({AgentRadius=obHorSep,AgentHeight=obVerSep})
020    --Creates path between points
021    path:ComputeAsync(npcHum.RootPart.Position,destination)
022    path:ComputeAsync(npcHum.RootPart.Position,destination)
023    --Checks if the path was successfully made
024    if path.Status == Enum.PathStatus.Success then
025        waypoints = path:GetWaypoints() --Gets waypoints
026        --Sets the index to the start
027        curWayIndex = 1
028        --Starts movement
029        moving = true
030        npcHum:MoveTo(waypoints[curWayIndex].Position)
031        --After first movement, the moveToFinish will complete the rest
032    else
033        --Path was not successfully made
034    end
035    --Connects pathBlocked for if the path gets blocked
036    path.Blocked:Connect(pathBlocked)
037    local color = BrickColor.Random()
038    for _, v in pairs(waypoints) do
039        local ball = Instance.new("Part",workspace)
040        ball.Size = Vector3.new(1,1,1)
041        ball.BrickColor = color
042        ball.Shape = Enum.PartType.Ball
043        ball.Material = Enum.Material.Neon
044        ball.Anchored = true
045        ball.Position = v.Position
046        ball.CanCollide = false
047    end
048end
049--Sets values
050function setValues(npc1,part1)
051    --Makes the values public
052    npc = npc1
053    part = part1
054    --Finds npc humanoid
055    for _,v in pairs(npc:GetDescendants()) do
056        --Checks if descendant is humanoid
057        if v:IsA("Humanoid") then
058            --Sets the public npcHum value to the humanoid
059            npcHum = v
060            --Connects moveToFinish to npcHum so it reached next waypoint.
061            npcHum.MoveToFinished:Connect(moveToFinish)
062            break
063        end
064    end
065    --Finds the destination depending on if it is a model or part
066    if part:IsA("Model") then --If part is a model
067        if part.PrimaryPart then --Check if it has a primary part
068            destination = part.PrimaryPart.position
069        else --if model doesn't have primary part
070            --Set destination as the first part object in it.
071            destination = part:FindFirstChildOfClass("Part").position
072        end
073    else --If the part isn't a model
074        --Just set the destination to the part's position
075        destination = part.Position
076    end
077    createPath()
078end
079--Runs if path gets blocked!
080function pathBlocked(blockIdx)
081    print("Oh no! Blockage in the path!")
082    --Checks if blockage is behind or ahead.
083    if blockIdx > curWayIndex then --It is infront
084        print("The blockage is infront, we need to make a new path!")
085        --create a new path
086        createPath()
087    else
088        --It is behind and nothing needs to happen
089        print("Dont worry, the blockage is behind!")
090    end
091end
092--Runs after the humanoid finishes moving to one waypoint
093function moveToFinish(reached)
094    --Checks if npc has reached point and hasnt reached endpoint.
095    if reached and curWayIndex < #waypoints then
096        --Ups waypoint Index
097        curWayIndex = curWayIndex+1
098        --Moves to next index
099        npcHum:MoveTo(waypoints[curWayIndex].Position)
100    elseif curWayIndex == #waypoints then
101        moving = false
102        print("Point Reached!")
103    end
104end
105setValues(script.Parent,workspace.Test.PartSpecial)
 View all 105 lines...

Oh also I read about this... https://devforum.roblox.com/t/pathfindingservice-path-blocked-not-firing-why/278379 They seemed to have the same problem as me. There was a kind of solution at the end, but I'm not sure how I'd even do that!

I tried to change it to:

01if path.Status == Enum.PathStatus.Success then
02        waypoints = path:GetWaypoints() --Gets waypoints
03        --Sets the index to the start
04        curWayIndex = 1
05        --Starts movement
06        moving = true
07        --START CHECKING FOR VELOCITY CHANGE
08        npcHum.RootPart:GetPropertyChangedSignal('Velocity'):Connect(velocityChanged)
09        npcHum:MoveTo(waypoints[curWayIndex].Position)
10        --After first movement, the moveToFinish will complete the rest
11    else

adding this to it:

`1`	`--START CHECKING FOR VELOCITY CHANGE`

`2`	`npcHum.RootPart:GetPropertyChangedSignal('Velocity'):Connect(velocityChanged)`

and then I added this function:

1--Checks if velocity has gone below 0.1 (not moving)
2function velocityChanged()
3    print("Changed")
4    if(npcHum.RootPart.Velocity.magnitude < 0.1) then
5        moving = false
6        createPath()
7    end
8end

Since if it did fire, it would create an infinite loop and moved the connection to here:

01for _,v in pairs(npc:GetDescendants()) do
02        --Checks if descendant is humanoid
03        if v:IsA("Humanoid") then
04            --Sets the public npcHum value to the humanoid
05            npcHum = v
06            --Connects moveToFinish to npcHum so it reached next waypoint.
07            npcHum.MoveToFinished:Connect(moveToFinish)
08            --START CHECKING FOR VELOCITY CHANGE
09            npcHum.RootPart:GetPropertyChangedSignal('Velocity'):Connect(velocityChanged)
10            break
11        end
12    end

But it still won't work?

2 answers

Answered by

sheepposu 561

5 years ago

I was reading at the end of the article, someone said to check the velocity. You can check it this way object.Velocity.magnitude where object is the object that you want the velocity of. You can check if it is less than 0.1 and make a new path if so, quoted from the article.

Is there a way to connect this to a changed event or something? Adv3rtizement 101 — 5y

object:GetPropertyChangedSignal('Velocity'):Connect(function() sheepposu 561 — 5y

if part.Velocity.Magnitude then sheepposu 561 — 5y

--code sheepposu 561 — 5y

View all comments (20 more)

Didn't work, my attempted code has been edited into the post. Adv3rtizement 101 — 5y

You could try checking for a change in the velocities magnitude sheepposu 561 — 5y

GetPropertyChangedSignal is not a valid member of Vector3 Adv3rtizement 101 — 5y

Thought that might happen sheepposu 561 — 5y

Is the velocity changing in the first place sheepposu 561 — 5y

Well the npc is moving Adv3rtizement 101 — 5y

Everything works fine except for the fact that it doesn't change it's pathway. Adv3rtizement 101 — 5y

Does the npc ever stop moving, or is it not moving in the first place sheepposu 561 — 5y

I did a test... it prints out different velocities. Adv3rtizement 101 — 5y

If I don't mess with anything, it will move all the way to the end without stopping. If I put a wall in the path while it is moving it will walk into the wall and stop Adv3rtizement 101 — 5y

hmm... then the event should go off. I also noticed something, the GetPropertyChangedSignal should only be executed once. sheepposu 561 — 5y

Otherwise it will start a bunch of events which can eventually crash the game sheepposu 561 — 5y

How come? Adv3rtizement 101 — 5y

Everytime you create a new path, it would create a new event (event will constantly check), after accumulating so many events running it will most likely crash sheepposu 561 — 5y

Ok, I'll move it to another then Adv3rtizement 101 — 5y

I changed it - look at the post Adv3rtizement 101 — 5y

I think it would work better if you just put it by itself, for example, directly after the velocityChanged function. sheepposu 561 — 5y

After you do that, test it out and see if the event is firing sheepposu 561 — 5y

npcHum isn't set uptil the setValues function Adv3rtizement 101 — 5y

You can put it after the setValues function sheepposu 561 — 5y

Answered by

Adv3rtizement 101

5 years ago

Well, apparently a while loop works too :P - Not laggy

001--Pahfinding service
002local pathfinding = game:GetService("PathfindingService")
003--Settings for the path
004local obHorSep = 3
005local obVerSep = 5
006--Variables for storing the waypoints and the current waypoint index
007local waypoints
008local curWayIndex
009--Variables for npc, npc humanoid, the object, and the destination.
010local npc
011local npcHum
012local part
013local destination
014--Stores if the npc is moving
015local moving = false
016local reachedDes = false
017--Creates the path type
018path = pathfinding:CreatePath({AgentRadius=obHorSep,AgentHeight=obVerSep})
019--Creates Path
020function createPath()
021    --Creates path between points
022    path:ComputeAsync(npcHum.RootPart.Position,destination)
023    path:ComputeAsync(npcHum.RootPart.Position,destination)
024    --Checks if the path was successfully made
025    if path.Status == Enum.PathStatus.Success then
026        waypoints = path:GetWaypoints() --Gets waypoints
027        --Sets the index to the start
028        curWayIndex = 1
029        --Starts movement
030        moving = true
031        npcHum:MoveTo(waypoints[curWayIndex].Position)
032        --After first movement, the moveToFinish will complete the rest
033    else
034        --Path was not successfully made
035    end
036    --Connects pathBlocked for if the path gets blocked
037    path.Blocked:Connect(pathBlocked)
038    local color = BrickColor.Random()
039end
040--Sets values
041function setValues(npc1,part1)
042    --Makes the values public
043    npc = npc1
044    part = part1
045    --Finds npc humanoid
046    for _,v in pairs(npc:GetDescendants()) do
047        --Checks if descendant is humanoid
048        if v:IsA("Humanoid") then
049            --Sets the public npcHum value to the humanoid
050            npcHum = v
051            --Connects moveToFinish to npcHum so it reached next waypoint.
052            npcHum.MoveToFinished:Connect(moveToFinish)
053            break
054        end
055    end
056    --Finds the destination depending on if it is a model or part
057    if part:IsA("Model") then --If part is a model
058        if part.PrimaryPart then --Check if it has a primary part
059            destination = part.PrimaryPart.position
060        else --if model doesn't have primary part
061            --Set destination as the first part object in it.
062            destination = part:FindFirstChildOfClass("Part").position
063        end
064    else --If the part isn't a model
065        --Just set the destination to the part's position
066        destination = part.Position
067    end
068    createPath()
069end
070--Runs if path gets blocked!
071function pathBlocked(blockIdx)
072    print("Oh no! Blockage in the path!")
073    --Checks if blockage is behind or ahead.
074    if blockIdx > curWayIndex then --It is infront
075        print("The blockage is infront, we need to make a new path!")
076        --create a new path
077        createPath()
078    else
079        --It is behind and nothing needs to happen
080        print("Dont worry, the blockage is behind!")
081    end
082end
083--Runs after the humanoid finishes moving to one waypoint
084function moveToFinish(reached)
085    --Checks if npc has reached point and hasnt reached endpoint.
086    if reached and curWayIndex < #waypoints then
087        --Ups waypoint Index
088        curWayIndex = curWayIndex+1
089        --Moves to next index
090        npcHum:MoveTo(waypoints[curWayIndex].Position)
091    elseif curWayIndex == #waypoints then
092        moving = false
093        reachedDes = true
094        print("Point Reached!")
095    end
096end
097--Checks if velocity has gone below 0.1 (not moving)
098function velocityChanged()
099    if(npcHum.RootPart.Velocity.Magnitude < 0.1) then
100        moving = false
101        createPath()
102    end
103end
104while wait(1)do
105    velocityChanged()
106    if reachedDes then
107        break
108    end
109end
110setValues(script.Parent,workspace.Test.PartSpecial)
 View all 110 lines...

So Path.Blocked doesnt work... Is there a way around it?

2 answers

Answer this question