widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/ai_productionsites_statistics into lp:widelands

To: mp+367613@xxxxxxxxxxxxxxxxxx
From: TiborB <tiborb95@xxxxxxxxx>
Date: Tue, 21 May 2019 15:44:31 -0000
In-reply-to: <155829938086.6632.17877699587168181307.launchpad@ackee.canonical.com>
Reply-to: mp+367613@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

I completely understand current code. If it was "attempts in last x minutes" - it would be fine for me. But 'last 20 attempts' - this I understand from coding perspective - but dont like it.

'official statistics' to me is like actual_statistics^2. 100% and 0% are right, but in between it is undervalued...

But I am willing to accept removal of this statistics and modify AI code and retrain it.... if there is consensus about it..
-- 
https://code.launchpad.net/~widelands-dev/widelands/ai_productionsites_statistics/+merge/367613
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_productionsites_statistics into lp:widelands.

References

[Merge] lp:~widelands-dev/widelands/ai_productionsites_statistics into lp:widelands
From: TiborB, 2019-05-19