Better detection of hunters, fishers etc. for the AI
Bug #1772845 reported by
GunChleoc
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
widelands |
Won't Fix
|
Low
|
Unassigned |
Bug Description
At the moment, the AI uses building inputs and outputs to detect whether a building is a hunter or fisher etc. This is brittle - adding "fur" as an output for the Frisian hunter broke the assumptions.
So, we need to parse the tribes' production and worker programs to provide the AI with accurate data about the buildings - let's call it "collects map resource".
Related branches
lp:~widelands-dev/widelands/frisian_balancing_with_ai_hints
- Klaus Halfmann: Approve (compile, regression tests, testplay, small review)
- TiborB: Approve
- hessenfarmer: Approve
-
Diff: 3614 lines (+516/-315)77 files modifieddata/campaigns/fri01.wmf/scripting/starting_conditions.lua (+2/-1)
data/campaigns/fri01.wmf/scripting/texts.lua (+2/-2)
data/campaigns/fri02.wmf/scripting/starting_conditions.lua (+3/-2)
data/scripting/win_conditions/collectors.lua (+2/-2)
data/tribes/buildings/militarysites/frisians/tower/init.lua (+4/-4)
data/tribes/buildings/productionsites/atlanteans/fishers_house/init.lua (+1/-0)
data/tribes/buildings/productionsites/atlanteans/hunters_house/init.lua (+1/-0)
data/tribes/buildings/productionsites/atlanteans/quarry/init.lua (+1/-1)
data/tribes/buildings/productionsites/atlanteans/well/init.lua (+1/-1)
data/tribes/buildings/productionsites/atlanteans/woodcutters_house/init.lua (+1/-1)
data/tribes/buildings/productionsites/barbarians/fishers_hut/init.lua (+1/-0)
data/tribes/buildings/productionsites/barbarians/hunters_hut/init.lua (+1/-0)
data/tribes/buildings/productionsites/barbarians/lumberjacks_hut/init.lua (+1/-1)
data/tribes/buildings/productionsites/barbarians/quarry/init.lua (+1/-1)
data/tribes/buildings/productionsites/barbarians/well/init.lua (+1/-1)
data/tribes/buildings/productionsites/empire/fishers_house/init.lua (+1/-0)
data/tribes/buildings/productionsites/empire/hunters_house/init.lua (+1/-0)
data/tribes/buildings/productionsites/empire/lumberjacks_house/init.lua (+1/-1)
data/tribes/buildings/productionsites/empire/quarry/init.lua (+1/-1)
data/tribes/buildings/productionsites/empire/well/init.lua (+1/-1)
data/tribes/buildings/productionsites/frisians/aqua_farm/init.lua (+2/-1)
data/tribes/buildings/productionsites/frisians/armor_smithy_large/init.lua (+25/-15)
data/tribes/buildings/productionsites/frisians/armor_smithy_small/init.lua (+18/-8)
data/tribes/buildings/productionsites/frisians/bakery/init.lua (+1/-2)
data/tribes/buildings/productionsites/frisians/barracks/init.lua (+2/-2)
data/tribes/buildings/productionsites/frisians/beekeepers_house/init.lua (+1/-0)
data/tribes/buildings/productionsites/frisians/berry_farm/init.lua (+1/-1)
data/tribes/buildings/productionsites/frisians/brewery/init.lua (+1/-1)
data/tribes/buildings/productionsites/frisians/brick_kiln/init.lua (+7/-7)
data/tribes/buildings/productionsites/frisians/charcoal_kiln/init.lua (+2/-3)
data/tribes/buildings/productionsites/frisians/clay_pit/helptexts.lua (+1/-1)
data/tribes/buildings/productionsites/frisians/clay_pit/init.lua (+5/-7)
data/tribes/buildings/productionsites/frisians/collectors_house/init.lua (+2/-1)
data/tribes/buildings/productionsites/frisians/drinking_hall/init.lua (+1/-1)
data/tribes/buildings/productionsites/frisians/farm/init.lua (+6/-6)
data/tribes/buildings/productionsites/frisians/fishers_house/init.lua (+1/-1)
data/tribes/buildings/productionsites/frisians/foresters_house/init.lua (+1/-1)
data/tribes/buildings/productionsites/frisians/furnace/init.lua (+1/-0)
data/tribes/buildings/productionsites/frisians/honey_bread_bakery/init.lua (+2/-2)
data/tribes/buildings/productionsites/frisians/hunters_house/init.lua (+9/-3)
data/tribes/buildings/productionsites/frisians/mead_brewery/init.lua (+2/-4)
data/tribes/buildings/productionsites/frisians/quarry/init.lua (+2/-2)
data/tribes/buildings/productionsites/frisians/recycling_center/init.lua (+5/-3)
data/tribes/buildings/productionsites/frisians/reed_farm/init.lua (+2/-3)
data/tribes/buildings/productionsites/frisians/reindeer_farm/init.lua (+5/-5)
data/tribes/buildings/productionsites/frisians/scouts_house/init.lua (+3/-3)
data/tribes/buildings/productionsites/frisians/shipyard/init.lua (+4/-4)
data/tribes/buildings/productionsites/frisians/tailors_shop/init.lua (+2/-3)
data/tribes/buildings/productionsites/frisians/weaving_mill/init.lua (+2/-2)
data/tribes/buildings/productionsites/frisians/well/init.lua (+6/-6)
data/tribes/buildings/productionsites/frisians/woodcutters_house/init.lua (+2/-2)
data/tribes/buildings/trainingsites/frisians/training_arena/init.lua (+15/-16)
data/tribes/buildings/trainingsites/frisians/training_camp/helptexts.lua (+1/-1)
data/tribes/buildings/trainingsites/frisians/training_camp/init.lua (+7/-7)
data/tribes/buildings/warehouses/frisians/port/init.lua (+2/-2)
data/tribes/frisians.lua (+3/-2)
data/tribes/init.lua (+2/-1)
data/tribes/scripting/starting_conditions/frisians/fortified_village.lua (+4/-3)
data/tribes/scripting/starting_conditions/frisians/headquarters.lua (+2/-1)
data/tribes/scripting/starting_conditions/frisians/trading_outpost.lua (+2/-1)
data/tribes/scripting/starting_conditions/frisians/village.lua (+2/-1)
data/tribes/wares/shovel/helptexts.lua (+1/-1)
data/tribes/wares/sword_broad/helptexts.lua (+2/-2)
data/tribes/wares/sword_broad/init.lua (+3/-3)
data/tribes/wares/sword_long/helptexts.lua (+1/-1)
data/tribes/workers/frisians/brickmaker/helptexts.lua (+1/-1)
data/tribes/workers/frisians/brickmaker/init.lua (+3/-19)
data/tribes/workers/frisians/claydigger/helptexts.lua (+4/-0)
data/tribes/workers/frisians/claydigger/init.lua (+42/-0)
data/tribes/workers/frisians/farmer/init.lua (+3/-3)
src/ai/ai_help_structs.cc (+1/-0)
src/ai/ai_help_structs.h (+4/-0)
src/ai/ai_hints.cc (+16/-24)
src/ai/ai_hints.h (+21/-31)
src/ai/defaultai.cc (+209/-73)
src/logic/map_objects/tribes/building.h (+2/-2)
src/logic/map_objects/tribes/productionsite.cc (+11/-0)
Changed in widelands: | |
status: | Fix Released → Won't Fix |
To post a comment you must log in.
But please keep in mind that we still need to distinguish:
well
fisher
hunter
woodcutter
farmers
quarry
and perhaps more....
They all collect map resources, but have different code paths in AI...