Page MenuHomeWildfire Games

Aura balancing, tweaking. Adding aura for scipio.
ClosedPublic

Authored by fatherbushido on May 22 2017, 6:14 PM.

Details

Summary

Some 'lobby players' had some requests about auras.
We could not listen all personal desires, we can't change design everyday, we must keep historic flavour, we must consider that there is another thing exepting multiplayer life (casual player, single player, lan player,...) and that perhaps those players won't play the game anymore in one or two years.
Nonetheless, I discussed about that a long time with Grugnas and Borg and try to find concensus in a constructive discussion.
That patch is a sugestion of balancing about those auras to make them a bit more decisive (but not 'OP').
There is also an added effect for alexander (cf d231).
There is also a tweak of roman hero (cf d345) based on historic version of the design docs (taking inspiration of https://trac.wildfiregames.com/wiki/Civ%3A_Romans_Republican?version=1#HEROES). So add the Scipio aura and rework the Marcellus one.

I don't want to impose my personal point of view and don't want to change those stats everyday, but it seems good to have that for the next alpha.

From a technical point of view, due to issue with arc, the files are not renamed and the eol:native is not set but it will be done when commiting.

Test Plan
  • balance
  • strings
  • use the Demo Units map to check that everything is ok

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

There are a very large number of changes, so older changes are hidden. Show Older Changes
This revision is now accepted and ready to land.May 23 2017, 10:49 PM
elexis added a subscriber: elexis.May 23 2017, 10:58 PM
elexis added inline comments.
binaries/data/mods/public/simulation/data/auras/units/heroes/athen_hero_pericles_2.json
8 ↗(On Diff #2133)

From cunobelin we know that healing is very effective and temples are about healing.

Add +2HP/sec garrisoned healrate to temples to the aura proposal above? Then it were useful in battle as well, in a different, even unique way (and more reasonable than building temples gratis in seconds)

Garrisoned temple healrate is currently 3 and that one tech adds 50% (apparently Health uses floats if I read the code correctly), so 4.5. (Maybe someone is up for nerfing that tech too, making it +1 instead of +50% maybe)

binaries/data/mods/public/simulation/data/auras/units/heroes/gaul_hero_brennus.json
9 ↗(On Diff #2146)

metal I guess

Just a last note: check that Alexander actually does affect the cc with his short 20m range. The center of the CC might be out of his range.

nice catch, I didn t take care of that.

fatherbushido added inline comments.May 23 2017, 11:13 PM
binaries/data/mods/public/simulation/data/auras/units/heroes/athen_hero_pericles_2.json
8 ↗(On Diff #2133)

uhm I get it. Though when I think Pericles I think Phidias so it s a need to keep something related to temple building. Adding that effect anf going to a lower cost bonus seems ok.

binaries/data/mods/public/simulation/data/auras/units/heroes/gaul_hero_brennus.json
9 ↗(On Diff #2146)

yes

Quick scan only, check for further typos

binaries/data/mods/public/simulation/data/auras/units/heroes/cart_hero_maharbal.json
11 ↗(On Diff #2152)

Make spear cavalry OP again?

binaries/data/mods/public/simulation/data/auras/units/heroes/mace_hero_alexander_2.json
10 ↗(On Diff #2152)

"ennemy"
"civic center" --> "civic centers"

binaries/data/mods/public/simulation/data/auras/units/heroes/ptol_hero_ptolemy_IV.json
8 ↗(On Diff #2152)

300 hp champion pikemen with advanced extra cheap mercenaries as support
Alpha 22: Make Ptolemies Great Again

binaries/data/mods/public/simulation/data/auras/units/heroes/rome_hero_maximus.json
10 ↗(On Diff #2152)

Nice one at last.

binaries/data/mods/public/simulation/data/auras/units/heroes/rome_hero_scipio.json
15 ↗(On Diff #2152)

Basically you switched the best aura from Marcellus to Scipio, IMO Marcellus should get this and Scipio the more exotic one.

binaries/data/mods/public/simulation/data/auras/units/heroes/spart_hero_brasidas.json
12 ↗(On Diff #2152)

Lovely champion skirmishers.

Grugnas added inline comments.May 24 2017, 10:54 AM
binaries/data/mods/public/simulation/data/auras/units/heroes/ptol_hero_ptolemy_IV.json
8 ↗(On Diff #2152)

if you test 20 pikemen with +40% hp vs 20 pikemen with +20% attack speed, the attack speed boosted unit will win. The difference between the 2 auras, is that the +20% attack speed aura affects ranged units aswell (their bonus is still low if compared to a +20% attack boost though).

binaries/data/mods/public/simulation/data/auras/units/heroes/spart_hero_brasidas.json
12 ↗(On Diff #2152)

Spartans have -10% population

I'll adress the relevant remarks.

binaries/data/mods/public/simulation/data/auras/units/heroes/cart_hero_maharbal.json
11 ↗(On Diff #2152)

Don't expect to get an answer.

binaries/data/mods/public/simulation/data/auras/units/heroes/mace_hero_alexander_2.json
10 ↗(On Diff #2152)

Thx

binaries/data/mods/public/simulation/data/auras/units/heroes/ptol_hero_ptolemy_IV.json
8 ↗(On Diff #2152)

(summary)

binaries/data/mods/public/simulation/data/auras/units/heroes/rome_hero_scipio.json
15 ↗(On Diff #2152)

(summary)

binaries/data/mods/public/simulation/data/auras/units/heroes/spart_hero_brasidas.json
12 ↗(On Diff #2152)

It was considered. I am still wondering.
(Else read the summary.)

Some tweaks with remarks since last update.

Ptol4 ranged.

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/1335/ for more details.

Build is green

Executing section Default...
Executing section Source...
Executing section JS...
Executing section Python...
Executing section Perl...

http://jw:8080/job/phabricator_lint/27/ for more details.

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/1338/ for more details.

Build is green

Executing section Default...
Executing section Source...
Executing section JS...
Executing section Python...
Executing section Perl...

http://jw:8080/job/phabricator_lint/30/ for more details.

borg- accepted this revision.May 24 2017, 10:48 PM

Buff Chanakya and nerf Cunobelin.

Executing section Default...
Executing section Source...
Executing section JS...
Executing section XML GUI...
Executing section Python...
Executing section Perl...

http://jw:8080/job/phabricator_lint/64/ for more details.

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Checking XML files...

http://jw:8080/job/phabricator/1385/ for more details.

Some of those numbers are concerningly high. I hope this won't change the "no walls, no wonders" rule to "no walls, no wonders, no heroes". Some undoubtedly good changes in there.

binaries/data/mods/public/simulation/data/auras/units/heroes/athen_hero_iphicrates_1.json
12 ↗(On Diff #2133)

✓ since its a formation aura, its not too much. So high risk, high reward currently. For future generations, time will tell if its OP.

binaries/data/mods/public/simulation/data/auras/units/heroes/athen_hero_pericles_2.json
8 ↗(On Diff #2133)

If we take +1 healrate as too much for cunobelin, these +2 might be too much too. Opinions? @Grugnas @borg-
Maybe 1.5. But then again units seem to be fully healed up in a really short amount of time already when garrisoning in a temple. So probably can stay at this rate too.
either way take that: ✓

binaries/data/mods/public/simulation/data/auras/units/heroes/athen_hero_themistocles_1.json
8 ↗(On Diff #2146)

11 ↗(On Diff #2253)
  • training units isn't really construction
  • should mention the batch speed bonus

When garrisoned in a ship, the ship is +50% faster and trains batches of units 50% faster?

binaries/data/mods/public/simulation/data/auras/units/heroes/athen_hero_themistocles_2.json
6 ↗(On Diff #2253)

-50% metal cost? Thats -25% total cost = -75 metal. That's quite a lot.
40% would be okay for me and borg- too (he is still awake in the lobby)

binaries/data/mods/public/simulation/data/auras/units/heroes/brit_hero_boudicca.json
6 ↗(On Diff #2253)

8 ↗(On Diff #2253)

meh

binaries/data/mods/public/simulation/data/auras/units/heroes/brit_hero_caratacos.json
12 ↗(On Diff #2253)

k

binaries/data/mods/public/simulation/data/auras/units/heroes/brit_hero_cunobelin.json
9 ↗(On Diff #2253)

ah, got something similar in D565, just even more. According to PhyZic and borg- 0.5 might be better. But it's hard to determine whether 0.5 is too drastic or just right.

(07:25:20) borg-: i think 0.5 is better
(07:25:27) elexis: maybe 0.6
(07:26:06) borg-: because is cav (fast, can kill womans) more life compared to chanakya, and more atack too

binaries/data/mods/public/simulation/data/auras/units/heroes/cart_hero_maharbal.json
11 ↗(On Diff #2152)

Does anyone know a good car wash where they serve chicken?

Since this aura wasn't famous in particular for being too strong, I don't think that adding 10 more percent will change tht drastically. On the other side, perhaps it was just an unexplored possibility before.

binaries/data/mods/public/simulation/data/auras/units/heroes/gaul_hero_brennus.json
3 ↗(On Diff #2253)

good

6 ↗(On Diff #2253)

Uh why? Brennus is a popular hero and people get good amounts of loot already. Make it 15 if you have to, no?

In https://trac.wildfiregames.com/ticket/4503 we intend(ed) to remove loot for women altogether, then we could increase further.

(07:34:43) borg-: I found it a lot, grugnas asked for this value
(07:35:09) borg-: i think 15 or 20 is ok
(07:35:43) borg-: or maybe add wood or stone or food

Wasn't it possible to change it to a relative bonus? If that question isn't trivial to answer, pick a number < 30 IMO

binaries/data/mods/public/simulation/data/auras/units/heroes/mace_hero_alexander_1.json
8 ↗(On Diff #2253)

related proposal should be abolished then

binaries/data/mods/public/simulation/data/auras/units/heroes/mace_hero_alexander_2.json
10 ↗(On Diff #2152)

It's a bit weird to affect the enemy and not the own troops. Might go for structure to make it more useful, might keep it at CC to make it a more characteristic feature.
Interesting aura either way.

binaries/data/mods/public/simulation/data/auras/units/heroes/mace_hero_demetrius.json
15 ↗(On Diff #2133)

group attack stats in the string.

+15% attack range, +10% crush attack and +1 armor for siege engines + some quick grep for capitalization

binaries/data/mods/public/simulation/data/auras/units/heroes/maur_hero_chanakya.json
11 ↗(On Diff #2253)

I'd rather see this guy running around healing some army (as effectively as cunobelin) as nigel87 proposed.

(That seems to be historically reasoned too, wasn't a pacifist / passivist apparently):
https://en.wikipedia.org/wiki/Chanakya
The army of Chanadragupta and Chanakya invaded Dhana Nanda's kingdom, but disbanded after facing a severe defeat

So if you agree with the change in D565, then you could just reduce this benefit to 20%, which is still quite considerably when thinking about those expensive techs.

binaries/data/mods/public/simulation/data/auras/units/heroes/pers_hero_darius.json
9 ↗(On Diff #2253)

Right, we didn't have enough trade bonuses yet. Why not add a resource trickle to find something original that doesn't stack dangerously?
Other than that idc

binaries/data/mods/public/simulation/data/auras/units/heroes/ptol_hero_ptolemy_IV.json
8 ↗(On Diff #2152)

heard this argument a lot that theyre slow and hence bad. But 50% seems way too much for champions.

binaries/data/mods/public/simulation/data/auras/units/heroes/ptol_hero_ptolemy_I_1.json
3 ↗(On Diff #2253)

ack

binaries/data/mods/public/simulation/data/auras/units/heroes/rome_hero_marcellus_2.json
15 ↗(On Diff #2253)

Combining the two auras, this hero got a buff even?

binaries/data/mods/public/simulation/data/auras/units/heroes/rome_hero_scipio.json
3 ↗(On Diff #2253)

Why is scipio limited but boudica and marcellus not?

binaries/data/mods/public/simulation/data/auras/units/heroes/sele_hero_antiochus_righteous.json
3 ↗(On Diff #2253)

k

binaries/data/mods/public/simulation/data/auras/units/heroes/spart_hero_leonidas.json
12 ↗(On Diff #2253)

k

fatherbushido planned changes to this revision.May 28 2017, 8:51 AM
fatherbushido added inline comments.
binaries/data/mods/public/simulation/data/auras/units/heroes/athen_hero_themistocles_2.json
6 ↗(On Diff #2253)

It seemed to much for me too (I was for -20% or -25%).

binaries/data/mods/public/simulation/data/auras/units/heroes/brit_hero_cunobelin.json
9 ↗(On Diff #2253)

ok
(I still don't understand why they compare with Chanakya :p)

binaries/data/mods/public/simulation/data/auras/units/heroes/gaul_hero_brennus.json
6 ↗(On Diff #2253)

Agree with you :) I just listened them without filtering enough :D
15 sounds good (as range btw)

binaries/data/mods/public/simulation/data/auras/units/heroes/mace_hero_alexander_1.json
8 ↗(On Diff #2253)

(sure see summary)

binaries/data/mods/public/simulation/data/auras/units/heroes/mace_hero_alexander_2.json
10 ↗(On Diff #2152)

My idea was exploring it for cc then see.

binaries/data/mods/public/simulation/data/auras/units/heroes/mace_hero_demetrius.json
15 ↗(On Diff #2133)

y

binaries/data/mods/public/simulation/data/auras/units/heroes/maur_hero_chanakya.json
11 ↗(On Diff #2253)

Yes 0.5 is huge.

binaries/data/mods/public/simulation/data/auras/units/heroes/pers_hero_darius.json
9 ↗(On Diff #2253)

Was for the removed:
<Tooltip>Hero Aura: "Merchant of the Empire." Boosts profitablity of trade during his lifetime (TBD).</Tooltip>

binaries/data/mods/public/simulation/data/auras/units/heroes/ptol_hero_ptolemy_IV.json
8 ↗(On Diff #2152)

agree

fatherbushido added inline comments.May 28 2017, 9:26 AM
binaries/data/mods/public/simulation/data/auras/units/heroes/athen_hero_themistocles_1.json
11 ↗(On Diff #2253)

ah yes train.
The other one is imo not descriptible (it's never described in any tooltip btw).

Adress last comments.

This revision is now accepted and ready to land.May 28 2017, 9:37 AM
Executing section Default...
Executing section Source...
Executing section JS...
Executing section XML GUI...
Executing section Python...
Executing section Perl...

http://jw:8080/job/phabricator_lint/69/ for more details.

Grugnas added inline comments.May 28 2017, 1:37 PM
binaries/data/mods/public/simulation/data/auras/units/heroes/brit_hero_cunobelin.json
9 ↗(On Diff #2253)

I still think that 1hp/sec is good because the problem was about randomly hitted targets by buildings. regaining 1 hp / sec in any units fight is like absorbing 1 hitpoint every 1 sec (not so far from +1 armor).

binaries/data/mods/public/simulation/data/auras/units/heroes/gaul_hero_brennus.json
6 ↗(On Diff #2269)

I disagree becauase with 30 metal the average metal amount yelt is between 4k and 7k. It is not op at all because this hero doesn't give any boost at all in terms of convenient attributes boost in order to win fights. Consider that most of times the player has to face heroes who boost attributes.
"high risk, high reward currently." cit.

binaries/data/mods/public/simulation/data/auras/units/heroes/ptol_hero_ptolemy_IV.json
8 ↗(On Diff #2152)

if the reasoning isn't clear enough, i invite you to read my previous comment.

binaries/data/mods/public/simulation/data/auras/units/heroes/rome_hero_scipio.json
3 ↗(On Diff #2253)

you forgot Vercingetorix who has basically same aura effect with 60 range.

Grugnas added inline comments.May 28 2017, 1:48 PM
binaries/data/mods/public/simulation/data/auras/units/heroes/athen_hero_pericles_2.json
8 ↗(On Diff #2133)

I'd rather increase +1 temples healing rate OUTSIDE the temple. In that way allies won't benefit from the aura effect.

Alternatively he could do a significant boost only to the temple where he is garrisoned in.

binaries/data/mods/public/simulation/data/auras/units/heroes/pers_hero_darius.json
9 ↗(On Diff #2269)

I thought we agreed on +10% (if not +15% foreseeing an overall trade nerf and a balancement with Ashoka bonus). Going for this hero means not training Cyrus and sacrifice the possibility to train champions on the run and +20% attack damage for cavalry.

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Checking XML files...

http://jw:8080/job/phabricator/1393/ for more details.

Last update. (Mv files, lineendings are not in the patch).

Executing section Default...
Executing section Source...
Executing section JS...
Executing section XML GUI...
Executing section Python...
Executing section Perl...

http://jw:8080/job/phabricator_lint/74/ for more details.

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Checking XML files...

http://jw:8080/job/phabricator/1398/ for more details.

Numbers seem about ok, nothing glaringly OP

This revision was automatically updated to reflect the committed changes.