-
Notifications
You must be signed in to change notification settings - Fork 5
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
Temperature spikes occurring frequently #251
Comments
Weird that they're all in a similar temp range... |
I wonder if it could be caused by older sensors? I think client could have sensors that are 2+ years old, hmm |
@adamdewey - This reminds me that we might want some kind of filter for this in notify? Should we ignore temperatures that are over or under certain limits? We don't want people getting alerts saying their fridge is 70 degrees we it's actually fine... |
Hey @jmbrunskill - yeah I agree, wouldn't want to alert for those fake spikes. I think that could be a good idea to implement even before hiding the spikes in CCA, Open mSupply etc as alerts would be highly visible. |
@jmbrunskill - do we need a separate issue in the Notify repo, or can we link multiple PRs from different repos to this issue? |
umm hold your horses... i don't think you should filter any temps? if its high it might be high for a legitimate reason.. i have no idea what the alert plans are but are they not going to get an alert only after having lets say 15-20 mins of high readings - rather than immediately? if that is the case then a spike will not affect anything... |
The current plan it to alert as soon as it goes high so someone can go and check why, e.g. if someone left the door open, or the power's off so they need to start a generator or something. |
hmm.. what is the temp considered "breach" here because temps can jump above 8 pretty quickly and then jump back down like from opening a fridge door to transfer stock. sometimes having too many alerts might make people desensitised. usually fridges have an alarm that beeps if it happened in the first instance and you would generally try and do something if you were already there... just weighing up instant notification vs notifying after x mins option anyway |
yeah good points Louisa... Maybe we need to think about it some more - i.e. how can we help them to prevent a breach happening in the first place? |
That last one is more worrying as it's a realistic temperature so harder to filter out |
top one is a spike the bottom one is not - can see the datapoints gradually going up and back down |
Ah good spotting LDP! |
For the client above spikes have been seen on 3 out of 19 sensors (so far) For another client they've been seen on 4 out of 12 sensors |
Temporary fix will be to mask high outliers (above 55 C) in both the dashboard and notification service. Full fix is likely to be to go with BLE packet advertising - added to backlog |
Experiment with finding BM Spikes:METHOD: LOL im gonna forget this ill write it:
(Notes re method: the sensors i use were all previously connected to my test tablet already at home so i am just disconnecting it from this tablet and installing it into the cca app on a new tablet for this test) RESULTS: OBSERVATIONS:
Have added the excel files for the rest of the records... |
I think I might have seen a temperature spike on the blue maestro app today wasn't fast enough to screenshot it though. Maybe I confused a humidity reading... |
hmm, you mean it showed and then you think they masked it after? Because the historical data should be accessible on the blue maestro app? |
Nah I just saw it on the home page of their app. Can try downloading logs... |
Ooh, that would be interesting if you did manage to catch one! |
@louisaw123 caught a spike using the new 0.5.7 version. Upon investigation it appears that CCA is occasionally taking the humidity reading instead of the temperature reading (😱) Spike noticed on dashboard at 27.7 °C: ![]() Spike seen on CCA at 27.7 °C: ![]() Sensor logs showing humidity reading of 27.7 °C: ![]() Export of sensor logs: Export of CCA logs: Debug logs for CCA: |
Captured a negative spike today, the spike doesn't exist on the sensor logs, only in CCA. Possibly has taken the dewpoint reading instead of temperature, although the logs don't line up exactly: ![]() E1185AAF693B-31-10-2024-10-11-51.csv |
Describe the bug
Temperature spikes occurring in fridge monitoring sensors frequently. Extremely high temperatures occurring that range 60 degrees and above. Spike temps recorded over 3 months:
Spike temperature is not accurate as other sensors are also kept near the affected sensor but show normal readings. Examples:
To Reproduce
Unknown
Expected behavior
Temperature should be reading accurately otherwise may cause inaccurate data analysis. It also makes it difficult for user data reliability particularly for those monitoring without a backup sensor.
Additional context
The issue is not isolated to a single site and has been noticed in other sites too. Manufacturer has said that they have not come across this issue on their end and suspect it might be something caused on our end - have been monitoring own sensors with CCA and tempo but have not come across any spikes yet to confirm this.
The text was updated successfully, but these errors were encountered: