Open solderq35 opened 8 months ago
This issue may be more complicated than anticipated, as as it seems like most metrics other than accumulated_real
(and a few others, like the default steam and gas measurements) use float data type in SQL database.
Also, I noticed that on the charts' frontend, all values are shown to 2 decimal places (ints shown as x.00
). Since everything is already shown to 2 decimal places on the frontend anyways, I'm not sure the current code matches Lety's intent of not showing decimals to reduce clutter.
Not sure if there is an actual reason for some values being int data type in SQL or if it's a performance optimization.
This doesn't seem to be very high priority and has potential complications so I recommend waiting on further feedback from Lety (need to email her or set up meeting) before proceeding further.
Might need to change
accumulated_real
to float in SQL workbenchSome buildings like https://dashboard.sustainability.oregonstate.edu/#/building/46/2 seem to show a reading of 1 every day when they are actually decimal values being rounded to 1.