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 - Change - Pain level affects stability #8098

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

Conversation

Bummeri
Copy link

@Bummeri Bummeri commented Feb 6, 2021

When merged this pull request will:

I think FullHealLocal is not an issue since pain is set to 0 in it:

_patient setVariable [VAR_PAIN, 0, true];

Setting uncon. by script ?:
https://github.com/acemod/ACE3/blob/8e7f9b6db53a279432a2372c04be311807d1193b/addons/medical/functions/fnc_setUnconscious.sqf

And pain supression should be considered in some way?
https://github.com/acemod/ACE3/blob/8e7f9b6db53a279432a2372c04be311807d1193b/addons/medical_vitals/functions/fnc_updatePainSuppress.sqf

PAK treatment prerequisite are effected:
https://github.com/acemod/ACE3/blob/171fbf1e9ec3ee593ee0fc9e5c04b1270db4d01f/addons/medical_treatment/ACE_Medical_Treatment_Actions.hpp

@Bummeri
Copy link
Author

Bummeri commented Feb 6, 2021

@jonpas is this way ok?

@@ -31,4 +31,7 @@ if (_bloodPressureL < 50 || {_bloodPressureH < 60}) exitWith { false };
private _heartRate = GET_HEART_RATE(_unit);
if (_heartRate < 40) exitWith { false };

private _painLevel = GET_PAIN(_unit);
Copy link
Contributor

Choose a reason for hiding this comment

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

If you want to account for medication, GET_PAIN_PERCEIVED(unit)

@jonpas jonpas added this to the 3.14.0 milestone Apr 20, 2021
@jonpas jonpas added the kind/enhancement Release Notes: **IMPROVED:** label Apr 20, 2021
@jonpas
Copy link
Member

jonpas commented Apr 20, 2021

Change it to a proper PR when you think it's ready.

@PabstMirror PabstMirror modified the milestones: 3.15.1, Ongoing May 24, 2022
@LinkIsGrim
Copy link
Contributor

LinkIsGrim commented Mar 26, 2024

I don't like this without a rework of pain simulation first (well, I don't like this at all). hasStableVitals is used for wake-up checks, so at the very least it should take medication into account. You shouldn't administer painkillers on unconscious patients, as it's a choking hazard, and our current morphine values bring down vitals too much for a patient that's below ~90% blood volume to remain stable.

You don't feel pain when you're asleep. If anything, pain wakes you up.

If the goal is just to affect PAKs (why?) then stable condition should be modified instead.

@johnb432
Copy link
Contributor

well, I don't like this at all

Same.

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
None yet
Development

Successfully merging this pull request may close these issues.

6 participants