Nuntar
Level: Smitemaster

Rank Points: 4590
Registered: 02-20-2007
IP: Logged

File: pathfinding bug.hold (1.1 KB)
Downloaded 42 times.
License: Public Domain

In the attached hold, the mud coordinator's script consists of "Move to (5,7) Single step" followed by a Speech command. He is completely blocked off, so he can't make a move.

The F1 in-game help says: "If no move can be made, the next script command is executed immediately." So you would expect him to make his speech on the first turn. He actually doesn't speak until the second turn, so the help is wrong and should be corrected.

The tar coordinator has the same script, but with Imperative Pathfinding. He doesn't reach the Speech command at all. This is definitely a bug; the whole point of "Single step" is that it doesn't keep a character occupied for more than one turn, making it possible to, for example, write loops that give characters regular behaviour such as trying to move every fifth turn.

____________________________
50th Skywatcher

IQubic
Level: Delver
Rank Points: 78
Registered: 11-22-2015
IP: Logged

I have fixed this. There was a similar issue of move (relative version) waiting for an extra turn. These commands are now skipped if no move is made. If a move is made, then, just like always, that is the character's entire turn.

I'll make a PR as soon as I get git to work.

____________________________
119th Skywatcher

---

[Last edited by IQubic at 05-16-2017 07:53 AM]

skell
Level: Legendary Smitemaster

Rank Points: 3734
Registered: 12-28-2004
IP: Logged

IQubic wrote:
I have fixed this. There was a similar issue of move (relative version) waiting for an extra turn. These commands are now skipped if no move is made. If a move is made, then, just like always, that is the character's entire turn.

Unfortunately at this point we can't change that. This behavior has been in the game for the good part of the decade... Actually for 14 years now I guess? And it's more than likely it'll break thousands of demos and even make rooms impossible.

What we have to do in this task is:

1. Make Move+Single Step work fine with Imperative: Pathfinding, that is if no move is made the command is skipped.
2. Update the docs. I'll actually do it myself so, please do the thing from point #1.

____________________________
My website | Facebook | Twitter

IQubic
Level: Delver
Rank Points: 78
Registered: 11-22-2015
IP: Logged

Skell, what would you like me to do? Can you explain more clearly?

____________________________
119th Skywatcher

skell
Level: Legendary Smitemaster

Rank Points: 3734
Registered: 12-28-2004
IP: Logged

IQubic wrote:
Skell, what would you like me to do? Can you explain more clearly?

Yes:

The tar coordinator has the same script, but with Imperative Pathfinding. He doesn't reach the Speech command at all. This is definitely a bug; the whole point of "Single step" is that it doesn't keep a character occupied for more than one turn, making it possible to, for example, write loops that give characters regular behaviour such as trying to move every fifth turn.

Make Move+Single Step work fine with Imperative: Pathfinding, that is if no move is made the command is skipped.

Basically, if a character has Imperative Pathfinding set, Move with Single Step should not wait until a move is available.

____________________________
My website | Facebook | Twitter

skell
Level: Legendary Smitemaster

Rank Points: 3734
Registered: 12-28-2004
IP: Logged

And additionally I'd like you to write a test case to cover that, hit me on the chat if you need help with that.

____________________________
My website | Facebook | Twitter

skell
Level: Legendary Smitemaster

Rank Points: 3734
Registered: 12-28-2004
IP: Logged

Alright, this has been fixed:

* Pathmapping movement types should work correctly with Single Step in Move/MoveTo.
* Updated help to correctly indicate that single step that fails still consumes a turn.

____________________________
My website | Facebook | Twitter

kieranmillar
Level: Smitemaster
Rank Points: 2670
Registered: 07-11-2014
IP: Logged

In 5.1.1.alpha.2020-10-16, in Nuntar's attached demonstration hold both characters speak on turn 2. This means the failing Move To Single Step is a blocking command. It is not clear to me reading the thread if this is intended or not, or if both characters should speak on turn 1.

I am unable to test if the help docs have been updated.

skell
Level: Legendary Smitemaster

Rank Points: 3734
Registered: 12-28-2004
IP: Logged

kieranmillar wrote:
In 5.1.1.alpha.2020-10-16, in Nuntar's attached demonstration hold both characters speak on turn 2. This means the failing Move To Single Step is a blocking command. It is not clear to me reading the thread if this is intended or not, or if both characters should speak on turn 1.

That's correct. Single Step should always be a blocking command if movement is impossible, the docs were incorrect in that regard. The bug was that when using pahtfinding movement Single Step would cause the command to never continue, as every turn it'd try to move again.

As for the docs, I'll change status to Fixed, Unreleased and change release version to 5.1.1 Release Candidate

____________________________
My website | Facebook | Twitter

---

[Last edited by skell at 10-18-2020 11:39 AM]