Topic: Widelands tournament 2017: subscriptions started!
WorldSavior |
Posted at: 2017-12-07, 14:14
I've found 2 small mistakes there: For empire we should have "increase_per_level = 812," instead of "increase_per_level = 920," and for atlanteans the same, while the attack values of barbarians stay identical. The health values of barbarians are correct. Wanted to save the world, then I got widetracked Top Quote |
GunChleoc |
Posted at: 2017-12-07, 16:08
Thanks, should be fixed now. I got confused with the scaling. Busy indexing nil values Top Quote |
king_of_nowhere Topic Opener |
Posted at: 2017-12-07, 17:11
wait a moment, why 812? the proposal was to increase by 1.2 per level over the (current) value, and the current value is 800, so 920 should be correct. why are you saying 812? Top Quote |
WorldSavior |
Posted at: 2017-12-07, 17:47
Edited a lot:
You're welcome, but king_of_nowhere was right, 920 is correct and not 812. I was wrong... So for fixing, both "812" should be replaced by "920" and that's it. At least I didn't mix up the tribes There is still that commit message: "- Buffed Barbarian and Empire soldiers." That's not wrong, but a little bit misleading. If those commit messages are important, it should rather be like "Fixed this bug: https://bugs.launchpad.net/widelands/+bug/1662425"
No, I did. Sorry...
Oh, sorry. I've never realized that the change is that huge. Edited: 2017-12-07, 19:14
Wanted to save the world, then I got widetracked Top Quote |
king_of_nowhere Topic Opener |
Posted at: 2017-12-07, 19:59
it looks huge, but it's actually 10% at full promotion. it won't affect how many hits it takes to kill a soldier in most circumstances, it will merely require one more shield/armor to survive that fourth blow. Top Quote |
GunChleoc |
Posted at: 2017-12-08, 08:14
Values are fixed now. And don't worry about the commit message. There will be a different message when the branch gets merged. Busy indexing nil values Top Quote |
GunChleoc |
Posted at: 2017-12-08, 10:36
Let's continue the balancing discussion at https://wl.widelands.org/forum/topic/4155/ It's off-topic for the thread and will be impossible to find in the future. Busy indexing nil values Top Quote |