Page MenuHomeWildfire Games

[gameplay] adjust some siege engine stats
ClosedPublic

Authored by Nescio on Tue, Jan 12, 10:33 PM.

Details

Summary

This patch is split of from D3319. It does the following:

  • changes ram and siege tower footprints to look better:

  • lowers ram cost by 50 wood and 50 metal to partially compensate for the fact they're much less effective than in A23, since they can no longer attack organic units;
  • standardizes siege engine resource loot to 20% of their costs (same ratio as structures);
  • sets experience loot to 50% of their health for the ram and siege tower;
  • sets experience loot to 100% of their health for the bolt-shooter (no change) and stone-thrower (a reduction of one-sixth);
  • because the kush ram uses the same actor as the pers ram, it gets +2 garrison capacity and +20% attack damage as well;
  • [EDIT] gives the fireraiser a reasonable cost, health, and loot.
Test Plan

Check for mistakes and omissions, agree with the proposed values.

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.Tue, Jan 12, 10:33 PM
Owners added a subscriber: Restricted Owners Package.Tue, Jan 12, 10:33 PM

Build is green

builderr-debug-macos.txt
ld: warning: text-based stub file /System/Library/Frameworks//CoreAudio.framework/CoreAudio.tbd and library file /System/Library/Frameworks//CoreAudio.framework/CoreAudio are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//AudioToolbox.framework/AudioToolbox.tbd and library file /System/Library/Frameworks//AudioToolbox.framework/AudioToolbox are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//ForceFeedback.framework/ForceFeedback.tbd and library file /System/Library/Frameworks//ForceFeedback.framework/ForceFeedback are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//CoreVideo.framework/CoreVideo.tbd and library file /System/Library/Frameworks//CoreVideo.framework/CoreVideo are out of sync. Falling back to library file for linking.
ld: warning: text-based stu

See https://jenkins.wildfiregames.com/job/macos-differential/2815/display/redirect for more details.

Nescio requested review of this revision.Tue, Jan 12, 10:39 PM
borg- added a comment.EditedWed, Jan 13, 3:04 AM

Cost reduction for ram is fair
Standardizing the loot is fair.
Shouldn't the xp loot of ram and siege_tower be bigger since in theory you need fight with the soldiers inside?
Fireraiser should be included?

Shouldn't the xp loot of ram and siege_tower be bigger since in theory you need fight with the soldiers inside?

Perhaps. I assume <xp> depends on how dangerous an unit is, e.g. ranged soldiers are much easier to kill than melee soldiers, but have the same <xp>, presumably because their long attack range makes them more effective when massed; the same applies to artillery.
I have no objections to giving rams and siege towers more <xp>, but it shouldn't be 100% of health. What value would you recommend for them?

Fireraiser should be included?

The fireraiser doesn't have a cost, so I don't think giving it a loot is appropiate.

Maybe something like 50%?
fireraise appears on the "survival" map, so it seems interesting to include.

Maybe something like 50%?

That would mean 200 for the ram and 250 for the siege tower. Fine by me, but in keep in mind that's enough for a basic soldier to promote to elite rank.

fireraise appears on the "survival" map, so it seems interesting to include.

It needs a cost then (resources and time). Any recommendations?

300 wood, 150 metal 30 secs?

The fireraiser also needs health and xp values.
By the way, the siege tower is quite costly (500 wood, 300 metal, 40 seconds); what do you think of giving it a bit more health (e.g. 500→600)?
And perhaps ram metal cost can be lowered a bit (e.g. 200→150)? A tower costs only 100 wood + 100 metal.

borg- added a comment.Wed, Jan 13, 2:16 PM

By the way, the siege tower is quite costly (500 wood, 300 metal, 40 seconds); what do you think of giving it a bit more health (e.g. 500→600)?

No, it is preferable to decrease cost.

And perhaps ram metal cost can be lowered a bit (e.g. 200→150)? A tower costs only 100 wood + 100 metal.

I can agree with the lower cost of metal for ram.

And what values would you recommend for the fireraiser's <xp> and health?

borg- added a comment.Wed, Jan 13, 2:27 PM

And what values would you recommend for the fireraiser's <xp> and health?

350 and 50%?

Nescio updated this revision to Diff 15258.Wed, Jan 13, 4:34 PM
Nescio edited the summary of this revision. (Show Details)
  • also lower ram metal cost by 50
  • raise ram and siege tower <xp> to 50% of health
  • give fireraiser a cost, health, and loot, per @borg-
borg- accepted this revision.Wed, Jan 13, 4:39 PM
This revision is now accepted and ready to land.Wed, Jan 13, 4:39 PM

Build is green

builderr-debug-macos.txt
ld: warning: text-based stub file /System/Library/Frameworks//CoreAudio.framework/CoreAudio.tbd and library file /System/Library/Frameworks//CoreAudio.framework/CoreAudio are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//AudioToolbox.framework/AudioToolbox.tbd and library file /System/Library/Frameworks//AudioToolbox.framework/AudioToolbox are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//ForceFeedback.framework/ForceFeedback.tbd and library file /System/Library/Frameworks//ForceFeedback.framework/ForceFeedback are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//CoreVideo.framework/CoreVideo.tbd and library file /System/Library/Frameworks//CoreVideo.framework/CoreVideo are out of sync. Falling back to library file for linking.
ld: warning: text-based stu

See https://jenkins.wildfiregames.com/job/macos-differential/2844/display/redirect for more details.

This revision was automatically updated to reflect the committed changes.