#881 closed задача (готово)
Новая опция в конфигурации TD-01 (в ревизии v11)
| Reported by: | vlad | Owned by: | alx |
|---|---|---|---|
| Priority: | средний | Milestone: | 1 очередь |
| Component: | sw | Keywords: | |
| Cc: |
Description
1. Добавить новую настройку для каждого конвертера:
- галочка Цикл.
Конфигурационная переменная .5.0 CONFIG[150B] Байты 17, 34, 51, 68 ( (17+17*n) n=0..3 - конвертеры 1..4) [Бит] значение [ 0 ] =1/0 - форматированный/неформатированный [ 1 ] =1/0 - формировать цикловый СС на передачу в КИ0/ прозрачная передача КИ0 При установленной галочке бит1 = 1, при снятой бит1 = 0
2. В окне конфигурации присутствует какой-то артефакт - лишний столбец, приложил скриншот.
3. После обсуждения в предыдущих тикетах, предлагаю не делать (не хранить в веб интерфейсе) разные настройки для разных ревизий плат. При открытии окна конфигурации, если ревизия платы ниже определённой (в данном случае это 11), выдавать предупреждение о необходимости обновить ПО платы и о том, что некоторые функции могут в данной ревизии не поддерживаться и вообще их запись производится на свой страх и риск.
Attachments (2)
Change History (22)
by , 8 hours ago
| Attachment: | td01cfg.png added |
|---|
comment:1 by , 8 hours ago
| Milestone: | 2 очередь → 1 очередь |
|---|---|
| Type: | баг → задача |
follow-ups: 3 5 comment:2 by , 8 hours ago
follow-up: 6 comment:3 by , 8 hours ago
Replying to alx:
Поясни, пожалуйста, как определить номер ревизии платы.
Ревизия прошивки платы отображается на вкладке Обновление ПО в веб-интерфейсе.
Извини, написал неправильно, имелась в виду Ревизия прошивки.
follow-ups: 7 8 comment:4 by , 8 hours ago
Replying to vlad:
1. Добавить новую настройку для каждого конвертера:
- галочка Цикл.
Конфигурационная переменная .5.0 CONFIG[150B] Байты 17, 34, 51, 68 ( (17+17*n) n=0..3 - конвертеры 1..4) [Бит] значение [ 1 ] =1/0 - формировать цикловый СС на передачу в КИ0/ прозрачная передача КИ0 При установленной галочке бит1 = 1, при снятой бит1 = 0
??? Указанные байты в указанной переменной уже заняты - в них хранится конфигурационный параметр "Формат". Или я что-то неверно понял/посчитал?
comment:5 by , 7 hours ago
Replying to alx:
Это предложение я не понял... Ты предлагаешь мне дождаться обсуждения в каких-то "предыдущих тикетах", и после их окончания кое-что не сделать? :) Как это? :)
Нет, под фразой "после обсуждения" я имел в виду "на основе предыдущих обсуждений".
Replying to alx:
А можно не сделать сразу, не дожидаясь обсуждений? :)
Да, именно это я и предлагаю - сразу не делать поддержку старых ревизий прошивки, но сообщать пользователю, что ревизия прошивки устарела.
follow-up: 10 comment:6 by , 7 hours ago
Replying to vlad:
Извини, написал неправильно, имелась в виду Ревизия прошивки.
В таком случае, такое предупреждение уже есть - в виде красной надписи на "ярлыке" упомянутой вкладки "Обновление ПО.
follow-up: 9 comment:7 by , 7 hours ago
Replying to alx:
??? Указанные байты в указанной переменной уже заняты - в них хранится конфигурационный параметр "Формат". Или я что-то неверно понял/посчитал?
Всё верно посчитано. Там целый байт, а бит0 в нем занят под галочку "Формат". Теперь займём бит1 под галочку ЦС
comment:8 by , 7 hours ago
Replying to alx:
??? Указанные байты в указанной переменной уже заняты - в них хранится конфигурационный параметр "Формат". Или я что-то неверно понял/посчитал?
Я сам разобрался. Как следует из написанного в тикете #504, этот байт может принимать только значения 0 и 1. То есть фактически для настройки "Формат" используется только один младший бит этого байта. Вопрос снимается.
comment:9 by , 7 hours ago
Replying to vlad:
Там целый байт, а бит0 в нем занят под галочку "Формат".
Как следовало из тикета #504, предыдущий разработчик отвел для хранения настройки "Формат" не один бит, а целый байт, соответственно и веб-интерфейс после получения конфигурации для отображения этой настройки проверял не один бит, а весь байт, что меня и смутило...
follow-ups: 11 20 comment:10 by , 7 hours ago
Replying to alx:
В таком случае, такое предупреждение уже есть - в виде красной надписи на "ярлыке" упомянутой вкладки "Обновление ПО.
Да, просто в таком случае (когда ревизия платы ниже определённой) окно конфигурации будет вводить в заблуждение, т.к. плата старой ревизии прошивки не поддерживает определённые опции (новые функции)
comment:11 by , 7 hours ago
Replying to vlad:
Да, просто в таком случае (когда ревизия платы ниже определённой) окно конфигурации будет вводить в заблуждение, т.к. плата старой ревизии прошивки не поддерживает определённые опции (новые функции)
Не понимаю, каким образом диалог конфигурации может вводить в заблуждение - в нем же нет утверждений о том, поддерживает или не поддерживает плата какие-либо функции...
comment:13 by , 7 hours ago
Replying to vlad:
2. В окне конфигурации присутствует какой-то артефакт - лишний столбец, приложил скриншот.
Это столбец, в котором были чекбоксы "Сброс счетчиков". Почему-то было сделано так, что из диалога удалялись собственно чекбоксы и надпись заголовка, но сам столбец оставался в таблице.
by , 5 hours ago
| Attachment: | td01cfg2.png added |
|---|
follow-ups: 15 18 comment:14 by , 5 hours ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
приложил скрин2, колонку с галочками "цикл" лучше сделать после колонки Формат
follow-up: 16 comment:15 by , 5 hours ago
comment:16 by , 5 hours ago
follow-up: 19 comment:18 by , 4 hours ago
| Resolution: | → готово |
|---|---|
| Status: | reopened → closed |
Replying to vlad:
колонку с галочками "цикл" лучше сделать после колонки Формат
Во-первых, клолнка "Цикл" и так была размещена после.
Во-вторых, даже если бы она была размещена до колонки "Формат, это не повод переоткрывать тикет - ни в описании тикета, ни в комментариях я не видел и не вижу указания, где именно следует разместить столбец "цикл". Отсутствие такого указания я понимаю так, что выбор места размещения оставлен на мое усмотрение.
comment:19 by , 4 hours ago
Replying to alx:
Во-вторых, даже если бы она была размещена до колонки "Формат, это не повод переоткрывать тикет - ни в описании тикета, ни в комментариях я не видел и не вижу указания, где именно следует разместить столбец "цикл". Отсутствие такого указания я понимаю так, что выбор места размещения оставлен на мое усмотрение.
При создании тикета я приложил скрин, на котором (кроме пустой колонки) отметил, куда вставить колонку цикл.
comment:20 by , 4 hours ago
Replying to vlad:
Да, просто в таком случае (когда ревизия платы ниже определённой) окно конфигурации будет вводить в заблуждение, т.к. плата старой ревизии прошивки не поддерживает определённые опции (новые функции)
Вот это я сейчас и наблюдаю: в слоте 8 находится плата TD-01 ревизии прошивки 10, которая не поддерживает настройку формировать "Цикл". Я открываю конфигурационное окно, вижу эти галочки, записываю в плату эти настройки и никто мне как пользователю не сообщает, что что-то идёт не так, хотя можно предупредить.
А в данный момент настройки "по-тихому" записываются в плату, как-будто всё отлично,
Replying to vlad:
Это предложение я не понял... Ты предлагаешь мне дождаться обсуждения в каких-то "предыдущих тикетах", и после их окончания кое-что не сделать? :) Как это? :) А можно не сделать сразу, не дожидаясь обсуждений? :)
Поясни, пожалуйста, как определить номер ревизии платы.