Page MenuHomeWildfire Games

[gameplay] tweak gate cost
ClosedPublic

Authored by Nescio on Mar 28 2020, 10:31 AM.

Details

Reviewers
borg-
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Commits
rP23886: Update gate cost to be proportional to wall costs & not cost stone.
Summary

Building long wall segments cost stone, and upgrading them gates costs even more stone. In reality gate doors are not made of stone. This patch therefore changes the upgrade cost to wood instead.
Moreover, D2611/rP23713 made long, medium, short wall cost (36, 24, 12) proportional to their health (3000, 2000, 1000). Since gates are in between long and medium segments (2500 health), so should their total cost (30).
The <Cost> component of gates is necessary for repairs; the value is less than the combined costs of building a long wall segment and subsequently upgrading it to a gate, but this patch makes it proportional to its health.

Test Plan

Check for mistakes and omissions.

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.Mar 28 2020, 10:31 AM
Owners added a subscriber: Restricted Owners Package.Mar 28 2020, 10:31 AM
Nescio updated this revision to Diff 11596.Mar 28 2020, 10:34 AM
Nescio edited the summary of this revision. (Show Details)

forgot maur gate loot

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

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1939/display/redirect

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

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1940/display/redirect

@ValihrAnt and others, any opinions on this?

borg- added a subscriber: borg-.Jun 4 2020, 7:10 PM

I'm not sure at two points.

1 - I think palisades are ok with only wood cost.
2 - Three resources to gate (stone wall) seems confused. Wood seems unnecessary. Maybe 20stone/20metal ? 60 res anyway it seems expensive for me.

Obs: I think you should also look at the wooden walls of the Romans.

Nescio added a comment.Jun 4 2020, 9:35 PM

Thanks for the feedback!

Three resources to gate (stone wall) seems confused. Wood seems unnecessary. Maybe 20stone/20metal ? 60 res anyway it seems expensive for me.

I agree the old and current value of 60 is rather high. A few days ago D2611/rP23713, making wall costs proportional. Since gate health is in between long (cost 36) and medium (24) segments, gates ought to cost 30.

I think palisades are ok with only wood cost.

I'll change that too.

I think you should also look at the wooden walls of the Romans.

I'll have a look.

Nescio updated this revision to Diff 12132.Jun 4 2020, 9:56 PM
Nescio edited the summary of this revision. (Show Details)
  • Different values.

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

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2314/display/redirect

borg- added a comment.Jun 5 2020, 2:53 AM

It looks better now, but I think the reduction of palisades from 20 to 6 is very big. Maybe something like 10?

I would change the value of the defensive wall gate to 20 stone and 10 wood, better numbers to see.

The same for the upgrade, seems better 20 than 18.

Sorry about that, haha.

I would change the value of the defensive wall gate to 20 stone and 10 wood, better numbers to see.

The <Cost> is not visible, it's only used for repairing.

It looks better now, but I think the reduction of palisades from 20 to 6 is very big. Maybe something like 10?
The same for the upgrade, seems better 20 than 18.

I'll change those.

Nescio updated this revision to Diff 12139.Jun 5 2020, 10:17 AM
  • Tweak upgrade cost, per @borg-.

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

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2317/display/redirect

borg- accepted this revision.Jun 5 2020, 1:54 PM

Ok, i can agree with the values now.

This revision is now accepted and ready to land.Jun 5 2020, 1:54 PM
wraitii added a subscriber: wraitii.Jun 5 2020, 6:51 PM

Will commit in a few days also barring complaints.

I like the idea, but one possible concern with very cheap gates is that in multiplayer, players could be encouraged to make gates everywhere in the walls to help pathfinding.

Stan added a subscriber: Stan.Jun 19 2020, 8:36 AM

I like the idea, but one possible concern with very cheap gates is that in multiplayer, players could be encouraged to make gates everywhere in the walls to help pathfinding.

Do you have a way it could be fixed?

Do you have a way it could be fixed?

Keep the <Cost> and <Loot> as proposed by increase the <Upgrade/Gate/Cost>. The question is to which values exactly.

Still seems worth trying. This time I'll commit this n a few days :p

wraitii updated this revision to Diff 12898.Jul 25 2020, 9:13 AM

Rebased, ready to merge.

This revision was landed with ongoing or failed builds.Jul 25 2020, 9:17 AM
This revision was automatically updated to reflect the committed changes.

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

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2774/display/redirect