Opened 6 лет ago

Closed 6 лет ago

Last modified 6 лет ago

#204 closed улучшение (сделано)

Задержка НД датчика температуры.

Сообщил: andrei Владелец: san
Приоритет: средний Этап разработки: 2-я очередь
Ключевые слова: Копия: san, Art_M, alx

Описание

Артем предлагает кроме задержки включения вентиляторов 90 с, сделать независимую задержку 10 с от момента пропадания показаний ДТ перед отображением НД на панели оператора.

"Включение вентилятора производится не через 90, а через ~110сек почему то... Видимо есть ещё какая то задержка.... Ну и нужно ещё, чтобы бы н/д сразу отображалось (допустим через 10сек) , а вентилятор уже позже включался через 90сек... Иначе невозможно понять сразу, рабочая проводка или снова отпала, пока не пройдут эти почти две минуты"

История изменений (9)

comment:1 by andrei, 6 лет ago

Оказывается у нас на время задержки выводится несуществующая температура.
Кажется мне, неправильно это.
Предложение:
Температуру на дисплее и в лог всегда писать текущую. Если ДТ не ответил, или ответил с ошибкой, тогда пишем в лог и на дисплей Н/Д (кажется именно так у нас и работало до введения задержки вентилятора?)
Аварию 23. Ошибка датчика температуры начинать если в течении 10 сек. не было корректной температуры.
Вентилятор включать с задержкой 80 сек с момента появления аварии.

comment:2 by san, 6 лет ago

Владелец: изменён с alx на san
Состояние: newassigned

Оказывается у нас на время задержки выводится несуществующая температура. Кажется мне, неправильно это.

Это следствие #185.
Андрей, ты смотрел фильмы про джинов? Нужно точнее формулировать желания :)

Температуру на дисплее и в лог всегда писать текущую. Если ДТ не ответил, или ответил с ошибкой, тогда пишем в лог и на дисплей Н/Д (кажется именно так у нас и работало до введения задержки вентилятора?)

Да. Но затем мы с тобой попросили Алексея определять аварию датчика температуры не по первому неуспешному чтению, а если результат не получен в течении 9 секунд.
А потом в #185 ты попросил увеличить задержку до 90 секунд.
Учитывая, что на опрос датчика требуется пара секунд, считаю что задержка на определение неисправности в 9 секунд адекватна и единичные сбои в журнал писать не нужно.

Предлагаю откатить значение таймаута на 9 секунд а условия Аварии 23 в тз изменить.
Т.к. нужно будет менять ТЗ, тикет забираю себе.

Андрей, ты согласен с предложением?

in reply to:  2 comment:3 by andrei, 6 лет ago

Replying to san:

Оказывается у нас на время задержки выводится несуществующая температура. Кажется мне, неправильно это.

Это следствие #185.
Андрей, ты смотрел фильмы про джинов? Нужно точнее формулировать желания :)

Нет. Порекомендуешь какой-нибудь конкретный?

Температуру на дисплее и в лог всегда писать текущую. Если ДТ не ответил, или ответил с ошибкой, тогда пишем в лог и на дисплей Н/Д (кажется именно так у нас и работало до введения задержки вентилятора?)

Да. Но затем мы с тобой попросили Алексея определять аварию датчика температуры не по первому неуспешному чтению, а если результат не получен в течении 9 секунд.
А потом в #185 ты попросил увеличить задержку до 90 секунд.

Это был Артем в костюме Андрея.

Учитывая, что на опрос датчика требуется пара секунд, считаю что задержка на определение неисправности в 9 секунд адекватна и единичные сбои в журнал писать не нужно.

Я не предлагаю писать в журнал каждый единичный сбой, я предлагаю писать в журнал правильную температуру, раз уж мы туда ее пишем.
А вот Аварию 23 писать в логи при возникновении и снятии. Но через 10 сек при непрерывном Н/Д.

Предлагаю откатить значение таймаута на 9 секунд а условия Аварии 23 в тз изменить.

Именно это я и ожидал

Т.к. нужно будет менять ТЗ, тикет забираю себе.

Хорошо.

Андрей, ты согласен с предложением?

Видимо не полностью.

comment:4 by andrei, 6 лет ago

Копия: alx added

comment:5 by san, 6 лет ago

  1. Таймаут датчика возвращаем в 9 секунд?
  2. Таймаут Аварии 23 устанавливаем в 80 секунд?

comment:6 by andrei, 6 лет ago

Нет.
Таймаут аварии 9 сек.
Таймаут включения вентилятора после возникновения аварии 80 сек
Пока не возникла авария, температуру выводим последнюю успешно считанную.

comment:7 by san, 6 лет ago

Решение: сделано
Состояние: assignedclosed
  1. Создал #205
  2. Внёс в ТЗ изменения:

Авария 23. Ошибка датчика температуры
Условие: отсутствие корректных данных о температуре масла(к.з. или обрыв, или ошибка датчика температуры) в течении 80 секунд
...

Условие снятия: появление новых корректных показаний термодатчика

in reply to:  description comment:8 by alx, 6 лет ago

Replying to andrei:

Артем предлагает кроме задержки включения вентиляторов 90 с,

Артем что-то путает. У нас нет зарержки включения вентилятора. Вентилятор включается немедленно при возникновении аварии 23, как того и требует ТЗ.

in reply to:  1 comment:9 by alx, 6 лет ago

Replying to andrei:

Оказывается у нас на время задержки выводится несуществующая температура.
Кажется мне, неправильно это.

Я думаю, что тебе кажется неправильно. Что значит "несуществующая"? Отображается ровно та температура, которую вернул датчик температуры.

Предложение:
Температуру на дисплее и в лог всегда писать текущую.

У нас так и делается. Если датчик температуры исправен, везде - и при выводе в лог, и при отображении на дисплее - используется последняя полученная от датчика температура. Если датчик температуры неисправен, везде отображается "н/д", то есть значение температуры недоступно.

Если ДТ не ответил, или ответил с ошибкой, тогда пишем в лог и на дисплей Н/Д (кажется именно так у нас и работало до введения задержки вентилятора?)
Аварию 23. Ошибка датчика температуры начинать если в течении 10 сек. не было корректной температуры.
Вентилятор включать с задержкой 80 сек с момента появления аварии.

Это противоречит требованию ТЗ:

Действия в авт. и руч. режиме режиме:

  • включить вентилятор маслоохладителя
Note: See TracTickets for help on using tickets.