← Back to team overview

widelands-dev team mailing list archive

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


I think entire get_stocklevel stuff can be simplified.
You just need to iterate over all production_hints (iterating over empty vector will not hurt), then iterate over all output (the same)
And now there is a question - can we apply get_stocklevel to a site that is neither supporting nor producing? We can put there as assert to make sure that stocklevel for at least one ware was calculated.... (f.e. it is not still std::max)
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/frisian_balancing_with_ai_hints into lp:widelands.