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

2018-08-23 Thread hessenfarmer
I think we should discuss this in the forum. I see the point but I believe your 
proposed solution would result in unreliable stats in the case that not enough 
construction material is delivered to the shipyard. 
Furthermore I think we should not have this in this branch cause it got already 
much bigger than its title suggests. I really would like if somebody could 
review the changes and approve this one. Still hope for Worldsaviors opinion 
over the forum (he has no Launchpad account).
-- 
https://code.launchpad.net/~widelands-dev/widelands/mines-worldsavior/+merge/350716
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/mines-worldsavior.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2018-08-23 Thread Toni Förster
One question regarding the shipyards. It takes 16 steps to complete a ship. But 
20 to reach 100% productivity. That somehow doesn't feel right; having built an 
entire ship but only 80% productivity.

What about moving the animation to the work program and remove the return from 
there to have statistics calculated. Instead move the sleep to the ship program.

It's like in the real world: preparing parts in the workshop, packinge them 
move to the dock, assemble them. 

One step in building a ship would then gain 10% in productivity. 
-- 
https://code.launchpad.net/~widelands-dev/widelands/mines-worldsavior/+merge/350716
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/mines-worldsavior.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp