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

Buff XM88 #6375

Merged
merged 2 commits into from
Jul 15, 2024
Merged

Buff XM88 #6375

merged 2 commits into from
Jul 15, 2024

Conversation

Steelpoint
Copy link
Contributor

@Steelpoint Steelpoint commented Jun 4, 2024

About the pull request

This PR changes the XM88 to have comparable values as if the weapon had a barrel charger attached to it, this includes a buff of its damage to 104, and a decrease to its inherent accuracy and rate of fire.

Explain why it's good for the game

The removal of the barrel charger from the gameplay loop had the unintended side-affect of indirectly nerfing the XM88 in a significant manner. Prior to its removal the barrel charger was a "must-have" attachment for the weapon, as it brought its damage up to a level of greater lethality, wherein without it the weapon now struggles more so against hardened targets. Furthermore, the weapons higher skill requirement means that you will regularly miss shots on a target, thus the weapon having a higher damage upfront rewards you for landing your shots.

To account for this, this PR effects the change to presume the weapon is under the benefits and drawbacks of having a barrel charger attached to it. Meaning the gun has a higher damage per bullet but suffers a rate of fire decrease and a major decrease in its accuracy.

Functionally this is bringing the weapon back to its older state that was what the majority of the player base grew to expect the weapon to perform.

Due to the major nerfs to the extended barrel, the now ability to attach it to this weapon comports a almost non-existant boon to the weapon and thus is not a point of balance contention.

Furthermore, after doing some offline testing I can ascertain that the average time-to-kill on a stationary target in white-room conditions is almost virtually unchanged. For example, a Warrior can expect to be dead in 8 seconds from both the original and this new version. The difference really comes down to the weapon having slightly higher damage up-front if you do land your shot.

Testing Photographs and Procedure

Screenshots & Videos

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

Changelog

🆑
balance: The XM88 now deals higher damage per-bullet at the cost of a slightly lower rate of fire and accuracy. This is identical to as if it had a barrel charger attached to it.
balance: Per the above, the barrel charger is no longer compatible with the XM88.
/: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 Jun 4, 2024
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label Jun 12, 2024
@Steelpoint
Copy link
Contributor Author

Stale comment

@github-actions github-actions bot removed the Stale beg a maintainer to review your PR label Jun 18, 2024
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label Jun 26, 2024
Copy link
Contributor

@Drulikar Drulikar left a comment

Choose a reason for hiding this comment

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

Please also make it incompatible with the barrel charger then. They are generally removed from the loop except when found in maps.

@Drulikar Drulikar added Stale Exempt PR can't go stale and removed Stale beg a maintainer to review your PR labels Jul 2, 2024
@Drulikar Drulikar marked this pull request as draft July 2, 2024 06:54
@Steelpoint
Copy link
Contributor Author

Done. (Maybe we should just remove the BC but that's another PR/topic)

@Steelpoint Steelpoint marked this pull request as ready for review July 2, 2024 07:52
@Drulikar Drulikar added the Balance Approved This PR has had its balance and gameplay-affecting aspects approved. Cry to the Head-maint about it. label Jul 15, 2024
@Drulikar Drulikar added this pull request to the merge queue Jul 15, 2024
Merged via the queue into cmss13-devs:master with commit 026efa2 Jul 15, 2024
29 checks passed
cm13-github added a commit that referenced this pull request Jul 15, 2024
@Steelpoint Steelpoint deleted the XM88 branch July 16, 2024 03:43
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. Stale Exempt PR can't go stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants