ScratMan / HASmartThermostat

Smart Thermostat with PID controller for HomeAssistant

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

pid_i is still an active attribute

Kolia56 opened this issue · comments

Describe the bug
As of 2023.1 it is said that "The pid_p, pid_i, pid_d and pid_e attributes are now removed from the extra state attributes of the thermostat to avoid overloading the HA database"
I just upgraded to v2023.2.1 under HASO 2023.3.0

Looking at attributes of one the thermostat, pid_i is still reported. Is it expected?

Running HAOS 2023.3.0

hvac_modes: heat, off min_temp: 2 max_temp: 22 preset_modes: none, away, eco, boost, comfort, home, sleep, activity current_temperature: 18.8 temperature: 18 hvac_action: idle preset_mode: none away_temp: 10 eco_temp: 15 boost_temp: 20 comfort_temp: 19 home_temp: 21 sleep_temp: 17 activity_temp: 20 control_output: 5.7 kp: 4.405673165173575 ki: 0.024274417346673835 kd: 199.90135871366417 ke: 0 pid_mode: auto pid_i: 9.2 friendly_name: Salle de bains amis supported_features: 17

Just random notes:
The commit that made this change: 9ef3a26
The commit that introduced pid_i back even in non-debug mode: 73d2e5a

Hi, it's an error in the commit message, the pid_i remains always active otherwise the integral would be cleared upon reboot, which is not recommended.