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

Across the board firerate nerfs #4120

Merged
merged 7 commits into from
Aug 9, 2023

Conversation

morrowwolf
Copy link
Member

@morrowwolf morrowwolf commented Aug 6, 2023

About the pull request

Increases firerate delay defines.

Explain why it's good for the game

As CM becomes less and less laggy weapons have been more and more able to fire consistently on delay. This has created an artificial increase in actual firerate of weapons and we are seeing the effects of that now.

Firerate may be reduced more. Likely some more nerfs coming as we move towards full auto integration.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑 Morrow
balance: Across the board firerate nerfs with the exceptions of: M37 shotgun, M240 Flamer, M42A Scope Rifle, XM88, Mateba, Desert Eagle
/:cl:

@github-actions github-actions bot added the Balance You need to be a professional veteran game maintainer to comprehend what is being done here. label Aug 6, 2023
@Nanu308
Copy link
Member

Nanu308 commented Aug 7, 2023

One of the prerequisites to full auto is a nerf in ROF. We'll experiment more as we go but tldr. approved subject to change as with anything.

@Nanu308 Nanu308 added the Balance Approved This PR has had its balance and gameplay-affecting aspects approved. Cry to the Head-maint about it. label Aug 7, 2023
@morrowwolf morrowwolf added the Do Not Merge If you merge this PR, I will annihilate you label Aug 7, 2023
@morrowwolf
Copy link
Member Author

Still playing with some numbers 🙏

This likely is going to have unintended effects and needs tested, only parts of .5 work properly
@morrowwolf morrowwolf removed the Do Not Merge If you merge this PR, I will annihilate you label Aug 9, 2023
@morrowwolf
Copy link
Member Author

Happy with this for now.

@harryob harryob added this pull request to the merge queue Aug 9, 2023
Merged via the queue into cmss13-devs:master with commit 20b21f7 Aug 9, 2023
25 checks passed
cm13-github added a commit that referenced this pull request Aug 9, 2023
@morrowwolf morrowwolf deleted the firerate-nerfs branch August 10, 2023 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Approved This PR has had its balance and gameplay-affecting aspects approved. Cry to the Head-maint about it. Balance You need to be a professional veteran game maintainer to comprehend what is being done here.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants