Page MenuHomeWildfire Games

Update AI difficulty level description.
ClosedPublic

Authored by Nescio on Jan 15 2019, 4:07 PM.

Details

Summary

When you are about to start a match and open the AI Configuration option, the description says the AI's resource stockpiling is −60% for Sandbox and +60% for Very Hard. Those values are actually incorrect. Presumably it was forgotten to update this description the last time the rate was tweaked.
The current values as defined in simulation/helpers/InitGame.js lines 44–48:

	// Sandbox, Very Easy, Easy, Medium, Hard, Very Hard
	// rate apply on resource stockpiling as gathering and trading
	// time apply on building, upgrading, packing, training and technologies
	let rate = [ 0.42, 0.56, 0.75, 1.00, 1.25, 1.56 ];
	let time = [ 1.40, 1.25, 1.10, 1.00, 1.00, 1.00 ];

This patch updates the description by listing the actual numbers for all difficulty levels.

Before:

After:

Test Plan

Nothing really.

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Nescio created this revision.Jan 15 2019, 4:07 PM
Owners added a subscriber: Restricted Owners Package.Jan 15 2019, 4:07 PM
Vulcan added a subscriber: Vulcan.Jan 15 2019, 4:10 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/differential/971/

Last time the values were changed in rP21256. In rP21257 the description was adapted.
So it was adapted. But it has been rounded. So not sure if we should use exact values, but I have nothing against it.
I don't find your new format that readable. I think the old had enough information.

Stan added a comment.Jan 15 2019, 8:14 PM

Yeah acronyms like this might be a bit annoying to translate and I'm not sure a neophyte would get it.

How about the following then?

The AI's resource gathering rate and trade gain depends on the difficulty level (Sandbox −58%, Very Easy −44%, Easy −25%, Medium normal, Hard +25%, Very Hard +56%); the easiest levels also have a slower production and building rate.

In D1749#70337, @Nescio wrote:

How about the following then?

The AI's resource gathering rate and trade gain depends on the difficulty level (Sandbox −58%, Very Easy −44%, Easy −25%, Medium normal, Hard +25%, Very Hard +56%); the easiest levels also have a slower production and building rate.

Sounds good for me.

Stan accepted this revision.Jan 16 2019, 7:14 AM
This revision is now accepted and ready to land.Jan 16 2019, 7:14 AM

Not to be annoying but wouldn't it be easier to say "from -X to +X, and "Normal" has no bonuses"?

Nescio updated this revision to Diff 7355.Jan 16 2019, 11:33 AM

Updated. This is how it looks in game:

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/differential/976/

Not to be annoying but wouldn't it be easier to say "from -X to +X, and "Normal" has no bonuses"?

I'm ok with both versions.

In D1749#70368, @Nescio wrote:

Updated. This is how it looks in game:

"Medium normal" -> "Medium ± 0%" (I had to read the text multiple times, bc I thought normal als refers to a difficulty level)

Nescio updated this revision to Diff 7356.Jan 16 2019, 12:08 PM

Updated: “Medium ±0%”.

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/differential/977/

Not to be annoying but wouldn't it be easier to say "from -X to +X

The purpose of user-facing text is to convey information to the player. Listing the values for all difficulty levels is desirable, because currently the only way for players to figure out the numbers is by looking up the InitGame.js file, something which ordinary people won't do. Besides, new players are more likely to play Medium (default) or Easy (because of name) rather than Sandbox (boring) or Very Hard (daunting), therefore displaying only the numbers for the highest and lowest levels is not really the most useful.

Imarok accepted this revision.Jan 16 2019, 12:59 PM

If nobody objects, I'll commit that soon.

Yes not a very strong objection anyway, go ahead.

Stan accepted this revision.Jan 16 2019, 3:21 PM
smiley added a subscriber: smiley.Jan 16 2019, 3:34 PM

(One might say that the info is badly placed. The better place would have been the gamesetup config pop-up itself. But that’s out of scope.)

Imarok added inline comments.Jan 17 2019, 4:56 PM
binaries/data/mods/public/simulation/ai/petra/data.json
3 ↗(On Diff #7356)

; the easiest levels

Shouldn't it be . The easiest levels?

Nescio marked an inline comment as done.Jan 17 2019, 7:30 PM
Nescio added inline comments.
binaries/data/mods/public/simulation/ai/petra/data.json
3 ↗(On Diff #7356)

In this case

 and the easiest levels
; the easiest levels
. The easiest levels

are all correct; the interpunction is a matter of personal taste. It can be changed, if you like.

Imarok added inline comments.Jan 18 2019, 12:02 AM
binaries/data/mods/public/simulation/ai/petra/data.json
3 ↗(On Diff #7356)

the interpunction is a matter of personal taste. It can be changed, if you like.

I've done so.

This revision was automatically updated to reflect the committed changes.