PathZilla (v6) - A networking AI

[Yexo]

Yexo (or other devs) - Can an AI library expose more than one class via the import() statement, or would I have to use an inner class, such as...

Currently creating subclasses is the only way. Also I can't really think of a nice syntax for import to allow multiple classes, and what should CreateInstance() (in info.nut) return in that case?

Edit: while I'm saying this I realize both CreateInstance() and import could use arrays of strings instead of just a string, but I don't really like that. Are subclasses a problem in your case?

[Zutty]

Yexo (or other devs) - Can an AI library expose more than one class via the import() statement, or would I have to use an inner class, such as...

Currently creating subclasses is the only way. Also I can't really think of a nice syntax for import to allow multiple classes, and what should CreateInstance() (in info.nut) return in that case?

Edit: while I'm saying this I realize both CreateInstance() and import could use arrays of strings instead of just a string, but I don't really like that. Are subclasses a problem in your case?

It would be nice if the classes in scope within the library could also be in scope within the AI that include()s it. The way include() works, it kind of assumes that each library uses only a single class, which is fine for the pathfinding stuff, but IMHO not as flexible as it could be.

However the design decision has long been made now, I dont think this small gripe justifies changing it. I'm happy enough just to use inner classes. Just thought I'd ask though.

Cheers.

[Yexo]

It would be nice if the classes in scope within the library could also be in scope within the AI that include()s it. The way include() works, it kind of assumes that each library uses only a single class, which is fine for the pathfinding stuff, but IMHO not as flexible as it could be.

The reason it's designed like this it that it allows you to change between different libraries that do the same very easily. Say you use the BinaryHeap library:

Code: Select all

import("queue.binary_heap", "Queue", 1);

After a few months of coding you discover that in your situation the FibonacciHeap library performs better. Then the only thing you have to do is change the import line to:

Code: Select all

import("queue.fibonacci_heap", "Queue", 2);

and you can keep on using the class named Queue just like before. Of course this only works if the classes have similar interfaces, but for the queue libraries this is the case.

[Blustuff]

Nested classes doesn't exists in squirrel but you can write :

Code: Select all

class Outer
{
  static Inner = class
  {
    ...
  }
  ...
}

Remember that classes are first-class values and thus you can assign them to a variable to achieve the same behavior described by Yexo :

Code: Select all

class SomeImplementationOfAQueue
{
  ...
}

class SomeOtherImplementationOfAQueue
{
  ...
}

Queue <- SomeOtherImplementationOfAQueue; /* Not much harder than Queue <- import(...); */

[parcel]

AI crash report ( Sorry for my poor English ).

Wenfingford_Transport.png (65.2 KiB) Viewed 6239 times

And 'Restart AI' button rasie assertion window(I guess It's not AI side problem)

Load and wait a few second, you will see AI crash message.

[Terkhen]

That assertion has been fixed in r19259. It shouldn't appear at the next nightly.

[parcel]

That assertion has been fixed in r19259. It shouldn't appear at the next nightly.

Thank you, Terkhen

[Morloth]

Hey Zutty,

I was running a couple of benchmarks with the AI and I came across the following error message:

Crash

I ran the competition overnight, but it seems it crashed quite early as it had no vehicles on the map what so ever.

Hope this helps.
Cheers,
Bram

[Kogut]

Rather bad idea (fix: canals construction around )

[City Builder]

Hi,

I've been getting these Pathzilla V6 crashes under normal play and was told that this would likely be the thread to report it in.

Hopefully i got the whole thing in one screenshot for ya. Hope it helps

The second part since the first part the window wasn't wide enough to get the full thing.

[Steffl]

Hi, PathZilla V6 has problems with newer OpenTTD-Versions. If you want to play with a similar AI, try "RoadRunner".

http://www.tt-forums.net/viewtopic.php?f=65&t=48281

[mwyeoh]

Just reporting a crash

v 1.0.2- RC-1

[Samu]

hello. PathZilla went bankrupt and it got this error on its second attempt.

ah, and it's 'compatibility', not 'compatability'. lol

[Lord Aro]

i am aware that this ai does not seem to getting updated anymore, but hey, 2 more bugs!:

[ABCRic]

I'm having the "top() on an empty array" issue as well.

[Lord Aro]

Basically, until Zutty fixes it, Pathzilla is broken

[Alteris]

Bug That showed up today:

[LamRis]

Bug report:
OpenTTD r20911 (r20994)
PathZilla (v6) r313

[planetmaker]

Pathzilla v6 is running here currently.

It considered obviously to build routes from a gold mine to a bank and correctly determined that it had no vehicles which could transport the gold. But that could certainly be optimized as that is not so much a route property (it checked for each route being considered) but rather a cargo property (gold has no vehicles in this game).

[Felix Atagong]

Is Pathzilla still under development, anyway here is a bug report...