Skip to content

Missile Guidance - Remove damage changes for javelin#11173

Merged
PabstMirror merged 1 commit intoacemod:masterfrom
rekterakathom:javelin-damagenerf
Dec 18, 2025
Merged

Missile Guidance - Remove damage changes for javelin#11173
PabstMirror merged 1 commit intoacemod:masterfrom
rekterakathom:javelin-damagenerf

Conversation

@rekterakathom
Copy link
Copy Markdown
Contributor

When merged this pull request will:

  • Remove very high hit value from ACE_Javelin_FGM148

The hit = 1400 was added 9-10 years ago with little justification, and it is a massive increase compared to what vanilla has in the parent class M_Titan_AT, which has hit = 95. This causes all weapons that use this missile to deal massive damage, achieving one-hit kills on basically any vehicle being targeted.

Considering the submunition penetrator (which was not around when this was originally added), this missile deals some ~2000 points of direct damage (hull damage) and penetrates some 900mm of RHA steel (caliber = 60 for the penetrator, inherited from vanilla).

Quick summary from my testing:

Before change:

  • T-100 Varsuk, DIR mode: instant kill
  • T-100 Varsuk, TOP mode: instant kill
  • T-14 Armata, DIR mode: no damage due to ERA plates
  • T-14 Armata, TOP mode: instant kill

After change:

  • T-100 Varsuk, DIR mode: heavy damage
  • T-100 Varsuk, TOP mode: heavy damage / instant kill
  • T-14 Armata, DIR mode: no damage due to ERA plates
  • T-14 Armata, TOP mode: heavy damage

IMPORTANT

  • If the contribution affects the documentation, please include your changes in this pull request so the documentation will appear on the website.
  • Development Guidelines are read, understood and applied.
  • Title of this PR uses our standard template Component - Add|Fix|Improve|Change|Make|Remove {changes}.

Remove very old damage changes (predating the Tanks DLC)
@PabstMirror PabstMirror added this to the 3.20.2 milestone Dec 16, 2025
@PabstMirror PabstMirror added the kind/change Release Notes: **CHANGED:** label Dec 16, 2025
Copy link
Copy Markdown
Contributor

@PabstMirror PabstMirror left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@PabstMirror PabstMirror merged commit 187db7a into acemod:master Dec 18, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/change Release Notes: **CHANGED:**

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants