[Gameplay] Ships less pierce attack, but can destroy other ships because of a multiplier
ClosedPublic
Actions

Authored by LetswaveaBook on Feb 25 2022, 11:25 PM.

Details

Reviewers

wowgetoffyourcellphone
marder

Group Reviewers

Balancing

Commits

rP26753: [Gameplay] Ships less pierce attack, but can destroy other ships because of a…

Summary

Currently garrisoned ships easily kill organic units that dare to get to close to them. A garrisoned ship is a killing machine like nothing else in the game.

This patch solves their absurd deadliness against organic units. Ships get 12 pierce attack and a 3x multiplier against other ships so they can sink enemy ships at comparable rate as in A24 and A25.

Test Plan

Wait for Vulcan.

Diff Detail

Repository

rP 0 A.D. Public Repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 19640
Build 47001: Vulcan Build	Jenkins
Build 47000: Vulcan Build (macOS)	Jenkins
Build 46999: Vulcan Build (Windows)	Jenkins

Event Timeline

LetswaveaBook created this revision.Feb 25 2022, 11:25 PM

Herald added a reviewer: Balancing. · View Herald TranscriptFeb 25 2022, 11:25 PM

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

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/6813/display/redirect

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/5721/display/redirect

Harbormaster completed remote builds in B19640: Diff 19758.Feb 25 2022, 11:49 PM

LetswaveaBook requested review of this revision.Feb 25 2022, 11:49 PM

Freagarach added a subscriber: Freagarach.Feb 27 2022, 6:27 AM

Freagarach added inline comments.

binaries/data/mods/public/simulation/templates/template_unit_ship_bireme.xml
13	`BonusShip`? ;)

I tested the patch and the values does feel nice does feel nice to play, so I would accept this.
But maybe there should be art changes (The big bolt shooter projectiles don't fit anymore with this)

marder mentioned this in D4510: [gameplay] increase attack of defensive structures slightly.Feb 27 2022, 8:22 AM

requesting changes as the art/ the projectiles needs to be changed (imo)

This revision now requires changes to proceed.Mar 11 2022, 5:46 PM

@marder what projectiles would you see instead?

since the pierce attack is now close to those of arrows: arrows.

Just doesn't seem right to me to use huge bolts, that only do very little damage.

Conceptual agree from me.

I agree there this patch has indeed its limitations, but my aim was not to create something that was perfect.

The aim is to create something that is better than the current situation. In A25 any garrisoned ship that passed by unleashes hell with the 35 pierce attack.

If my change request are the only thing blocking this, then ignore them / just keep it in mind as a future TODO.
I do agree with the general idea of the patch

@marder We can play with the projectiles later. :) That's an art thing.'

I agree with the intention of this patch. Ships should primarily counter other ships. Perhaps this bonus can apply against Docks as well, since that is the Ship-building structure? I'll accept as-is and you can add the Dock bonus if you think it's appropriate. Fix @Freagarach 's inline comment first. ;)

I don't think it is particular good to also have the bonus vs docks as well

This revision is now accepted and ready to land.Mar 22 2022, 10:42 AM

This revision was landed with ongoing or failed builds.Apr 3 2022, 11:24 AM

Closed by commit rP26753: [Gameplay] Ships less pierce attack, but can destroy other ships because of a… (authored by Silier). · Explain Why

This revision was automatically updated to reflect the committed changes.

Silier added a commit: rP26753: [Gameplay] Ships less pierce attack, but can destroy other ships because of a….

Owners added a subscriber: Restricted Owners Package.Apr 3 2022, 11:24 AM