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

Reduces Viper and echis fire rate down to 4 #34893

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

keronshb
Copy link
Contributor

@keronshb keronshb commented Feb 5, 2025

About the PR

Viper rate 6 > 4
echis rate 5 > 4

Why / Balance

The current full auto fire rate is ridiculous for a pistol and can down a full armor wearing sec relatively fast. This gives sec a little bit more of a chance.

Media

Content.Client_PMh0Ta9SX2.mp4

Requirements

Breaking changes

Changelog

🆑

  • tweak: Vesper fire rate lowered
  • tweak: echis fire rate lowered

@github-actions github-actions bot added Changes: No C# Changes: Requires no C# knowledge to review or fix this item. S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/S Denotes a PR that changes 10-99 lines. labels Feb 5, 2025
@1TALERD
Copy link

1TALERD commented Feb 5, 2025

Why not set the rate of fire to 5, like the MK58? Why make it worse than a standard pistol

@keronshb
Copy link
Contributor Author

keronshb commented Feb 5, 2025

Why not set the rate of fire to 5, like the MK58? Why make it worse than a standard pistol

Reason being it and the Echis are the only two pistols that have an auto fire mode (both semi and auto fire) and we don't have a way (right now) to separate Semi Auto and Auto Fire firing speeds.

@keronshb keronshb changed the title Reduces Viper fire rate down to 4 Reduces Viper and echis fire rate down to 4 Feb 5, 2025
@FairlySadPanda
Copy link
Contributor

FairlySadPanda commented Feb 5, 2025

  1. This risks being a disordered set of simultaneous PRs and it feels like an issue - or better yet, a doc hub that doesn't require PRs to edit - would be the best way to coordinate on sec balance.
  2. This is yet more stuff that I think needs test server balance passes rather than allowing untested balance modifications ship to stable.

@keronshb
Copy link
Contributor Author

keronshb commented Feb 5, 2025

  1. This risks being a disordered set of simultaneous PRs and it feels like an issue - or better yet, a doc hub that doesn't require PRs to edit - would be the best way to coordinate on sec balance.

    1. This is yet more stuff that I think needs test server balance passes rather than allowing untested balance modifications ship to stable.

Honestly the real fix to this is to adjust fire rate independent of mode. Then this can stay at 6 but auto fire goes down to 4.

@Everturning
Copy link

why is there all of a sudden a bunch of PRs specifically for the benefit of security and the detriment of everyone else how fucking stupid does the average secoff player appear

@ArtisticRoomba ArtisticRoomba added P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. S: Needs Review Status: Requires additional reviews before being fully accepted T: Balance Change Type: Balance changes through direct value changes, or changes to mechanics that affect it DB: Beginner Friendly Difficulty: Great for beginners. Unambiguous in scope, and explains how to achieve the result. A: Combat Area: Combat features and changes, balancing, feel A: Roundflow/Antag Area: Roundflow - "What happens in the game", including antagonist roles and their capabilities and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Feb 5, 2025
@TurboTrackerss14
Copy link
Contributor

2. This is yet more stuff that I think needs test server balance passes rather than allowing untested balance modifications ship to stable.

We test on prod here OORAH

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: Combat Area: Combat features and changes, balancing, feel A: Roundflow/Antag Area: Roundflow - "What happens in the game", including antagonist roles and their capabilities Changes: No C# Changes: Requires no C# knowledge to review or fix this item. DB: Beginner Friendly Difficulty: Great for beginners. Unambiguous in scope, and explains how to achieve the result. P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. S: Needs Review Status: Requires additional reviews before being fully accepted size/S Denotes a PR that changes 10-99 lines. T: Balance Change Type: Balance changes through direct value changes, or changes to mechanics that affect it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants