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

Medical - Add fracture chance setting #7396

Merged
merged 4 commits into from
Feb 22, 2020
Merged

Medical - Add fracture chance setting #7396

merged 4 commits into from
Feb 22, 2020

Conversation

mharis001
Copy link
Member

When merged this pull request will:

@mharis001 mharis001 added the kind/enhancement Release Notes: **IMPROVED:** label Jan 4, 2020
@mharis001 mharis001 added this to the 3.13.1 milestone Jan 4, 2020
@PabstMirror
Copy link
Contributor

What if we made the setting control FRACTURE_DAMAGE_THRESHOLD
or have magnitude of damage somehow influence the random chance

&& {_woundDamage > FRACTURE_DAMAGE_THRESHOLD}
&& {random 1 < (linearConversion [FRACTURE_DAMAGE_THRESHOLD, 2*FRACTURE_DAMAGE_THRESHOLD, _woundDamage, 1, 1.5, true] * EGVAR(medical,fractureChance))}

@mharis001
Copy link
Member Author

Can change to that, would need to update the tooltip to make that clear. Maybe something like: "The probability of a fracture causing wound resulting in a fracture. This value is increased by the amount of damage, maxing out at a 50% increase to this probability at double the minimum fracture damage threshold."

@jonpas jonpas merged commit a7bd05d into master Feb 22, 2020
@jonpas jonpas deleted the fracture-chance branch February 22, 2020 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Release Notes: **IMPROVED:**
Projects
Development

Successfully merging this pull request may close these issues.

3 participants