Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix SACUs having less effective range due to low projectile lifetimes #5958

Merged
merged 2 commits into from Mar 7, 2024

Conversation

Basilisk3
Copy link
Contributor

@Basilisk3 Basilisk3 commented Mar 6, 2024

In the current branch, the projectile lifetime of three of the four SACUs is extremely low. This means, that at the edge of their effective ranges, their shots disappear in front of units before they can hit them. This is especially bad for the Seraphim SACU, as that one does not have an AOE upgrade (ignoring OC). The projectile lifetime for the Cybran SACU was excessive, so I reduced it to be more reasonable.

Changes:
ProjectileLifetimeUsesMultiplier: 1.15 --> 1.3 (Aeon; Seraphim; UEF)
ProjectileLifetimeUsesMultiplier: 2.4 --> 1.4 (Cybran)

A higher multiplier is required for the Cybran SACU, as its projectile's muzzle velocity is lower.

Only relevant with the respective range-increasing upgrades.

@Basilisk3 Basilisk3 marked this pull request as draft March 6, 2024 18:03
@Basilisk3 Basilisk3 marked this pull request as ready for review March 6, 2024 18:15
@Basilisk3
Copy link
Contributor Author

I believe in the case of the Aeon, Seraphim and UEF SACUs, this has been an issue for over 10 years, which is disheartening to think about😃

@Garanas
Copy link
Member

Garanas commented Mar 7, 2024

There's been a lot more findings the past few years that makes you wonder why it's been like that for ten years 😃

@Garanas Garanas added type: bug area: balance related to units balance labels Mar 7, 2024
@Garanas Garanas merged commit f1c4d5c into FAForever:deploy/fafdevelop Mar 7, 2024
@Basilisk3 Basilisk3 deleted the SACU_projectiles branch March 7, 2024 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: balance related to units balance type: bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants