Page MenuHomeWildfire Games

Buff Carthaginian naval shipyard capture regen rate
ClosedPublic

Authored by elexis on Jul 6 2017, 3:42 PM.

Details

Summary

When buildings give root territory, capturing them shouldn't be too easy (so that the buildings connected to that territory don't switch ownership too easily/quickly/often).
Every building with that property, besides the Carthaginian naval shipyard, therefore have a significant garrisoned capture regeneration rate (or disable capturing, like monuments).

Since the model of this building doesn't appear to have a lot of space for units to garrison (5), nor a lot of defensive capabilities, it should be still easier to capture than a civic center or fortress (20).

Therefore it seems reasonable to increase the capture rate inherited from general special structures and keep it below that of a civic center.

The british crannog is also 'super dock' which gives root territory, but it is a civic center and not really a shipyard, so that doesn't appear to need tuning.

Test Plan

Use the developer overlay (alt+d) and the "gift from the gods" cheat in a singleplayer match to build a naval dockyard, garrison it.
Then switch the perspective to the enemy, use the cheat again and try to capture it with 35 citizen soldiers or 25 champs and a hero.
Consider whether that capturing is quick enough to be rewarding enough to the attacker and whether it is slow enough to be reasonable to defend the building when having to walk a bit.

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

elexis created this revision.Jul 6 2017, 3:42 PM
Vulcan added a subscriber: Vulcan.Jul 6 2017, 4:30 PM

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/1700/ for more details.

Vulcan added a comment.Jul 6 2017, 4:31 PM
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/289/ for more details.

mimo added a comment.Jul 6 2017, 7:11 PM

changes are ok. I would nonetheless rather use absolute values here instead of the mul operator.

While capture points for a more expensive and lets say "hierarchically" more important than a dock indeed needs more time to be captured.
The dock is even hard to defend with ships because of its huge size and more units can try to capture it at same time because of it.
Perhaps 30 pierce armor is too low for a super dock considering that any dock has 35.

elexis updated this revision to Diff 2848.Jul 7 2017, 2:53 PM

Absolute numbers.
Armor consistency with the regular dock.

Vulcan added a comment.Jul 7 2017, 5:39 PM

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/1713/ for more details.

Vulcan added a comment.Jul 7 2017, 5:40 PM
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/299/ for more details.

This revision was automatically updated to reflect the committed changes.