Opened 6 лет ago

Closed 6 лет ago

Last modified 6 лет ago

#137 closed баг (fixed)

Не работает функция разворота по числу качаний

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

Описание (последним изменил alx)

Прошивка 636
Уставки контроллера в прилагаемых логах работы:
способ задания числа качаний (далее ЧК) - прямое указание
число качаний (ЧК) - 6
мин ЧК - 3
макс ЧК - 4
максимальное время хода вниз - 30сек
03.12.2018 с 10:45:00 до 10:47:30 уставка условия разворота по максимальному времени хода вниз
03.12.2018 с 10:47:30 до 10:50 уставка условия разворота по числу качаний

Привод при любом положении уставки разворота выполняет разворот только по максимальному времени хода, т.е. ход вниз до разворота всегда занимал 30сек. Ожидалось, что при уставке разворота по числу качаний, привод вниз будет двигаться не более 10сек (а фактически 10сек минус вермя хода вверх).

Вложения (1)

Ёж-ABCD1234567.rar (67.9 KB ) - added by Art_M 6 лет ago.
Лог к багу

Download all attachments as: .zip

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

by Art_M, 6 лет ago

Вложение: Ёж-ABCD1234567.rar added

Лог к багу

comment:1 by Art_M, 6 лет ago

Описание: изменено (отличие)

comment:2 by Art_M, 6 лет ago

Описание: изменено (отличие)

comment:3 by alx, 6 лет ago

Краткое описание: Не работает функция разворота по числу качаний, плюс баг кириллицы в наименовании скважиныНе работает функция разворота по числу качаний
Описание: изменено (отличие)

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

Сообщение о проблеме с кириллицей перенесено в тикет #138.

comment:4 by Art_M, 6 лет ago

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

comment:5 by alx, 6 лет ago

Владелец: установлено в alx
Состояние: newaccepted

comment:6 by alx, 6 лет ago

Думаю, что я нашел причину бага.

Согласно ТЗ, при настройке “Разворот при долгом ходе вниз” в значении “По числу качаний” таймаут условия разворота зависит от того, исчерпаны ли уже меры по повышению производительности (частоты НУ достигли максимума, включены все разрешенные ЭК и т.п.). Раньше последней из мер повышения производительности было отключение задержек, но затем требования ТЗ были изменены, и отключение задержек было исключено из мер по повышению производительности. Но в коде, определяющем критерий разворота при долгом ходе вниз, проверка отключения задержек по ошибке осталась, в результате чего этот код, так как задержки теперь никогда не отключаются, всегда считал, что меры повышения производительности еще не исчерпаны, и таймаут разворота при долгом ходе вниз определял по настройке "Максимальное время хода вниз".

comment:7 by alx, 6 лет ago

Решение: fixed
Состояние: acceptedclosed

In 641/smartHDC:

Исправлена ошибка: при определении условия разворота при долгом ходе вниз
по-прежнему проверялось отключение задержек, хотя отключение хадержек
было исключено из мер по повышению производительности и никогда не выполняется.
Теперь параметр disableDelays исключен из параметров производительности,
убран из веб-интерфейса. При запросе входного регистра 0x0b по шине modbus
в целях совместимости всегда возвращается 0. Соответственно, проверка отключения
задержек при определении критерия разворота при долгом ходе вниз убрана.
Closes #137.

comment:8 by alx, 6 лет ago

Если есть возможность, было бы неплохо проверить работу разворота по числу качаний и убедиться, что теперь настройка действует.

comment:9 by Art_M, 6 лет ago

Ребята, сбросьте прошивку на "npkperm@…", пожалуйста, я проверю на столе

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

Replying to Art_M:

Ребята, сбросьте прошивку на "npkperm@…", пожалуйста,

Если под прошивкой подразумевался пакет smarthdc, то отправил.

in reply to:  10 comment:11 by alx, 6 лет ago

Replying to alx:

Если под прошивкой подразумевался пакет smarthdc, то отправил.

Ваш почтовый релей отказывается принять сообщение, говорит: "550 virus message".

comment:12 by alx, 6 лет ago

Пакет выложил сюда.

Note: See TracTickets for help on using tickets.