Code review comment for lp:~chuckw20/widelands/Add_new_barbarian_farmer_worker

On Mon, Apr 26, 2010 at 4:38 AM, SirVer <email address hidden> wrote:

> ooh, i forgot: the animations are much longer than before which reduces the
> throughput of the farm significantly and you will need much more farms, so
> the sleep time in between work should be adopted.
>
>

Barbarian farmer animation time has increased as follows:

   - planting was 1 sec + 7 sec increase = 8 sec now
   - harvesting was 12 sec + 2 sec increase = 14 sec now

for a total increase of 9 sec or 169%.

The current/original plant-sleep duration is 20 sec and the harvest-sleep
duration is 5 sec. I will reduce plant-sleep duration by 6 sec to 14 sec
and harvest-sleep duration by 1 sec to 4 sec. This yields a 7 sec
adjustment to mitigate the 9 sec increase and reduce the impact on
production. I feel any greater reduction in the sleep cycles would yield a
frenetic farmer. :)

So, can you live with an extra 2 sec of animation per field knowing that it
still takes 11.25 minutes for an individual field's life cycle?