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

Run vitals loop during cardiac arrest #7013

Closed
wants to merge 1 commit into from
Closed

Conversation

kymckay
Copy link
Member

@kymckay kymckay commented May 21, 2019

The bleedout condition will never occur currently since bleeding stops during cardiac arrest unless CPR is administered and the vitals loop doesn't even run so the condition is never checked.

This fixes that by running the loop during cardiac arrest and also allows bleeding to continue if heart rate is 0, just at a fixed reduced rate.

Additionally this fixes #6530

As far as I can see we don't actually need to run the vitals loop during
cardiac arrest.

It also looks like the bleedout condition will never occur currently
(bleeding stops during cardiac arrest unless CPR is administered). I'll
fix this in a separate PR.
@kymckay kymckay added kind/bug-fix Release Notes: **FIXED:** area/focus-feature labels May 21, 2019
@kymckay kymckay added this to the Medical Rewrite milestone May 21, 2019
@kymckay
Copy link
Member Author

kymckay commented May 24, 2019

I think I want to fix this by having the loop running and then making bleeding continue to occur but at quarter speed or something if heart rate is 0.

Need to investigate how CPR can continue to work if the loop is still running (either having it act as a heart rate modifier or just skip heart rate updates if in cardiac arrest).

Also need to prevent the status change events from firing a bunch (e.g. low blood puts you into cardiac arrest and because the loop keeps running the event keeps firing).

Edit: The events firing repeatedly might not be an issue, currently the same thing already happens for other states

@kymckay kymckay changed the title Fix vitals loop upon cardiac arrest recovery Run vitals loop during cardiac arrest May 24, 2019
@alganthe alganthe mentioned this pull request Jun 9, 2019
28 tasks
@kymckay kymckay closed this Jun 19, 2019
@kymckay kymckay deleted the medical-vitals-fix branch June 19, 2019 08:55
@PabstMirror PabstMirror modified the milestones: Medical Rewrite, 3.13.0, 3.13.0-temp2 Dec 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/focus-feature kind/bug-fix Release Notes: **FIXED:**
Projects
Development

Successfully merging this pull request may close these issues.

time delta is incorrectly large when exiting cardiac arrest
2 participants