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

Weighs misfire chances + sanitizes code #4120

Merged
merged 3 commits into from
Feb 23, 2025

Conversation

Erikafox
Copy link
Contributor

@Erikafox Erikafox commented Feb 1, 2025

About The Pull Request

weighs misfire bodyzone chances

Why It's Good For The Game

more mechanically interesting than #4119. more mechanically interesting than prior system

Changelog

🆑
balance: misfire targetting is now weighed towards the legs
/:cl:

@github-actions github-actions bot added the Code change Watch something violently break. label Feb 1, 2025
Copy link
Member

@thgvr thgvr left a comment

Choose a reason for hiding this comment

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

The odds on hitting anything non-leg need to be very, very low. 2.5% or lower ideally.

@Erikafox Erikafox requested a review from thgvr February 14, 2025 01:01
@FalloutFalcon FalloutFalcon added this pull request to the merge queue Feb 22, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 22, 2025
@FalloutFalcon FalloutFalcon added this pull request to the merge queue Feb 23, 2025
Merged via the queue into shiptest-ss13:master with commit 85f5ead Feb 23, 2025
16 checks passed
MrCat15352 pushed a commit to MrCat15352/MrCat that referenced this pull request Feb 25, 2025
## About The Pull Request

weighs misfire bodyzone chances

## Why It's Good For The Game

more mechanically interesting than shiptest-ss13#4119. more mechanically interesting
than prior system

## Changelog

:cl:
balance: misfire targetting is now weighed towards the legs
/:cl:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants