add thermalcontrol module by schtangel · Pull Request #40 · wirenboard/wb-rules-system

schtangel · 2023-05-19T11:09:57Z

Module for heating and cooling.
Now just deadband control.

Module for heating and cooling. Now just deadband control.

aadegtyarev · 2023-05-19T11:15:08Z

Для пользователя выглядит так:

Для интегратора/программиста так:

Changed titles and off status

* Changed order of controls * Modified logging (format and logging levels) * Changed config.modes from obj to array of strings

Add thermalcontrol module Co-authored-by: Nikolay Korotkiy <sikmir@disroot.org>

* added resistance to the absence of optional fields in the config * added checks for the required fields

Add Hysteresis field to virtual device

Add persistent storage for setpoint

aadegtyarev

Не работает. Инициализирую так по примеру, только поправил скобку в строке 102:

var libtherm = require("thermalcontrol");

config = ({
    devName: "climate_kitchen",
    devTitle: "Температура на кухне",
    modes: ["heating", "cooling"],
    temperatureSource: "wb-w1/somesensor",
    heatingChannel: "wb-gpio/EXT2_R3A1",
    coolingChannel: "wb-gpio/EXT2_R3A2",
    setpointrange: {
        "min": 6,
        "max": 35
    },
    hysteresis: 1,
    degug: true
});

var climate_in_kitchen = new libtherm.ThermalControlDevice(config);

Получаю ошибку в модуле:

Ошибка правила: TypeError: invalid base value
duk_hobject_props.c:2000
anon /usr/share/wb-rules-system/scripts/lib.js:813 preventsyield
anon thermalcontrol:312 construct preventsyield
F /etc/wb-rules/thermostat-test.js:18 preventsyield

Ещё надо статус OFF заменить на DISABLED — это ближе к выключателю ENABLED по смыслу и не будет потом путаницы с OFF_RELAY.

fixed example

Changed 'OFF' status to 'DISABLED'

not everywhere changed "OFF" to "DISABLED", now everywhere

* add default setpoint setting to config * remove persistent storage

aadegtyarev

Посмотрел — все ок. Оставил несколько комментариев, надо поправить и будет совсем хорошо.

Надо как можно больше параметров сделать необязательными с описанными в документации значениями: https://wirenboard.com/wiki/Wb-rules-modules#%D0%A1%D0%BE%D0%BA%D1%80%D0%B0%D1%89%D1%91%D0%BD%D0%BD%D0%B0%D1%8F_%D0%B7%D0%B0%D0%BF%D0%B8%D1%81%D1%8C

Давай поправим и вольём, пока я не ушёл в отпуск.

* Add units * Add checks for setpoint * Fixed cofusion with status * Fixed JSDOC example

* Add check of topics exist

KraPete

Надо описание структуры топиков создаваемого в MQTT устройства. Надо описать возможные значения этих топиков, всякие SUSPEND, DISABLED и т.п.
Может вместо SUSPEND писать IDLE? Встречал такое в холодильных установках
Кто должен сохранять значения setpoint, установленные пользователем? Сейчас при любом перезапуске, они будут сброшены, и будет установлено значение по умолчанию. Это может быть неожиданно.
Я бы всякие однообразные проверки наличия топиков вынес в отдельные функции, чтоб меньше повторений кода было.
Надо ли обрабатывать ситуации, когда исчезли устройства, выдающие температуру и реализующие управление?
Надо ли обрабатывать ошибку чтения у устройства, которое выдаёт температуру? Кажется, что стоит отключить нагрев и охлаждение.
Для обсуждения: возможно, при создании термостата не надо выкидывать исключение, если нужных устройств нет в MQTT. Возможно стоит сигнализировать в топиках ошибкой, а при появлении устройств начинать работать, согласно настройкам?
Для обсуждения: было бы удобно, если бы конфиг термостатов читался из файлика с диска. Такой конфиг можно было бы делать средствами homeui. Тогда пользователь мог бы настроить его сам без погружения в правила.

KraPete · 2023-05-30T04:54:14Z

