vitotai / BrewManiacEsp8266

BrewManiac on ESP8266. Only ESP8266 needed.
158 stars 71 forks source link

PID Autotune wrong Setpoint #106

Closed Malord958 closed 2 years ago

Malord958 commented 2 years ago

In manual mode, starting autotune changes the target temperature to the current temperature. (setpoint=CurrentTemperature instead of setpoint=SettingTemperature). Tested with version 0.5.1 (bin file). maybe the cause of issue #93? behavior is the same photo_2022-04-06_21-45-11 photo_2022-04-06_21-45-18 .

vitotai commented 2 years ago

The target temperature of autotune is the temperature reading when you start, not the setting temperature.

Malord958 commented 2 years ago

OK sorry. The screenshots in the documentation made me think there was a issue. image https://github.com/vitotai/BrewManiacEsp8266/blob/master/docs/UserManual/3.ManualModeAndAutoTune.md