Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/mines-worldsavior into lp:widelands
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
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