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

I need more bullets! #4774

Closed
wants to merge 1 commit into from
Closed

Conversation

Diegoflores31
Copy link
Contributor

@Diegoflores31 Diegoflores31 commented Oct 23, 2023

About the pull request ----

Weapons:
VP78:
Increases Firerate of VP78 by 2
Decreases penetration by 1 tier
Increases Shrapnel chance by 4 tiers (squash head lol)

Deagle
Increased damage ( new damage = 70 aprox)
removed variable damage ( most weapons dont use it anymore)

CO deagle
-decreased accuracy penalty
-Renamed Caliber to .50s ( super!!)
-damage increased to 98
-Penetration removed on High impact rounds
-Stun removed on AP rounds

Code:
-removed empty lines
-Merged .50 ammunition with Heavy bullets.
-Removed CO bullet parent will now be /super , not /superhighimpact
-deleted repeated arguments

Explain why it's good for the game

Deagle and heavy pistols in general currently feel in a bad shape . they are slow . have little to no ammo and outgunned by smaller calibers . this PR aims to make your portable bazooka a viable solution winout being overpowered.

regarding CO weapon:
Its been tested ingame and it deals about the same DMG than the mateba . In the future i plan to remove the stun from the mateba AP but that may be a task for later.

Testing Photographs and Procedure

Screenshots & Videos

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

Changelog

🆑
balance: Increased VP78 Firerate by 2 tiers , reduced AP by 1 tier
balance: Increased Deagle damage to 70.
balance: increased CO deagle damage to 98. adjusted Accuracy and penetration on alternative ammo
removal: removed stun from AP high impact rounds.
/: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 Oct 23, 2023
Copy link
Contributor

@nauticall nauticall left a comment

Choose a reason for hiding this comment

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

AP Deagle or Mateba rounds do not need stun removed from them. It defeats the purpose of their viability as BE weapons.

@Diegoflores31
Copy link
Contributor Author

Diegoflores31 commented Oct 23, 2023

AP Deagle or Mateba rounds do not need stun removed from them. It defeats the purpose of their viability as BE weapons.

I can assure you after the changes made they are very viable. One made as AP and the other a HE with a stun.

@nauticall
Copy link
Contributor

I can assure you after the changes made they are very viable. One made as AP and the other a HE with a stun.

Not going to debate this further over GitHub but I would prefer a better reassurance for this than "trust me bro". We can testmerge this if it comes down to it to check real-world usage, but as it stands, I am against removing the stun.

The Deagle changes, as miniscule-facing as they may seem, are a whitelist change that affects COs; as such I would at least like for you to discuss this with the Council in development chat. Then we can talk out some kind of compromise or negotiations. I'm willing to listen, personally.

@nauticall nauticall added the Do Not Merge If you merge this PR, I will annihilate you label Oct 23, 2023
@morrowwolf
Copy link
Member

At bare minimum it needs to continue stunning humanoids.

@Zonespace27
Copy link
Contributor

Zonespace27 commented Oct 23, 2023

Atomize this. You're rebalancing (quite majorly, I should add) three different guns of three different availabilities whose only relation is that they're all handguns.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Do Not Merge If you merge this PR, I will annihilate you
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants