NoAI Branch - An AI Framework

Zuu · Post by **Zuu** » 19 Mar 2007 22:05

I've looked a bit into it now, and it seams like when the game quits the AI get's controll over the player. See this output: (notice, I loaned some extra money as player to show that it is the player that get messed with.)

Code: Select all

 ./openttd -a "Zuu AI" -g                                                                                                                                           
ZuuAI::Start()
ManageLoan: Want to pay back
Company: LT Transportation Transport
balance: 100000
loan: 100000
max_balance: 40000
pay_back: 60000
new_loan: 40000
Successfully paid back loan
ZuuAI::Stop()
ManageLoan: Want to pay back
Company: Unnamed
balance: 130000
loan: 130000
max_balance: 40000
pay_back: 90000
new_loan: 40000
openttd: /home/leif/Spel/OpenTTD/noai/noai/src/ai/ai_threads.cpp:130: void AIThreadState::Suspend(int): Assertion `this->state == RUNNING' failed.
zsh: abort      ./openttd -a "Zuu AI" -g

The interesting part is that after Zuu::AI() my MangeLoan() gets called for company "Unnamed" which have the same balance as my player did have. Also anther interesting thing is that "ZuuAI::Start()" is not printed, which is printed in the very start of my Start()-function.

EDIT: Seeing that you plan to change the Start/Stop mechanism it might not be worth to look deeper into this.

Post by **TrueBrain** » 19 Mar 2007 22:49

Zuu wrote:(..)

* It would be nice if there was a distance-function for calculating the distance between two tiles that uses the same distance algorithm as the rest of OpenTTD. Motivation: When comparing towns to link together it is nice to use the same distance calculations as when the AI will earn profit for it's line.

(..)

Already planned. Will most likely be done this week

Let us know if your AI is worth showing

Post by **TrueBrain** » 19 Mar 2007 22:50

Zuu wrote:I've looked a bit into it now, and it seams like when the game quits the AI get's controll over the player. See this output: (notice, I loaned some extra money as player to show that it is the player that get messed with.)

(..)

The interesting part is that after Zuu::AI() my MangeLoan() gets called for company "Unnamed" which have the same balance as my player did have. Also anther interesting thing is that "ZuuAI::Start()" is not printed, which is printed in the very start of my Start()-function.

EDIT: Seeing that you plan to change the Start/Stop mechanism it might not be worth to look deeper into this.

It in fact is a known problem to us. When Stop() is issued, it is assumed that you stop directly, no Sleep(), no DoCommands like SetLoan. This only is impossible to do in any design of an AI, unless you check all the time if Stop() is called. So it is flawed by design. Don't worry about it too much, we will fix it up before merging to the trunk

Zuu · Post by **Zuu** » 19 Mar 2007 23:21

TrueLight wrote:Let us know if your AI is worth showing

At some point yes, but currently it can only manage it's loan and I'm working on search for town-pairs with high population and a reasonable distance to connect with busses. For actually connecting the towns I think I need to wait for some more stuff to be implemented. Such as more information about if a tile contain a house or not, slope etc.

I really like the nice work with the noai-branch.

Something that I think would be good for debugging, which you might or not might have though of is to make it possible for an AI to place signs on the map.

Post by **TrueBrain** » 19 Mar 2007 23:45

Zuu wrote:(..)

* It would be nice if the Squirrel standard library "math" could be initialized so that AIs can use the abs-functions etc. See the documentation.

(..)

abs() is added now in latest SVN. The reason why we don't load up the math library of SQ, is that we have some hooks and patches to make everything safe over all the targets we support. We can't be sure the default functions of SQ are safe, and therefor they are disabled. So if you have such functions you would like to be added, feel free to post them here, or ask me (TrueBrain on IRC, in #openttd, OFTC.net) to add them.

[Edit TrueLight] Distances functions are now added too

Ailure · Post by **Ailure** » 20 Mar 2007 15:57

Sorry if I seem impatient, but I just never been excited over a openTTD as now, especially since I have a few weeks of free time right now too.

But... what's currently left for the Win32 support? <<

XeryusTC · Post by **XeryusTC** » 20 Mar 2007 17:55

Win32 support isn't there ATM, but if you want I can stick a win32 executable to my post, it is a bit edited with some extra API though

.

Zuu · Post by **Zuu** » 21 Mar 2007 16:57

Bug: When building RoadDepots and RoadStations facing N-W or S-E the depot/station will face in the opposite direction as you tell it to do.

I've made a demo-AI that demonstrates this.

To run use this command:

Code: Select all

./openttd -a "Bus AI" -g ai/bus/scenario.scn

The main.nut.txt should be renamed to main.nut and placed in ai/bus. The scenario.scn-file should also be placed in ai/bus. (ai/bus have to be created)

Zuu · Post by **Zuu** » 21 Mar 2007 17:34

A fix to the bugs in BuildRoadStation and BuildRoadDepot

There is anther issue in the BuildRoad-function that is not solved in this patch. For the direction Y -1, the road is not built correctly, in the sense that it does not goes from center to center of the two tiles.

Edit: Removed a challenging language twist (in the post).

Edit2:
Attached an image that shows a test where I built roads like a star out from the tiles adjacent to the center and outwards. This test shows two things. The result is the same if you build independent of if the roads are built outwards or inwards. Secondly it shows that the S-W/S-E ends are built wrong.

For reference the code for generating the "star" is comes here:

Code: Select all

function BusAI::Road(center, direction)
{
    local tile = [null, null];
    tile[1] = this.map.GetTileIndex( this.map.GetTileX(center) + direction[0], this.map.GetTileY(center) + direction[1] );
    tile[0] = this.map.GetTileIndex( this.map.GetTileX(center) + 2*direction[0], this.map.GetTileY(center) + 2*direction[1] );

    this.road.BuildRoad(tile[0], tile[1]);
}

And in the ::Start() function:

Code: Select all

    local tile = this.map.GetTileIndex(25, 25);
    Road(tile, [1, 0]);
    Road(tile, [-1, 0]);
    Road(tile, [0, 1]);
    Road(tile, [0, -1]);

Post by **TrueBrain** » 23 Mar 2007 08:42

All problems reported above should be fixed now.

Ailure · Post by **Ailure** » 23 Mar 2007 09:03

XeryusTC wrote:Win32 support isn't there ATM, but if you want I can stick a win32 executable to my post, it is a bit edited with some extra API though .

Well, I wouldn't mind any extra undocumented features. I wouldn't use them though.

XeryusTC · Post by **XeryusTC** » 23 Mar 2007 10:53

Ailure wrote:
XeryusTC wrote:Win32 support isn't there ATM, but if you want I can stick a win32 executable to my post, it is a bit edited with some extra API though .
Well, I wouldn't mind any extra undocumented features. I wouldn't use them though.

Ok, here we go.

[Edit TrueLight] As we are almost finished with official Win32 support, we kindly ask you to wait just a few more days, or send those binaries per PM or IRC. [/Edit]

Ailure · Post by **Ailure** » 23 Mar 2007 12:30

Thank you. After downloading pthreadGC2.dll as well, I got it to work. I go and play around with noAI a bit and see what I come up with.

Zuu · Post by **Zuu** » 24 Mar 2007 12:03

Bug: "AIVehicle" class was not registered properly. Fix is attached.

Zuu · Post by **Zuu** » 24 Mar 2007 13:30

It seams like it is impossible to find an (road)engine for any cargo. I made a test-ai that for all cargos looks for an engine with reliability >= 0 for each cargo type.

Code: Select all

./openttd -a "Bug AI" -g

Output:

Code: Select all

BugAI::Start()
cargo with id=0 and name=PASS is a valid cargo
 - Did NOT found a valid engine for this cargo
cargo with id=1 and name=COAL is a valid cargo
 - Did NOT found a valid engine for this cargo
cargo with id=2 and name=MAIL is a valid cargo
 - Did NOT found a valid engine for this cargo
cargo with id=3 and name=OIL_ is a valid cargo
 - Did NOT found a valid engine for this cargo
cargo with id=4 and name=LVST is a valid cargo
 - Did NOT found a valid engine for this cargo
cargo with id=5 and name=GOOD is a valid cargo
 - Did NOT found a valid engine for this cargo
cargo with id=6 and name=GRAI is a valid cargo
 - Did NOT found a valid engine for this cargo
cargo with id=7 and name=WOOD is a valid cargo
 - Did NOT found a valid engine for this cargo
cargo with id=8 and name=IORE is a valid cargo
 - Did NOT found a valid engine for this cargo
cargo with id=9 and name=STEL is a valid cargo
 - Did NOT found a valid engine for this cargo
cargo with id=10 and name=VALU is a valid cargo
 - Did NOT found a valid engine for this cargo

Post by **TrueBrain** » 24 Mar 2007 15:09

Win32 support is now done. The API for making road AIs is done. Or anyway, the basics are. So, you should be able to start with your first AI. Of course important parts like pathfinders and simple tilelist filters are missing, which will be added in the next few weeks. But, the basics are there. So, if you like, download the upcoming binaries (should be there this week), and start building your AI. Report any problems here. Also if you have function requests feel free to post them. Just remind yourself that we might not agree, or already plan to do that. Have fun!

Post by **TrueBrain** » 24 Mar 2007 15:09

Zuu wrote:It seams like it is impossible to find an (road)engine for any cargo. I made a test-ai that for all cargos looks for an engine with reliability >= 0 for each cargo type.
(..)

Now fixed

Zuu · Post by **Zuu** » 24 Mar 2007 17:32

It would be nice if there where a way to get the direction of a road station. The reason why this would be nice is that I want to find the road tile from which busses/trucks enter/exits from the station. Reason for that is that it makes no sense to connect two station-tiles with road if one can not make sure their entrances are reachable from the new built connection.

Two possible functions:
?? GetDirection(TileIndex station)
or
TileIndex GetFrontTile(TileIndex station)

I think the later would be easier to use in my case. Both because it makes less code for me and because I don't want to do bit-magic if I can avoid it. It also makes sense to use the concept of front-tile both when building the station and when acquiring information about the station.

Zuu · Post by **Zuu** » 25 Mar 2007 19:53

AIOrder-bug.

Orders are not added properly.

AI + scenario that demonstrates the bug are attached.

Code: Select all

./openttd -a "Bus AI" -g ai/bus/scenario.scn

Post by **TrueBrain** » 25 Mar 2007 20:55

Zuu wrote:AIOrder-bug.

Orders are not added properly.

AI + scenario that demonstrates the bug are attached.
Code: Select all
./openttd -a "Bus AI" -g ai/bus/scenario.scn

Fixed in SVN.

Transport Tycoon Forums

NoAI Branch - An AI Framework

Who is online