#43 closed улучшение (fixed)
Улучшение журнала: добавить фильтры
| Сообщил: | san | Владелец: | alx |
|---|---|---|---|
| Приоритет: | средний | Этап разработки: | 2-я очередь |
| Ключевые слова: | Копия: | andrei |
Описание (последним изменил )
При выводе журнала на экран, в связи с маленьким размером экрана, информация воспринимается сложно. Предлагаю дать пользователю возможность настроить фильтры для вывода только нужной ему информации.
Фильтры сообщений:
- аварии ("Авария привода:", "Снятие аварии:")
- неисправности датчиков
- параметры (P=0.6 атм. G=-34.9 кгс I1=0.0 A I2=...)
- Запуск, останов ("НУ1:", "НУ2:")
- датчики положения (сработал, несработал)
- modbus (smbus.cpp: ?)
- модуль SIM (sim5320.cpp:)
- flash (flash.cpp:)
- ОС (kernel: , root:)
- остальное (то что не попало в фильтры)
Может быть что-то ещё?
История изменений (14)
comment:3 by , 6 лет ago
Было бы хорошо, если бы ты сразу для каждого из перечисленных пунктов указал сигнатуру, по которой соответствующие записи журнала должны распознаваться. А то по большей части пунктов я сходу их придумать не могу...
следующий: 6 comment:4 by , 6 лет ago
будем делать во вторую очередь...
Удобства нужны для удобства дебага, который непосредственно связан с работой, которая в первую очередь...
comment:5 by , 6 лет ago
| Описание: | изменено (отличие) |
|---|
Действительно то что относится к controller.cpp: сложно распознать
comment:6 by , 6 лет ago
Replying to san:
будем делать во вторую очередь...
Удобства нужны для удобства дебага, который непосредственно связан с работой, которая в первую очередь...
Не думаю, что оператор, стоящий перед приводом, может выполнять какой-то дебаг. Максимум помощи, которую он может оказать в плане дебага - это скопировать весь лог и переслать его нам, то есть разработчикам. А нам здесь порядок отображения колонок на дисплее уже будет "до лампочки"... :)
comment:7 by , 6 лет ago
| Приоритет: | средний → низкий |
|---|---|
| Этап разработки: | 2-я очередь → 1-я очередь |
Высказанные устно аргументы убедили меня вернуть тикет в первую очередь.
comment:8 by , 6 лет ago
| Описание: | изменено (отличие) |
|---|---|
| Приоритет: | низкий → средний |
| Этап разработки: | 1-я очередь → 2-я очередь |
comment:9 by , 6 лет ago
Вот "неисправности датчиков" и "параметры" не понятно как отличить...
Может быть, при выводе их в лог снабдить их какой-то пометкой? или вообще всё что выводит controller.cpp выводить сразу с каким-то признаком принадлежности к определённой группе сообщений?
comment:10 by , 6 лет ago
Параметры отличить легко - I1=.* I2=.* F1=.* F2= (это регулярное выражение). Неисправности датчиков пока сделал как КЗ|[Оо]брыв, в первом приближении работает, позже можно уточнить. Не очень понятен смысл пункта "остальные". Саша, поясни, пожалуйста.
следующий: 12 comment:11 by , 6 лет ago
Не очень понятен смысл пункта "остальные". Саша, поясни, пожалуйста.
Смысл - вывести то что не попало в другие фильтры
comment:12 by , 6 лет ago
Replying to san:
Смысл - вывести то что не попало в другие фильтры
Спасибо. Ты повторил то, что уже написано в описании тикета, и не дал дополнительной разъясняющий информации. Тогда задам следующий вопрос: в чем смысл выводить то, что не попало под условия фильтров? Насколько я понимаю, смысл фильтрации состоит ровно в том, чтобы отображать то, что удовлетворяет выбранным условиям. Соответственно, вывод того, что не удовлетворяет ни одному из выбранных пользователем условий, на мой взгляд, противоречит этому смыслу...
comment:13 by , 6 лет ago
Я подумал, что это может кому-то понадобиться... Вдруг мы добавим какие-то сообщения а условия фильтра ещё нет... и кому-то понадобятся именно те сообщения, так он хотя-бы сможет выключить из вывода все известные условия оставив "остальные" и останется уже меньше сообщений для поиска :) Примерно такая логика.
Думаю, что всяческие удобства, не связанные непосредственно с работой привода, мы все-таки будем делать во вторую очередь...