search

caenHV / caen_tools

Set of microservices for CAEN management (WebService, DeviceBackend, SystemCheck, Monitor)
0 stars 0 forks source link

Requests from Popov #26

Closed AliensWithdrawPeople closed 3 weeks ago

AliensWithdrawPeople commented 3 weeks ago

Для Даниила Иванова - в HVmonitor требуется: 1) ввести допуск на время превышения софтового предела по току (дать возможность самому затухнуть) 2) Если диагностирован пробой и принято решение снизить напряжение, cнижать с рабочего до множителя 0.8 (можно побыстрее). Проверить, есть ли аварийные токи; если да, то снижать до нуля, если аварийных токов нет - ждем 5 секунд и ворачиваем напряжение до штатного (тоже быстро), разблокируем МЧС и начинаем писать. 3) если в течении 1 минуты таких пробоев произошло 5 штук, то снижаем напряжение до 0.4, ждем 2-3 минуты, и пытаемся поднять напряжение. /aspopov1/

AliensWithdrawPeople commented 3 weeks ago

Попрбовал реализовать изменения.

Главная проблема, которую я вижу в логике это общий счётчик для down и reduce (снижение напряжения после пробоя). Ещё мне не нравится, что я превратил shared_parameters у HealthControl в свою кладовку для всякого хлама.

Также решение вынести всё, связанное с падением, в виде обычных функций в отдельный файл конкретно протива принципов ООП. Но мне уже некомфортно работать с овер 500loc файлом. Будем жить так пока.

AliensWithdrawPeople commented 3 weeks ago

Всё находится в ветке dev_issue2. Ещё кое-какие изменения внесу и создам PR в dev.

AliensWithdrawPeople commented 3 weeks ago

Ну я закончил то, что хотел сделать. Локально протестирую и буду PR создавать.

AliensWithdrawPeople commented 3 weeks ago

Попробовал на железе потестить. Исправил кучу ляпов, но потом соединение c сетью slcmd оборвалось (больше не даёт подключиться). Чудом успел обратно поднять прошлую версию. Завтра продолжу тесты.