tanjera / infirmary-integrated

Medical device simulator for training healthcare professionals.
http://www.infirmary-integrated.com/
Other
40 stars 5 forks source link

Default QRS Width for Cardiac Rhythms #188

Closed tanjera closed 3 weeks ago

tanjera commented 1 year ago

Since implementing adjustable QRS and QTc times, some rhythms have unrealistic default times. Needs hard-coded default range (add to existing sets of defaults) specifically for QRS and/or QTc.

Most notable for reproducing: from default settings, select SVT... waveform presents w/ extremely narrow QRS, extended QTc, and has an unrealistic elongated isoelectric ST segment. Noted this issue only began after implementation of adjustable QRS/QTc and can be fixed by elongating the QRS (which auto-calculates a shorter QTc; can be achieved using the reverse process too).

Suggested fix:

tanjera commented 3 weeks ago

Redacting "Only apply default QRS/QTc if "use default ranges" is selected!" ... I don't think this is a good idea. It will cause unwanted behavior. People may not want vital signs to change, but may expect cardiac morphology to follow cardiac rhythm changes.

tanjera commented 3 weeks ago

"Ensure default QRS width is implemented from rhythm selection through to Tracing drawing" involves redrawing some rhythms geometrically (specifically plotted ECG rhythms. Making a new issue for this

tanjera commented 3 weeks ago

Implemented in 66511cb