Latest Posts

Topic: Widelands tournament 2017: subscriptions started!

WorldSavior
Avatar
Joined: 2016-10-15, 04:10
Posts: 2091
OS: Linux
Version: Recent tournament version
Ranking
One Elder of Players
Location: Germany
Posted at: 2017-12-07, 15:14

GunChleoc wrote:

I have created a merge request, so we will get AppVeyor builds for those who want to test on Windows.

https://code.launchpad.net/~widelands-dev/widelands/bug-1662425-training-balancing/+merge/334893

There will be a post to the merge request with a link once the builds are available.

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
Avatar
Joined: 2013-10-07, 15:56
Posts: 3324
Ranking
One Elder of Players
Location: RenderedRect
Posted at: 2017-12-07, 17:08

Thanks, should be fixed now. I got confused with the scaling.


Busy indexing nil values

Top Quote
king_of_nowhere
Avatar
Topic Opener
Joined: 2014-09-15, 18:35
Posts: 1668
Ranking
One Elder of Players
Posted at: 2017-12-07, 18:11

WorldSavior wrote:

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.

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
Avatar
Joined: 2016-10-15, 04:10
Posts: 2091
OS: Linux
Version: Recent tournament version
Ranking
One Elder of Players
Location: Germany
Posted at: 2017-12-07, 18:47

Edited a lot:

GunChleoc wrote:

Thanks, should be fixed now.

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 face-smile.png

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"

I got confused with the scaling.

No, I did. Sorry...

king_of_nowhere wrote:

WorldSavior wrote:

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.

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?

Oh, sorry. I've never realized that the change is that huge.

Edited: 2017-12-07, 20:14

Wanted to save the world, then I got widetracked

Top Quote
king_of_nowhere
Avatar
Topic Opener
Joined: 2014-09-15, 18:35
Posts: 1668
Ranking
One Elder of Players
Posted at: 2017-12-07, 20:59

WorldSavior wrote: Oh, sorry. I've never realized that the change is that huge.

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
Avatar
Joined: 2013-10-07, 15:56
Posts: 3324
Ranking
One Elder of Players
Location: RenderedRect
Posted at: 2017-12-08, 09: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
Avatar
Joined: 2013-10-07, 15:56
Posts: 3324
Ranking
One Elder of Players
Location: RenderedRect
Posted at: 2017-12-08, 11: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