widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #10491
Re: [Merge] lp:~widelands-dev/widelands/ai-post-b19-2 into lp:widelands
Gun,
thanks for review!
Savegame compatibility is quite tricky. I will look at this but I believe that proper strategy is just to ignore previous AI data (from older packet) and initialize AI data from scratch.
I also dont like addressing buildings by name, and when we discuss AI changes on forum, most players gives advices specific for buildings&tribes. I am trying to avoid this as much as possible.
I can introduce bunch of new ai_hints like "is_bakery", though opposite would be better (per each tribe):
bakery:"barbarian_bakery"
woodcutter:"barbarian_lumberjack"
For now AI expects on some spaces in code that there is exactly one building of the type. But entries in "ai_hints" are OK, and AI will make sure the building is identified, and just one.
I will start with save/load stuff....
--
https://code.launchpad.net/~widelands-dev/widelands/ai-post-b19-2/+merge/325796
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai-post-b19-2 into lp:widelands.
Follow ups
References