Page MenuHomeWildfire Games

workshop repair auras
ClosedPublic

Authored by mimo on Nov 15 2017, 8:21 PM.

Details

Summary

It could be interesting to add a repair aura for workshop, both to compensate for the fact that mace have to build an extra building compared to other civs, and to give an incentive to capture workshop by civ without workshop.
Currently 2 siege units can be garrisoned inside.

Test Plan

Check if you agree on the principle, and if yes, on the values.

Event Timeline

mimo created this revision.Nov 15 2017, 8:21 PM
Owners added a subscriber: Restricted Owners Package.Nov 15 2017, 8:21 PM
elexis accepted this revision.Nov 15 2017, 8:44 PM
elexis added a subscriber: elexis.

I thought it was a ranged aura and would have suggested Repairable.RepairTimeRatio then.
But since it's a garrisoned one, sounds ok to me. (Maybe the repair one - if it's supported - would be interesting as well, possibly additionaly?)
400HP / 4HP/s = 100s seems ok too.

Patch should be complete, since war elephants do heal in elephant stables last time I checked.
Commit (I didn't test this, so care ) if the repair thing is disapproved.

binaries/data/mods/public/simulation/data/auras/structures/cart_super_dock_repair.json
8

and this (no further occurrences exist)

binaries/data/mods/public/simulation/data/auras/structures/workshop_repair.json
10

Delete this line (extra mile would be adding a warning for auras with icons in a suitable location I didn't find)

This revision is now accepted and ready to land.Nov 15 2017, 8:45 PM
Nescio added a subscriber: Nescio.Nov 15 2017, 8:47 PM

Principle, yes, I agree.
Values, no, not really. Temple heals 20 units at 3 health per second, barracks (with technology) 10 units at 1 health. So maybe allow workshop to heal 5 garrisoned units at, let's say 2 health?

In D1043#40961, @Nescio wrote:

Principle, yes, I agree.
Values, no, not really. Temple heals 20 units at 3 health per second, barracks (with technology) 10 units at 1 health. So maybe allow workshop to heal 5 garrisoned units at, let's say 2 health?

Last time I played temples healed too quickly. At least it seemed cheap to build them near the enemy and jump in and out.

mimo updated this revision to Diff 4205.Nov 15 2017, 9:17 PM

Updated following comments. I'm not really in favor of increasing the garrison capacity to 5 (the structure is supposed to be relatively small, may be 3 if people think that's needed).

mimo updated this revision to Diff 4206.Nov 15 2017, 9:20 PM

make the description in agreement with the value

Vulcan added a subscriber: Vulcan.Nov 15 2017, 10:00 PM

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

Updating workspaces...
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (307 tests)...................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (307 tests)...................................................................................................................................................................................................................................................................................................................OK!
Checking XML files...
Relax-NG validity error : Extra element props in interleave
/public/art/actors/structures/carthaginians/stable_elephant.xml:0: Relax-NG validity error : Element variant failed to validate content
/public/art/actors/structures/carthaginians/stable_elephant.xml:0: Relax-NG validity error : Element group has extra content: variant
Relax-NG validity error : Extra element group in interleave
/public/art/actors/structures/carthaginians/stable_elephant.xml:0: Relax-NG validity error : Element actor failed to validate content
Executing section Default...
Executing section Source...
Executing section JS...
Grugnas added a subscriber: Grugnas.EditedNov 15 2017, 10:24 PM

is this aura json file necessary?
standing at GarrisonHolder component, there is the BuffHeal entry which job is to regen health of garrisoned unit class already.
( while at it, i guess that 1hp / sec for elephants is kinda low ).

( same for ships in the super dock )

BuffHeal

That doesn't restrict the 'healing' to siege units does it?
In fact to me it sounds like a good reason to delete BuffHeal.
Auras also have the advantage that we can add tooltips.

mimo added a comment.Nov 15 2017, 10:46 PM

Futhermore, currently BuffHeal applies only to Healable units while mechanical ones (like ship and siege) are not Healable but Repairable. Of course, the BuffHeal code could be tweaked, but cleaner to use Auras for repairables.

In D1043#40987, @elexis wrote:

That doesn't restrict the 'healing' to siege units does it?

you are right.

<GarrisonHolder>
  <Max>5</Max>
  <EjectHealth>0.1</EjectHealth>
  <EjectClassesOnDestroy datatype="tokens">Unit</EjectClassesOnDestroy>
  <List datatype="tokens">Support Infantry Cavalry Ship</List>
  <BuffHeal>0</BuffHeal>
  <LoadingRange>2</LoadingRange>
</GarrisonHolder>

this piece of code is from the carthaginian super dock and, at this point, i wonder if ships garrisoned by elephants and sieges can garrison the super dock.
In any case, i can't think any valid reason from the practical point of view to let other than mechanical units garrison such buildings.
By preventing any unit other than ships in the dockyard and sieges in the workshop to garrison, would resolve the issue ( this approach perhaps may conflict with Chanakya mauryan hero aura which decreases research time of buildings in which he can't garrison even if there are other buildings not able to being garrisoned like docks, storehouses and farmsteads ).
on the other hand, having a dedicated aura and tooltip would be more modular but wonder if the healing feature would have better visibility if directly shown on the structure tooltip since it isn't a feature unlocked by technology as happens with barracks.
I don't know if those 2 different approaches are relevant from a performance point of view.

Executing section Default...
Executing section Source...
Executing section JS...
This revision was automatically updated to reflect the committed changes.

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

Updating workspaces...
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (307 tests)...................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (307 tests)...................................................................................................................................................................................................................................................................................................................OK!
Checking XML files...
Relax-NG validity error : Extra element props in interleave
/public/art/actors/structures/carthaginians/stable_elephant.xml:0: Relax-NG validity error : Element variant failed to validate content
/public/art/actors/structures/carthaginians/stable_elephant.xml:0: Relax-NG validity error : Element group has extra content: variant
Relax-NG validity error : Extra element group in interleave
/public/art/actors/structures/carthaginians/stable_elephant.xml:0: Relax-NG validity error : Element actor failed to validate content

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

Updating workspaces...
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (307 tests)...................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (307 tests)...................................................................................................................................................................................................................................................................................................................OK!
Checking XML files...
Relax-NG validity error : Extra element props in interleave
/public/art/actors/structures/carthaginians/stable_elephant.xml:0: Relax-NG validity error : Element variant failed to validate content
/public/art/actors/structures/carthaginians/stable_elephant.xml:0: Relax-NG validity error : Element group has extra content: variant
Relax-NG validity error : Extra element group in interleave
/public/art/actors/structures/carthaginians/stable_elephant.xml:0: Relax-NG validity error : Element actor failed to validate content