search

luktinghin / simtiva

Free web app to simulate TCI/TIVA (target-controlled infusion/total intravenous anaesthesia)
http://simtiva.app
GNU General Public License v3.0
7 stars 2 forks source link

A User Requested labels on graph #8

Closed David8L closed 10 months ago

David8L commented 10 months ago

A user requested labels on graph. The x-axes with time in minutes, and y-axes with concentration. I tried adding the variable "conc_units" to the y-axes but it would not update from mcg to ng. Maybe it needs a chart.update() somewhere. What do you think, do you think we need labels? The isobologram has them, so it would be consistent.

netlify[bot] commented 10 months ago

Deploy Preview for simtiva ready!

Name Link
Latest commit 867c678cae65fec83da1bf3e5a2269212ff54c53
Latest deploy log https://app.netlify.com/sites/simtiva/deploys/656bac7cf1c15600080177a2
Deploy Preview https://deploy-preview-8--simtiva.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

luktinghin commented 10 months ago

Hi, thank you for your suggestion. I do agree it is more consistent and it will make the app appear more scientifically valid. If need to be changed I think it will require changes in JavaScript in maybe init_submit and tabswitch functions so the NG/MCG in conc unit can switch appropriately. Let me follow this up. Also noted the other changes in commit, will merge. Have a nice weekend