-
Notifications
You must be signed in to change notification settings - Fork 739
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 Rewrite - TODO #6913
Comments
For heal hitpoints, I suggest naming it along the lines of exactly what it does: "Bandages heal damage" |
I have not been able to reproduce the Morphine bug. Both morphine and epinephrine got removed. Test conditions @alganthe ? |
@TheMagnetar dedicated environment, shot myself with a remote controlled unit. |
Unconscious person can access medical menu and bandage themselves |
Heal hitpoints setting related to #6458 where some useful previous discussion has taken place. Apparently my opinion on the setting name hasn't changed since then 😆 |
I'm not sure if this is a bug, but when you got a patient. You will get the button for CPR, but this will immediately stop if the target isn't in cardiac arrest. For bandaging and stitching we got the button only when it's needed. Maybe do it here too.
These are different conditions. Maybe merge them to one. Because clicking the button and immediately stop the action isn't nice. |
Can we open the discuss about #5293 again? I mean atropine should really raise the hr |
If I rember correctly I was the one adding the possibility to do cpr if the target is not in cardiac arrest. It was by design to make sure people check whether or not the target needs cpr. My test group quickly found out that the cpr button is a reliable option to tell the status of a person. Maybe introducing an setting would be an option. If not I would vote to be able to do cpr even if the target is not in cardiac arrest. |
That would be nice |
I'd say make CPR always an option if patient isn't responsive (as you'd never accidentally do it to someone who is) and have it cause damage to the torso, then if you do it on someone who doesn't need it it's counterproductive - incentivising status checks first. |
You will never deal damage with CPR but some rip breaks if you do it too long. This is normal even for patients who are in the need to get CPR. In real life you can even do CPR on reponsive persons (Normally it comes with a lot of pain) |
That is why you will get hit in face first.
Adenosine and Atropine should be removed from game. There is no use for them. There is no poisoning implemented. |
By damage I was thinking along the lines of bruising. To be honest, back when I had more time, I wanted to overhaul how CPR works and just make it a continuous action until disengaged which would intermittently perform a status check for you. |
I suggest being able to set how long a unit remains unconscious for after being wounded. Kind of stupid for someone to be unconscious for 25 seconds. Never seen that in my life as a EMT. |
Should note as discussed in slack:
This is due to the updateHeartRate function in vitals and it's unclear if intentional (we really need to be commenting why code is doing what it's doing). ACE3/addons/medical_vitals/functions/fnc_updateHeartRate.sqf Lines 34 to 42 in 9c55c36
The first condition is false so the targetBP stays at 107 and the second condition is true so the HR keeps increasing. |
I think this is the implementation of #1733, although the HR change is significantly different with a HR of 220 being reached just before Class 4 Hemorrhage.
Although borderline settings bloat, making this a setting that users can define could be an easier alternative |
tag older design issue #3134 |
I think this issue is outdated and resolved. Any specific points of discussion we want to track should be opened as fresh issues for tracking. |
Needs Testing:
To Do (Unclaimed):
To Do (In Progress):
Think about:
Fixed:
delayUnconCaptive
The text was updated successfully, but these errors were encountered: