← Back to team overview

widelands-dev team mailing list archive

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


Review: Needs Fixing

After discussing this in the Forum the marble mine probability settings need to be changed back to 20%.
found another issue in the code of the mine program 
in lines following 

a penalty for depleting fields is defined. this leads to a failed production program without taking into account the chance when depleted. see line 

so the situation where 2 mined resources could be thrown away (3rd mining step failing seem to be much more frequent than I thought. This effect will be worse the bigger the radius is (worst for Atlanteans) So we need to consider this in the programs probably or we need to change the mining code
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/mines-worldsavior.