widelands

update Ai statistic information

Bug #1829898 reported by hessenfarmer
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Won't Fix
Undecided
Unassigned

Bug Description

This is the bug to support the current AI statistics update with some files

Tags: ai

Related branches

lp:~widelands-dev/widelands/ai_productionsites_statistics
hessenfarmer: Approve
Diff: 91 lines (+30/-6)
2 files modified
src/logic/map_objects/tribes/productionsite.cc (+21/-3)
src/logic/map_objects/tribes/productionsite.h (+9/-3)
Revision history for this message
hessenfarmer (stephan-lutz) wrote :

Added a table to compare the output of the different formulas

tags: added: ai
Revision history for this message
hessenfarmer (stephan-lutz) wrote :

new table found my error

Revision history for this message
hessenfarmer (stephan-lutz) wrote :
  • Statistics.xlsx Edit (16.8 KiB, application/vnd.openxmlformats-officedocument.spreadsheetml.sheet)
Revision history for this message
TiborB (tiborb95) wrote :

Now I think it works as expected. I like the results, just maybe 300000 can be reduced to half or so..

Revision history for this message
hessenfarmer (stephan-lutz) wrote :

I would have chosen to have it doubled to smooth the results a bit. If you build a new building the economy needs some time to settle so I think a 10 minutes statisitcs period might be sufficient. (we laready had this as a comment somewhere in the code that 10 minutes might be good for the econkmy to settle.

hessenfarmer (stephan-lutz)
Changed in widelands:
status: New → Fix Committed
Revision history for this message
GunChleoc (gunchleoc) wrote :

Migrated to https://github.com/widelands/widelands/issues/3441

Changed in widelands:
status: Fix Committed → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.