Speedwalk doesn't recognize down

[Chris Leacy]

OS: Windows7 Version: 3.0-delta

The speedwalk command is not properly recognizing the down direction. It might be a problem with how speedWalkDir is built, rather then how speedwalk interprets it. The below code gives the following results:

echo("Path we need to take: " .. table.concat(speedWalkDir, ", ") .. "\n")
echo("Rooms we'll pass through: " .. table.concat(speedWalkPath, ", ") .. "\n")
speedwalk(table.concat(speedWalkDir, ", "), false)

Path we need to take: down, n, e, s, w
Rooms we'll pass through: 48603, 48602, 48577, 48578, 84203
d
ow
n
n
e
s
w

For now, i'm getting around it by prefixing the call to speedwalk with the following:

for k,v in pairs(speedWalkDir) do
 if v == "down" then
  speedWalkDir[k] = "d"
 end
end

[Vadim Peretokin (vperetokin)]

The speedwalk function is whatever you define it to be - so this isn't a bug against Mudlet, but your implementation (or someone's that you're using) of speedwalk().

[Chris Leacy (cleacy1972)]

Understood -

The doSpeedWalk() function is defined by me, but speedwalk() is defined by mudlet.

http://wiki.mudlet.org/w/Manual:Mapper_Functions#speedwalk

The return value I receive from the mapper in speedWalkDir with the above sample is

{
  "down",
  "n",
  "e",
  "s",
  "w"
}

I thought perhaps the value inserted in speedWalkDir should have been 'd' rather then 'down' so the speedwalk() command itself would work, but if it's by design that's ok also.

[Stephen Lyons (slysven)]

The commands for the non-XY-Plane are indeed "up", "down", "in" and "out" - it is anticipated that there will be the ability for the user to customise what gets put into speedWalkDir for each of the "normal" exits in the future as that will be needed for proper support of non-English language MUDs/Users... 8-P

[Vadim Peretokin (vperetokin)]

Migrating issues to Github, please follow the new discussion here: https://github.com/Mudlet/Mudlet/issues/503

This issue needs to be closed and there is no appropriate status, so will set it to "Opinion" just for migration purposes.