+      },
+      setpoint: {
+        title: "Setpoint",
+        type: "value",


Может, range сделать? В интерфейсе будет красивее выглядеть

Да мы что-то так и эдак крутили это. А есть способ сделать шаг не в целый градус, а задаваемым?

есть meta/precision, не уверен, что wb-rules это умеют

Не умеют :(
Оставил пока value, всё-таки его по одному градусу двигать удобней.

Co-authored-by: KraPete <86825564+KraPete@users.noreply.github.com>

* add jsdoc for virtualdevice * minor fixes * remove bindcontrols * fix checks for heatingChannel/coolingChannel exist * changed order

schtangel · 2023-05-30T12:00:56Z

Надо описание структуры топиков создаваемого в MQTT устройства. Надо описать возможные значения этих топиков, всякие SUSPEND, DISABLED и т.п.

Может вместо SUSPEND писать IDLE? Встречал такое в холодильных установках

Кто должен сохранять значения setpoint, установленные пользователем? Сейчас при любом перезапуске, они будут сброшены, и будет установлено значение по умолчанию. Это может быть неожиданно.

Я бы всякие однообразные проверки наличия топиков вынес в отдельные функции, чтоб меньше повторений кода было.

Надо ли обрабатывать ситуации, когда исчезли устройства, выдающие температуру и реализующие управление?

Надо ли обрабатывать ошибку чтения у устройства, которое выдаёт температуру? Кажется, что стоит отключить нагрев и охлаждение.

Для обсуждения: возможно, при создании термостата не надо выкидывать исключение, если нужных устройств нет в MQTT. Возможно стоит сигнализировать в топиках ошибкой, а при появлении устройств начинать работать, согласно настройкам?

Для обсуждения: было бы удобно, если бы конфиг термостатов читался из файлика с диска. Такой конфиг можно было бы делать средствами homeui. Тогда пользователь мог бы настроить его сам без погружения в правила.

1 - добавил описание
2 - да, наверное так лучше. Тоже видел на регуляторах всяких.
3 - мне казалось, это силами самого wb-rules как-то делается? в любом случае, оно сейчас успешно переживает перезагрузку контроллера.
4 - да, надо. Посмотрю, какие из них можно нормально завернуть. С дебагом аналогично.
5 и 6 - стоит, сделаю вот прямо сейчас.
7 и 8 - надо будет обсудить с Сашей, да

add description of config.setpoint object

aadegtyarev · 2023-06-13T15:41:36Z

Дальше делает @aadegtyarev

aadegtyarev · 2024-02-28T07:36:17Z

Решили не делать. Вместо этого будет визуальный конфигуратор.

schtangel added 2 commits May 19, 2023 14:09

add thermalcontrol module

53972ff

Module for heating and cooling. Now just deadband control.

Update changelog

9c6f8e4

schtangel marked this pull request as ready for review May 19, 2023 11:25

Update thermalcontrol.js

70638f2

Changed titles and off status

aadegtyarev reviewed May 19, 2023

View reviewed changes

Comment thread modules/thermalcontrol.js Outdated

Comment thread modules/thermalcontrol.js Outdated

Comment thread modules/thermalcontrol.js Outdated

Comment thread modules/thermalcontrol.js Outdated

Comment thread modules/thermalcontrol.js Outdated

Update thermalcontrol.js

ea2463c

* Changed order of controls * Modified logging (format and logging levels) * Changed config.modes from obj to array of strings

sikmir reviewed May 23, 2023

View reviewed changes

Comment thread debian/changelog Outdated

schtangel and others added 4 commits May 23, 2023 12:21

Update debian/changelog

d39b93f

Add thermalcontrol module Co-authored-by: Nikolay Korotkiy <sikmir@disroot.org>

Update thermalcontrol.js

e992aad

* added resistance to the absence of optional fields in the config * added checks for the required fields

Update thermalcontrol.js

0424f6d

Add Hysteresis field to virtual device

Update thermalcontrol.js

688d583

Add persistent storage for setpoint

aadegtyarev reviewed May 24, 2023

View reviewed changes

Comment thread modules/thermalcontrol.js Outdated

schtangel added 4 commits May 25, 2023 10:20

Update thermalcontrol.js

0d18819

fixed example

Update thermalcontrol.js

61f22ff

Changed 'OFF' status to 'DISABLED'

Update thermalcontrol.js

cf3e4eb

not everywhere changed "OFF" to "DISABLED", now everywhere

Update thermalcontrol.js

f9453ea

* add default setpoint setting to config * remove persistent storage

aadegtyarev reviewed May 25, 2023

View reviewed changes

Comment thread modules/thermalcontrol.js

Comment thread modules/thermalcontrol.js Outdated

Comment thread modules/thermalcontrol.js

Comment thread modules/thermalcontrol.js

Comment thread modules/thermalcontrol.js

Comment thread modules/thermalcontrol.js Outdated

schtangel added 2 commits May 26, 2023 13:52

Update thermalcontrol.js

8cf7d2e

* Add units * Add checks for setpoint * Fixed cofusion with status * Fixed JSDOC example

Update thermalcontrol.js

ae6f170

* Add check of topics exist

webconn requested review from a team and KraPete May 29, 2023 08:24

KraPete requested changes May 30, 2023

View reviewed changes

schtangel and others added 4 commits May 30, 2023 10:21

Update modules/thermalcontrol.js

37f6353

Co-authored-by: KraPete <86825564+KraPete@users.noreply.github.com>

Update modules/thermalcontrol.js

466c3fd

Co-authored-by: KraPete <86825564+KraPete@users.noreply.github.com>

Update modules/thermalcontrol.js

791fa25

Co-authored-by: KraPete <86825564+KraPete@users.noreply.github.com>

Update thermalcontrol.js

090b865

* add jsdoc for virtualdevice * minor fixes * remove bindcontrols * fix checks for heatingChannel/coolingChannel exist * changed order

Update thermalcontrol.js

8ff13c8

add description of config.setpoint object

sikmir marked this pull request as draft February 4, 2026 13:07

Conversation

schtangel commented May 19, 2023

Uh oh!

aadegtyarev commented May 19, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

aadegtyarev left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

aadegtyarev left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

KraPete left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

KraPete May 30, 2023

Choose a reason for hiding this comment

Uh oh!

schtangel May 30, 2023

Choose a reason for hiding this comment

Uh oh!

KraPete May 30, 2023

Choose a reason for hiding this comment

Uh oh!

schtangel Jun 2, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

schtangel commented May 30, 2023

Uh oh!

aadegtyarev commented Jun 13, 2023

Uh oh!

aadegtyarev commented Feb 28, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

aadegtyarev commented May 19, 2023 •

edited

Loading

aadegtyarev left a comment •

edited

Loading

schtangel Jun 2, 2023 •

edited

Loading