← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~gunchleoc/widelands/bug-1454371 into lp:widelands

 

What I am bit afraid is that there will be too many messages (notes) to AI now (AI has to go over all buildings to identify the proper one), because the counter is gone - that itself increases the count of messages 10-fold per a building (specifically mines - as only these used to send&receive the messages), but also much more (types of) buildings will be sending messages now. So now there could be like 20-50-fold increase in number of messages.

For now you can restring the messages only to mines, and I (or whoever will work on AI) will enable them for relevant buildings if AI will be ready to make use of them.

Also I found that I was wrong about reseting the counter, it was reset to 0 after achieving the treshold and sending the message, of course it is still not the same as reseting it after successful production.
-- 
https://code.launchpad.net/~gunchleoc/widelands/bug-1454371/+merge/261375
Your team Widelands Developers is requested to review the proposed merge of lp:~gunchleoc/widelands/bug-1454371 into lp:widelands.


References