ronlix / webrts

Browserbased WebRTS engine (clean version of ronco.packagecloud.com)
ronco.packagecloud.com
11 stars 1 forks source link

Astar implementation #1

Open jimisaacs opened 12 years ago

jimisaacs commented 12 years ago

Felix,

Hello, I found your webtrs repo after stumbling on your astar library from a link to is on Brian Grinstead's astar repo. I've been testing all of the open source javascript astar implementations out there that I can find, and so far yours seems to work the best and is the fastest.

There is one thing I wanted to ask you about though, specifically with lines 62 through 64 in a_star.js. You say you are creating an array for each row, when in fact it is based on the X (width) dimension of the map which I thought represented the columns not the rows. This is confusing, and I wanted to ask what you were going for here.

Thanks, Jim

ronlix commented 12 years ago

Hi Jim,

And you are right! There is a mistake in my documentation. It creates a array for each colum ..

I will change it soon :)

Thanks for showing it .. if you find more mistakes .. please let me know :)

Felix

2011/9/12 Jim Isaacs reply@reply.github.com:

Felix,

Hello, I found your webtrs repo after stumbling on your astar library from a link to is on Brian Grinstead's astar repo. I've been testing all of the open source javascript astar implementations out there that I can find, and so far yours seems to work the best and is the fastest.

There is one thing I wanted to ask you about though, specifically with lines 62 through 64 in a_star.js. You say you are creating an array for each row, when in fact it is based on the X (width) dimension of the map which I thought represented the columns not the rows. This is confusing, and I wanted to ask what you were going for here.

Thanks, Jim

Reply to this email directly or view it on GitHub: https://github.com/ronlix/webrts/issues/1

Felix Niessen Tel.:0212/6588820 Schwarz/Niessen/Götz GbR caendle Nils Schwarz, Felix Niessen, Stefan Götz Lehner Str. 33, 42655 Solingen