#204 closed улучшение (сделано)
Задержка НД датчика температуры.
Сообщил: | andrei | Владелец: | san |
---|---|---|---|
Приоритет: | средний | Этап разработки: | 2-я очередь |
Ключевые слова: | Копия: | san, Art_M, alx |
Описание
Артем предлагает кроме задержки включения вентиляторов 90 с, сделать независимую задержку 10 с от момента пропадания показаний ДТ перед отображением НД на панели оператора.
"Включение вентилятора производится не через 90, а через ~110сек почему то... Видимо есть ещё какая то задержка.... Ну и нужно ещё, чтобы бы н/д сразу отображалось (допустим через 10сек) , а вентилятор уже позже включался через 90сек... Иначе невозможно понять сразу, рабочая проводка или снова отпала, пока не пройдут эти почти две минуты"
История изменений (9)
следующий: 3 comment:2 by , 6 лет ago
Владелец: | изменён с | на
---|---|
Состояние: | new → assigned |
Оказывается у нас на время задержки выводится несуществующая температура. Кажется мне, неправильно это.
Это следствие #185.
Андрей, ты смотрел фильмы про джинов? Нужно точнее формулировать желания :)
Температуру на дисплее и в лог всегда писать текущую. Если ДТ не ответил, или ответил с ошибкой, тогда пишем в лог и на дисплей Н/Д (кажется именно так у нас и работало до введения задержки вентилятора?)
Да. Но затем мы с тобой попросили Алексея определять аварию датчика температуры не по первому неуспешному чтению, а если результат не получен в течении 9 секунд.
А потом в #185 ты попросил увеличить задержку до 90 секунд.
Учитывая, что на опрос датчика требуется пара секунд, считаю что задержка на определение неисправности в 9 секунд адекватна и единичные сбои в журнал писать не нужно.
Предлагаю откатить значение таймаута на 9 секунд а условия Аварии 23 в тз изменить.
Т.к. нужно будет менять ТЗ, тикет забираю себе.
Андрей, ты согласен с предложением?
comment:3 by , 6 лет ago
Replying to san:
Оказывается у нас на время задержки выводится несуществующая температура. Кажется мне, неправильно это.
Это следствие #185.
Андрей, ты смотрел фильмы про джинов? Нужно точнее формулировать желания :)
Нет. Порекомендуешь какой-нибудь конкретный?
Температуру на дисплее и в лог всегда писать текущую. Если ДТ не ответил, или ответил с ошибкой, тогда пишем в лог и на дисплей Н/Д (кажется именно так у нас и работало до введения задержки вентилятора?)
Да. Но затем мы с тобой попросили Алексея определять аварию датчика температуры не по первому неуспешному чтению, а если результат не получен в течении 9 секунд.
А потом в #185 ты попросил увеличить задержку до 90 секунд.
Это был Артем в костюме Андрея.
Учитывая, что на опрос датчика требуется пара секунд, считаю что задержка на определение неисправности в 9 секунд адекватна и единичные сбои в журнал писать не нужно.
Я не предлагаю писать в журнал каждый единичный сбой, я предлагаю писать в журнал правильную температуру, раз уж мы туда ее пишем.
А вот Аварию 23 писать в логи при возникновении и снятии. Но через 10 сек при непрерывном Н/Д.
Предлагаю откатить значение таймаута на 9 секунд а условия Аварии 23 в тз изменить.
Именно это я и ожидал
Т.к. нужно будет менять ТЗ, тикет забираю себе.
Хорошо.
Андрей, ты согласен с предложением?
Видимо не полностью.
comment:5 by , 6 лет ago
- Таймаут датчика возвращаем в 9 секунд?
- Таймаут Аварии 23 устанавливаем в 80 секунд?
comment:6 by , 6 лет ago
Нет.
Таймаут аварии 9 сек.
Таймаут включения вентилятора после возникновения аварии 80 сек
Пока не возникла авария, температуру выводим последнюю успешно считанную.
comment:7 by , 6 лет ago
Решение: | → сделано |
---|---|
Состояние: | assigned → closed |
- Создал #205
- Внёс в ТЗ изменения:
Авария 23. Ошибка датчика температуры
Условие: отсутствие корректных данных о температуре масла(к.з. или обрыв, или ошибка датчика температуры) в течении 80 секунд
...
Условие снятия: появление новых корректных показаний термодатчика
comment:8 by , 6 лет ago
Replying to andrei:
Артем предлагает кроме задержки включения вентиляторов 90 с,
Артем что-то путает. У нас нет зарержки включения вентилятора. Вентилятор включается немедленно при возникновении аварии 23, как того и требует ТЗ.
comment:9 by , 6 лет ago
Replying to andrei:
Оказывается у нас на время задержки выводится несуществующая температура.
Кажется мне, неправильно это.
Я думаю, что тебе кажется неправильно. Что значит "несуществующая"? Отображается ровно та температура, которую вернул датчик температуры.
Предложение:
Температуру на дисплее и в лог всегда писать текущую.
У нас так и делается. Если датчик температуры исправен, везде - и при выводе в лог, и при отображении на дисплее - используется последняя полученная от датчика температура. Если датчик температуры неисправен, везде отображается "н/д", то есть значение температуры недоступно.
Если ДТ не ответил, или ответил с ошибкой, тогда пишем в лог и на дисплей Н/Д (кажется именно так у нас и работало до введения задержки вентилятора?)
Аварию 23. Ошибка датчика температуры начинать если в течении 10 сек. не было корректной температуры.
Вентилятор включать с задержкой 80 сек с момента появления аварии.
Это противоречит требованию ТЗ:
Действия в авт. и руч. режиме режиме:
- включить вентилятор маслоохладителя
Оказывается у нас на время задержки выводится несуществующая температура.
Кажется мне, неправильно это.
Предложение:
Температуру на дисплее и в лог всегда писать текущую. Если ДТ не ответил, или ответил с ошибкой, тогда пишем в лог и на дисплей Н/Д (кажется именно так у нас и работало до введения задержки вентилятора?)
Аварию 23. Ошибка датчика температуры начинать если в течении 10 сек. не было корректной температуры.
Вентилятор включать с задержкой 80 сек с момента появления аварии.