|
I'm using pathfinding and a clickdetector to move a bot from its position to a workbench. However sometimes the bot decides to walk over the workbench or get stuck on the side of it, I tried reducing the EmptyCutoff but it still happens.
The code involved:
function moveToPoint(pointName, AI)
local path = pathfindingService:ComputeRawPathAsync(AI.Torso.Position, pointName.Position, 500)
for _, point in pairs(path:GetPointCoordinates()) do
AI.Humanoid:MoveTo(point)
repeat
local distance = (point - AI.Torso.Position).magnitude
wait()
until distance < 3
end
print("MovedToPoint "..pointName.Name)
return "Done"
end
MyModule.RequestMovement = function(model)
local chosenAi = workspace.ShopWorker
if chosenAi.Busy.Value == true then
print('Busy')
else
chosenAi.Busy.Value = true
moveToPoint(model['StopPoint'], chosenAi)
chosenAi:MoveTo(model['StopPoint'].Position)
chosenAi.Torso.CFrame = CFrame.new(chosenAi.Torso.Position)
wait(3)
chosenAi.Busy.Value = false
end
end
|
|
|
|
|
b3 like a butterfly. b4 you consider bumping to b5, try changing the until distance <3 to until distance < 1, idk, might work.
#Strikin' |
|
|
The problem is not how close he is getting to the part, the problem is the path that is being constructed :/ It goes over parts or too close to them, which I assumed must be due to the voxel detection finding the parts were too small, but reducing the EmptyCutoff did nothing :(
|
|
RelrionJoin Date: 2010-06-17
Post Count: 577 |
Why not make it so instead of going straight to the bench, it just gets within 6 studs and then sets Humanoid:MoveTo(insertBenchPos)? BTW it might help to add something to check for path occlusion, not uncommon for them to get stuck and need to recalculate. |
|
|
Could you give an example of path occlusion? Can't find it on wiki.
|
|
|
|
|
|
|
|
|
|
|
RelrionJoin Date: 2010-06-17
Post Count: 577 |
path:CheckOcclusionAsync(starting Index)
|
|
|
So would it be:
for _, point in pairs(path:GetPointCoordinates()) do
if path:CheckOcclusionAsync(point) == -1 then
AI.Humanoid:MoveTo(point)
repeat
local distance = (point - AI.Torso.Position).magnitude
wait()
until distance < 3
else
--Make new path
end
|
|
RelrionJoin Date: 2010-06-17
Post Count: 577 |
I would move the check to the repeat loop, because sometimes the path will become blocked during the MoveTo. Also, path:CheckOcclusionAsync(prevPoint) so it uses the point that was previously achieved instead of the one its trying to get to.
path:CheckOcclusionAsync(start) - If you use the point your trying to get to it will only check the path 'After'. So I recommend storing the previously achieved point outside the loop for the later iteration.
|
|
|
This line errors:
path:CheckOcclusionAsync(prevPoint)
Attempt to cast Vector3 to Int
|
|
|
local path = pathfindingService:ComputeRawPathAsync(AI.Torso.Position, pointName.Position, 500)
local prevPoint = 0
for i, point in pairs(path:GetPointCoordinates()) do
if i == 1 then prevPoint = point end
AI.Humanoid:MoveTo(point)
repeat
local distance = (point - AI.Torso.Position).magnitude
wait()
print(path:CheckOcclusionAsync(prevPoint))
if path:CheckOcclusionAsync(prevPoint) == -1 then
print('is ok')
else
print('wow stupid path')
end
until distance < 3
prevPoint = point
|
|
RelrionJoin Date: 2010-06-17
Post Count: 577 |
prevPoint = point
prevPoint Should be the index for the point not it's Vector3 value.
prevPoint = i <- Current loop iteration is equal to the index of the point in the points array.
|
|
RelrionJoin Date: 2010-06-17
Post Count: 577 |
for i, point in pairs(path:GetPointCoordinates()) do
instead:
for index,point in pairs(path:GetPointCoordinates()) do
|
|
|
Now it still doesn't take the real obstruction into account(ie workbench) and still gets stuck on it or tries walking over it without and it always says there is obstruction at the second to last point?
local prevPoint = 0
for i, point in pairs(path:GetPointCoordinates()) do
if i == 1 then prevPoint = i end
AI.Humanoid:MoveTo(point)
repeat
local distance = (point - AI.Torso.Position).magnitude
wait()
if path:CheckOcclusionAsync(prevPoint) == -1 then
print('is ok')
else
print(i)
print('obstruction at: '..path:CheckOcclusionAsync(prevPoint))
end
until distance < 3
prevPoint = i
end
|
|
RelrionJoin Date: 2010-06-17
Post Count: 577 |
If the obstruction can be jumped over or climbed, it isn't considered and obstacle.
|
|