-
Notifications
You must be signed in to change notification settings - Fork 747
Medical Treatment - Realistic pulse checking #7740
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
base: master
Are you sure you want to change the base?
Conversation
I have something in the pipeline that also might not be epileptic friendly. Should we introduce a setting in common for epileptic friendly UX? I think the flashing is a bit too much, love the idea though. What if the user has no watch in his inventory? What about custom watches? |
|
The flashing seems a bit much. |
|
Maybe instead of a dark blurred sphere use something like a small white dot increasing in size/pulsing |
|
Why not a heart icon pulsing? |
|
What about a sound effect instead of / in addition to pulsing? |
|
Useful. I think it's should be better if you'll change size little bit smaller and add some icons instead of blinking. |
|
I love the idea of adding mini-games like this - if medical were less about navigating menus and watching progress bars it'd be a lot more engaging. |
|
I want this... |
As far as I can tell, 4 things remain:
I'll commit what I think would be suitable for the first three cases. |
| displayName = CSTRING(Actions_CheckBloodPressure); | ||
| displayNameProgress = CSTRING(Check_Bloodpressure_Content); | ||
| allowedSelections[] = {"LeftArm", "RightArm", "LeftLeg", "RightLeg"}; | ||
| treatmentTime = 15; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
15 is too long imo. I think 6 would be more reasonable, given the estimates you can do after 6 seconds (I would also change the pulse checking time to 6 too when realistic pulse checking is disabled).
| private _fade = (linearConversion [-0.1, 0, CBA_missionTime - _nextPulse, 0, 0.85, true] max linearConversion [0.35, 0, CBA_missionTime - _lastPulse, 0, 0.75, true]) ^ 2; | ||
|
|
||
| if (EGVAR(common,epilepsyFriendlyMode)) then { | ||
| _fade = _fade min 0.2; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm unsure if this is enough to not trigger seizures.
Regardless, it would be better to implement an icon.
| allowedSelections[] = {"All"}; | ||
| condition = QUOTE(GVAR(advancedDiagnose) != 0); | ||
| treatmentTime = QUOTE([ARR_2(2.5,15)] select GVAR(enableRealisticPulseChecking)); | ||
| condition = QUOTE(GVAR(advancedDiagnose) != 0 && {!GVAR(enableRealisticPulseChecking) || {(_medic getSlotItemName TYPE_WATCH) isKindOf [ARR_2('ItemWatch',configFile >> 'CfgWeapons')]}}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to require a watch for basic pulse checking or just have the watch requirement for realistic pulse checking?
https://www.youtube.com/watch?v=kGFEYOfDQZo
Pulsing to show target's pulse (possible seizure warning?)
There is no way you are going to check someone's pulse in 2.5 seconds
With this you get their actual pulse and a time source
Also disables blood pressure checking for non-medics as this requires specialized gear that a basic rifleman is not going to be carrying.