Page MenuHomeWildfire Games
Differential D1749

Update AI difficulty level description.
ClosedPublic
Actions

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

Details

Reviewers
Stan
Imarok
Commits
rP22056: Update AI difficulty level description.
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
Branch
/ps/trunk
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 6804
Build 11193: Vulcan BuildJenkins
Build 11192: arc lint + arc unit

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/

Harbormaster completed remote builds in B6797: Diff 7349.Jan 15 2019, 4:10 PM
Imarok added a comment.Jan 15 2019, 8:11 PM

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.

Nescio added a comment.Jan 15 2019, 8:37 PM

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.

Imarok added a comment.Jan 16 2019, 12:29 AM
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
wraitii added a subscriber: wraitii.Jan 16 2019, 9:38 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:

Vulcan added a comment.Jan 16 2019, 11:36 AM

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

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

Harbormaster completed remote builds in B6803: Diff 7355.Jan 16 2019, 11:36 AM
Imarok added a comment.Jan 16 2019, 11:39 AM
In D1749#70366, @wraitii wrote:

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%”.

Vulcan added a comment.Jan 16 2019, 12:10 PM

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

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

Harbormaster completed remote builds in B6804: Diff 7356.Jan 16 2019, 12:10 PM
Nescio added a comment.Jan 16 2019, 12:20 PM

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.

wraitii added a comment.Jan 16 2019, 1:05 PM

Yes not a very strong objection anyway, go ahead.

Stan accepted this revision.Jan 16 2019, 3:21 PM
lyv added a subscriber: lyv.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

; 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

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

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

I've done so.

Closed by commit rP22056: Update AI difficulty level description. (authored by Imarok). · Explain WhyJan 18 2019, 12:06 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
binaries/
data/
mods/
public/
simulation/
ai/
petra/
2 lines

Diff 7356

View Options

binaries/data/mods/public/simulation/ai/petra/data.json

Loading...
Wildfire Games · Phabricator