BrewPi / firmware

Brewing temperature control firmware for the BrewPi Spark (Particle Photon inside)
http://www.brewpi.com
GNU Affero General Public License v3.0
97 stars 55 forks source link

Beer temperature filter can oscillate when hot plugging a sensor #18

Closed elcojacobs closed 9 years ago

elcojacobs commented 9 years ago

To reproduce:

The cause of this bug: Temperature sensor filters could be used uninitialized due to a bug in the failedReadCounter: when a fridge or beer sensor was plugged in 1-60 seconds after start, the filter was used with uninitialized data and returned invalid temperature.

Fixed in: https://github.com/BrewPi/firmware/commit/3c19c1a672391f7f93f00571e644ea47abdf486a

elcojacobs commented 9 years ago

Should we hotfix this before next release?

m-mcgowan commented 9 years ago

Can you describe the problem in more detail? Releasing a hotfix depends on the nature of the fix. Once the fix is coded, push the fix to a feature branch and we can then decide what to do with it.

elcojacobs commented 9 years ago

A very high result, then a negative result, then very high again and so on. I suspect the filter is not initialized .

elcojacobs commented 9 years ago

Fixed in 0.2.9