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: Increase reload time of USA Patriot missile assist weapons from 1000 to 2000 #1580

Merged
merged 1 commit into from
Apr 9, 2023

Conversation

xezon
Copy link
Collaborator

@xezon xezon commented Jan 21, 2023

This change has been reverted with #2422

This change increases the reload time of USA Patriot missile assist weapons from 1000 to 2000.

Original

See the below footage for reference. In 1.04, the assisting Patriot Battery reloads its second (assist) volley faster than the left Patriot. (Note: The assist weapon's DelayBetweenShots is set to 100 here for consistency.)

ASSIST.mp4

First assist volley times:

00:11.25 [355]
00:11.29 [359]
00:12.03 [363]
00:12.07 [367]

Second assist volley times:

00:13.15 [405]
00:13.19 [409]
00:13.23 [413]
00:13.27 [417]

Conclusion:

The time between the first and second assist volleys in the above demonstration is only 38 frames. Normally there are exactly 60 frames (2000ms) between continuous non-assist volleys. This change removes this discrepancy and nerfs the assist weapon of the EMP Patriot by increasing its reload from 1000ms to 2000ms.

Rationale

Patriot missile assist weapons no longer reload twice as fast as non-assisted Patriot missile weapons. This in turn prevents the Patriot to attack a new target after just 1000 ms after an assisted attack, which makes it consistent with non-assisted Patriot missile attacks. This likely is what player would naturally expect always.

@xezon xezon added the Minor Severity: Minor < Major < Critical < Blocker label Jan 21, 2023
@commy2 commy2 self-assigned this Jan 27, 2023
@commy2
Copy link
Collaborator

commy2 commented Jan 27, 2023

This should be thoroughly tested before merge.

@Stubbjax
Copy link
Collaborator

Isn't this change a nerf? I can hear echoes "This is bad because it nerfs SWG therefore we should not merge it".

@xezon

This comment was marked as outdated.

@xezon

This comment was marked as outdated.

@commy2

This comment was marked as outdated.

@xezon

This comment was marked as outdated.

@MTKing4

This comment was marked as outdated.

@xezon

This comment was marked as outdated.

@MTKing4

This comment was marked as outdated.

@Stubbjax

This comment was marked as outdated.

@xezon

This comment was marked as outdated.

@Stubbjax

This comment was marked as outdated.

@xezon

This comment was marked as outdated.

@Stubbjax

This comment was marked as outdated.

@Stubbjax
Copy link
Collaborator

I went and tested this anyway. This change does have an impact on the reload time when a Patriot switches assist targets after having fired.

See the below footage for reference. In 1.04, the assisting Patriot Battery clearly reloads its second (assist) volley faster than the left Patriot. (Note: The assist weapon's DelayBetweenShots is set to 100 here for consistency.)

ASSIST.mp4

First assist volley times:

00:11.25 [355]
00:11.29 [359]
00:12.03 [363]
00:12.07 [367]

Second assist volley times:

00:13.15 [405]
00:13.19 [409]
00:13.23 [413]
00:13.27 [417]

Conclusion:

The time between the first and second assist volleys in the above demonstration is only 38 frames. Normally there are exactly 60 frames (2000ms) between continuous non-assist volleys. This change removes this discrepancy and nerfs the assist weapon of the EMP Patriot by increasing its reload from 1000ms to 2000ms.

I can hear those echoes again.

@xezon xezon added Design Is a matter of game design Controversial Is controversial USA Affects USA faction Nerf Makes a thing less powerful labels Feb 25, 2023
@xezon xezon changed the title Fix: Streamline values for ClipReloadTime and AutoReloadWhenIdle in Patriot Missile Assist Weapons Change: Increase reload time of USA Patriot missile assist weapons from 1000 to 2000 Feb 25, 2023
@xezon
Copy link
Collaborator Author

xezon commented Feb 25, 2023

Thank you for investigation. Now everything is clear. I have updated change description and have split this change into two: this one for the nerfed reload time, and #1755 for buffed auto reload when idle.

@MTKing4
Copy link
Collaborator

MTKing4 commented Feb 26, 2023

Now that we know what this change does, we should close it :D

We have no real consensus to do thie change, Normal Patriots are bad and don't need a nerf, and SWG is quite weak and relies a lot on it's EMPs

@xezon
Copy link
Collaborator Author

xezon commented Feb 26, 2023

I think this should be merged, along with the buff #1755. The annoying thing about quirks like this is that it makes setups more complicated and more difficult to tweak accordingly because things do not work the way one would expect them to work.

@penfriendz
Copy link

If it makes it easier and you know the effects, I think it's okay. I bet no one would notice anyway.

@xezon xezon added Bug Something is not working right and removed Design Is a matter of game design labels Apr 9, 2023
@xezon xezon changed the title Change: Increase reload time of USA Patriot missile assist weapons from 1000 to 2000 Fix: Increase reload time of USA Patriot missile assist weapons from 1000 to 2000 Apr 9, 2023
@xezon
Copy link
Collaborator Author

xezon commented Apr 9, 2023

Looks good.

@xezon xezon merged commit 16d1e4b into main Apr 9, 2023
@xezon xezon deleted the patriot-clipreloadtime branch April 9, 2023 13:24
@commy2
Copy link
Collaborator

commy2 commented Apr 10, 2023

Why?

@MTKing4
Copy link
Collaborator

MTKing4 commented Apr 10, 2023

Indeed why

We expressed that this isn't a good change yet you have merged it anyway

@xezon
Copy link
Collaborator Author

xezon commented Apr 11, 2023

This is merged because this is the way it should be. Whether or not this is an ever so slight Nerf is inconsequential. Patriot Battery was already buffed with Auto Reload added and there is this #1412 which would Buff Patriot a thousand times more than this Nerf.

@MTKing4
Copy link
Collaborator

MTKing4 commented Apr 11, 2023

This is merged because this is the way it should be.

According to what basis? That is a poor reason to make such balance altering change which the majority have agreed not to merge

I don't see any convincing argument do this change, there's no consensus here, you can't just change something just so that it looks good to you, normal attack and assist don't need to have the same rate

@xezon
Copy link
Collaborator Author

xezon commented Apr 11, 2023

If we went with your logic, we would need to revert the following 2 bug fixes too: #294, #1334. This issue is literally the same thing, just a little less extreme than the other 2.

@MTKing4
Copy link
Collaborator

MTKing4 commented Apr 11, 2023

Those are not the same thing, those are double fire and are due to having two weapons iirc and that reads more like a bug, where this one shows some sign of intent to be this way, for being assisted, also we the majority agreed on it, this one we didn't, this is not my concern alone

@commy2
Copy link
Collaborator

commy2 commented Apr 12, 2023

It's not a bug. Assisting Patriot fires faster and aims at farther away targets. That's how they're supposed to snowball in groups.

@xezon
Copy link
Collaborator Author

xezon commented Apr 12, 2023

The fire rate of the assisting Patriot is actually bound to its master Patriot, 2000 ms. It only fires faster when switching from an old Master to a new Master, min 1000 ms. I consider this a bug, because it can effectively halve its fire rate by switching targets, and only then. In every other scenario the Battery will reload in 2000 ms, also when assisting.

@MTKing4
Copy link
Collaborator

MTKing4 commented Apr 12, 2023

Wait so assisting the same pat will have them shoot at 2000ms? Hmm

@xezon
Copy link
Collaborator Author

xezon commented Apr 12, 2023

Wait so assisting the same pat will have them shoot at 2000ms? Hmm

Yes. Because they wait for the Master Patriot to shoot. Then they shoot at the same time.

xezon added a commit that referenced this pull request Jun 12, 2024
xezon added a commit that referenced this pull request Jun 12, 2024
…ns from 1000 to 2000 (#1580)" (#2422)

This reverts commit 16d1e4b.
And adds documentation for the ClipReloadTime value of PatriotMissileAssistWeapon and variants of it.
@xezon xezon added the Invalid Invalid label Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is not working right Controversial Is controversial Invalid Invalid Minor Severity: Minor < Major < Critical < Blocker Nerf Makes a thing less powerful USA Affects USA faction
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants