Opened 6 years ago

Closed 5 years ago

Last modified 15 months ago

#183 closed улучшение (готово)

Изменение ПО УГП-04 по запросу НПК

Reported by: andrei Owned by: san
Priority: critical Component: UGP
Keywords: Cc: Director, andrei, vlad, Art_M

Description

Добавить в прошивку блоков УГП хотелки НПК и легализовать её.
Хотелки согласовать с Артемом.

Attachments (2)

viber image.jpg (246.5 KB ) - added by andrei 6 years ago.
1.png (2.0 KB ) - added by san 6 years ago.

Download all attachments as: .zip

Change History (38)

comment:1 by andrei, 6 years ago

Cc: Director added

comment:2 by san, 6 years ago

Owner: changed from san to andrei
Status: newassigned

Жду согласования "хотелок".

comment:3 by san, 6 years ago

Summary: Саша, прошивкаИзменение ПО УГП-04 по запросу НПК

by andrei, 6 years ago

Attachment: viber image.jpg added

comment:4 by andrei, 6 years ago

Артем предложил скорректировать алгоритм.

  • вместо часов на "экран по умолчанию" выводить текущее давление.
  • редактирование даты/времени притащить в основную ветку.
  • добавить возможность скорректировать кнопками значение площади поршня.


Last edited 6 years ago by san (previous) (diff)

comment:5 by andrei, 6 years ago

Cc: andrei added
Owner: changed from andrei to san

comment:6 by san, 6 years ago

исправил в коменте ссылку на картинку

comment:7 by san, 6 years ago

Cc: vlad added

В целом, не вижу никаких препятствий сделать так, как просит Артём.
Единственное что мне не нравится символ обозначающий площадь поршня - это уже дополнительный изврат поверх изврата, предлагаю выводить вообще без символа

Не понятно:

  • в каких пределах и с какой дискретностью изменяются значения площади поршня
  • в блок-схеме не раскрыт механизм коррекции площади поршня(напомню что кнопок цифр у нас нет)
  • в какой регистр сохранять площадь поршня, в каком виде, как интерпретировать значения выходящие за пределы (наверное больше к Владу вопрос, т.к. этот параметр будет считываться СД-01 и UGP_config)

comment:8 by andrei, 6 years ago

Хорошо, ждём отмашки директора на проведение работ.

comment:9 by andrei, 6 years ago

От Артема:

  1. Регулировка с точностью до десятых кнопками вверх и вниз.

Диапазон можно сделать от 000,0 до 199,9.

  1. второй вариант, можно сделать символ S, а диапазон от 20,0 до 119,9, т.е в таком случае переход будет S20,0... S99.8, S99.9, S00.0, S00,1... S19.9, т.е. указывать без разряда соток.

comment:10 by andrei, 6 years ago

Cc: Art_M added

comment:11 by san, 6 years ago

S20,0... S99.8

на самом деле это будет выглядеть как 520.0 ... 599.8
так что предлагаю просто без символов вывести число, как есть

comment:12 by andrei, 6 years ago

Согласен, квадратная S от квадратной 5 не отличаются.
Артём, оставляем первый вариант?
Только очень долго придется перебирать. Можно корректировать каждый разряд, а вводом сдвигать курсор вправо. И часы я бы так же настраивал. Но тогда будет отличаться от других настроек.

comment:13 by san, 6 years ago

очень долго придется перебирать

если кнопку зажать - включается быстрый перебор значений

comment:14 by andrei, 6 years ago

Надо еще сделать "двойной клик" - переключение на сотню.

comment:15 by Art_M, 6 years ago

Ребята, приветствую! Давайте тогда оставим первый вариант. Способ ввода оставить так, как в остальных настройках. Ничего страшного, что долго, регулировать это значение не нужно каждый день. Такая нужда возникает при заменах блоков УГП или при замене модулей гидростанций и там, где нет связи.

comment:16 by san, 6 years ago

Хорошо, вопросов больше нет, с Владом мы отдельно разберёмся.

ждём отмашки директора на проведение работ.

comment:17 by andrei, 6 years ago

Артем, у нас есть заказ на 4 блока УГП04-Д2.
Насколько для них необходима новая прошивка с учетом этого тикета?

comment:18 by Art_M, 6 years ago

Давайте сделаем это!

Момент1: в площади поршня не забудьте запятую (точку), т.е. "i111,3"

Момент2: по умолчанию площадь поршня в этих блоках выставьте, пожалуйста, 42.2!

Вопрос: Ребята, вопрос, что будет если при отображении давления в текущем состоянии блока нажать на ввод? Перейдем сразу к настройкам площади поршня? Боюсь это может вызвать частое случаное попадание в эту настройку, что может привести к сбою значения. Можно ли сделать, что бы при нажатии ввод при отображения давления в текущем состоянии не возможно было переходить на изменение площади поршня? Если так, то можно ли сделать что бы нажатие ввода из текущего состояния ничего не вызывало?

Вопрос совместимости: при использовании СД-01 с предыдущими прошивками с каким нибудь дремучим UGP_config ничего же не поменяется? Правильно ли я понимаю, что старые СД просто не будут подхватывать значение площади поршня и не будут передавать его в старый UGP_config, который и так по умолчанию считает, что площадь 42,2? Надо бы это проверить!

Что бы не тратить много времени на РЭ, я думаю будет достаточно вставить схему...

Last edited 6 years ago by Art_M (previous) (diff)

comment:19 by san, 6 years ago

i111,3
что такое i ?

comment:20 by san, 6 years ago

а понял, это нижний левый сегмент индикатора, увидел в блок-схеме

in reply to:  18 comment:21 by andrei, 6 years ago

Перейдем сразу к настройкам площади поршня?

Так это ты и просил.

Можно ли сделать, что бы при нажатии ввод при отображения давления в текущем состоянии не возможно было переходить на изменение площади поршня?

Можно при этом, например, инвертировать ход поршня.
Предлагаю не нажимать кнопку ввод, подходя к скважине.
Вообще предлагаю не трогать руками то, о работе чего не имеешь представления.
Артем, начинаешь противоречить сам себе.
Так мы дойдем до "нельзя ли не останавливать привод если кто-то случайно выключил вводной автомат?"

comment:22 by Art_M, 6 years ago

Не согласен с тобой, Андрей!

Ребята, я просил сделать изменение поршня, и первым делом предложил внести эту настройку в череду имеющихся настроек. На что Вы же мне и сообщили, что там и так сейчас сложно разобрать что за что отвечает, и предложили ее спрятать под какую нибудь другую настройку. Я не против, но я считаю крайне не удобным, если из экрана "текущего отображения давления", появляющемуся по умолчанию при автоматическом запуске или при долгом отсутствии нажатий, будет возможность с нажатия кнопки "ВВОД" начинать вводить площадь поршня, т.к. это интуитивно не понятно, оператор может посчитать это, например, вводом числа качаний. Этой функцией будут пользоваться раз в 5лет, когда Блок с одного привода, переставят на другой. Зачем ее регулирование выносить в первую очередь, при том что назначение настройки не очевидно сходу?

Поэтому предлагаю игнорировать кнопку "ВВОД" при "текущем отображении давления". Что бы попасть к площади цилиндра можно было, пусть через нажатие "ВВОД" при "отображении давления", но только когда ты дошел до отображения давления через перелистывание настроек кнопкой "R".

Где тут противоречие? Я говорю о логичности интерфейса...


Андрей, и вот тебе на это наглядный пример. Представь у тебя есть телевизор и пульт к нему, на пульте есть кнопки "больше" и "меньше" громкости, кнопка "R" и кнопка "ВВОД".

При нажатии "R" тебе предлагают меню "настройка яркости", "настройка контрастности", "настройка цветности", "настройка частот каналов", "настройка языка" и т.д

Допустим тебе нужно поменять какую то настройку например контраст, ты нажимаешь "R", кнопками "больше" и "меньше" выбираешь нужный пункт "настройка контраста", нажимаешь "ВВОД" ты видишь шкалу нажимая на кнопки "больше" и "меньше" регулируешь ее. Тут все логично.

А теперь представь, пульт взял твой дедушка у которого хреново со зрением, и он хочет прибавить на телике яркости, смотрит в пульт и думает на что нажать? "R", "ВВОД", "Больше", "Меньше" Кнопки то имеют логичные названия, так ведь?
Ну пусть это будет "ВВОД", предположит дедушка! А тут как раз и шкала вылетает! Дедушка предполагает, может быть это и есть яркость. Логично же, что если человек не может ничего разобрать на экране, то вероятно ему первым делом предлагают настройку яркости экрана, нажимает "больше" или "меньше", а у него раз и включается поиск частот, дедушка смотрит на тусклый снежок на экране и думает "ну и какая тупица могла придумать при нажатии на кнопку "ВВОД" при просмотре ТВ начинать регулировать пойманную частоту? я и так ни хрена не вижу, так теперь и послушать не смогу"

Ясен пень, такой исход событий следует из мануала, но где же дружелюбность интерфейса, да где же, черт побери, банальная логика?

Last edited 6 years ago by Art_M (previous) (diff)

comment:23 by andrei, 6 years ago

Дружелюбность интерфейса? Банальная логика? Из меню отображения давления попадать в редактирование площади))) это да, это прям логично! Может в схеме рисунки неправильные?
А дедушек не нужно подпускать в блоку УГП, на этом я настаиваю.

comment:24 by Art_M, 6 years ago

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

comment:25 by san, 6 years ago

Ок. Добавляем в главное меню, после элемента "время" элемент "площадь поршня" в виде

Диапазон значений от 0,0 до 199,9.

by san, 6 years ago

Attachment: 1.png added

comment:26 by Art_M, 6 years ago

Александр, я согласен с этим!

Небольшая поправочка не просто "Площадь поршня", а "Рабочая площадь поршня"

Ребята, еще момент, который мы начали не довели до ума и чуть было не забыли, это по поводу наименования и заводского номера блока УГП.

Т.к. нас единственная возможность идентифицировать динамограммы, снимаемые через СД, это по наименованию Блока УГП, которое без ПК нельзя ни посмотреть, ни поменять, то соответственно, я предлагал везде задавать в наименовании Блока УГП заводской номер Блока УГП, при этом динамограммы будут сыпаться в папки с заводскими номерами блоков УГП, что в дальнейшем существенно упростит задачу идентифицирования динамограмм. Так же я просил сделать вывод на экран этот заводской номер.

После чего родилось какое то решение. При совместном рассмотрении (Андрей, Влад, Сергей и я, а Александр был в командировке, если не ошибаюсь) мы пришли к выводу, что решение не самое удачное. Правда, сейчас я уже точно не помню почему. Если не ошибаюсь, то заводской номер появился, отдельно от наименования, при этом его так и не возможно было просмотреть, при этом UGP_Config продолжал игнорировать этот заводской номер, и продолжал сыпать либо по наименованию либо в Станции1,2,3... Я полагаю сложность в том, что это поле является текстовым, а где то уже могут быть даны имена с буквами и отображать на нашем экране это проблемно. На тот момент Андрей предложил и в "заводской номер" и в "наименование" задавать заводской номер. Однако в таком случае теряется смысл, потому что даже если мы увидим "заводской номер", то UGP_config будет сыпать динамограммы в "Наименование1,2,3", которые могут не соответствовать фактическому расположению Блоков (например в одном месте, где была телеметрия Блоку УГП было дано текстовое наименование "скважина123", а после привод переставили туда, где нет связи на скважину456).

Единственное решение, которое я сейчас вижу, чтобы избежать недоразумений - это оставить и наименование и заводской номер, при выпуске и в "наименование" и в "заводской номер" ставить заводской номер, а при формировании папок на выходе UGP_config сбрасывать динамограммы в папке "data/Принятые с СД" (в противоположность от "data/Принятые по GPRS")

1.Если нет ничего, то "Станция1,2,3", тут ничего не сделаешь без установки наименования.

2.Если есть только наименование, то в папку с наименованием , т.к. с этим уже ничего не поделаешь, пока не проедешь с ПК по скважинам и не поменяешь это значение. Слава Богу, что в конце 2015 года мы с Сергеем проехали по 50-ти с лишним приводам, т.е. практически по всем, за исключением 5 или 6 Блоков, приводов которые не эксплуатировались или были обесточены, которые работают в ТатРИТЭКе и задали заводские номера в наименовании.

3.Если есть заводской номер и наименование и они разные, то в блоке УГП предлагать опцию сохранять в папку "Номер_Наименование" (вариант по умолчанию) или "Наименование_Номер" или только "Номер". Если заводской номер и наименование совпадают, то соответственно только "Номер". Пусть технолог сам выберет, как ему удобнее работать.

А сам номер сделать просматриваемым на самом Блоке УГП, что бы можно было удостовериться что шильдик соответствует плате, или в случаях когда шильдик потерялся или испортился. Таким образом, в наименовании создаваемой папки будет всегда фигурировать заводской номер Блока УГП, где бы он не эксплуатировался, что существенно упростит идентификацию динамограмм.

Остается лишь вопрос, стоит ли создавать подпапку с датой снятия динамограмм с СД. Т.к. на многих Блоках УГП таймеры уплывают существенно либо умирают батарейки, то в наименовании динамограмм будут указано время от балды, то мы опять столкнемся с хаосом. Понятно, что можно упорядочить не по имени а по дате создания файла, но технолог может не уметь пользоваться такой функцией Windows, тем более в выпадающем окне "Считать с Card" у ПО "Dinamograph". Поэтому предлагаю сделать подпапки с датой.

Вроде бы такое решение по UGP_config и Блоку УГП никак не противоречит уже имеющемуся фонду, но в дальнейшем должно существенно упростить работу технолога.

Ребята, может у Вас есть какие то предложения или замечания? Как еще можно сделать что бы избегать хаоса при работе с СД?

comment:27 by san, 6 years ago

Подвожу итог заданий.

УГП:

  1. вместо часов на "экран по умолчанию" выводить текущее давление.
  2. выводить в основном меню Рабочую площадь поршня добавить возможность скорректировать кнопками значение площади поршня.
  3. выводить номер блока УГП(аналогично версии ПО УГП)

СД-01:

  1. Кроме имени считывать ещё и номер блока
  2. Считывать рабочую площадь поршня

Ugp_config:

  1. Добавить в таблицу номер блока
  2. Добавить в таблицу Рабочую площадь поршня с возможностью редактирования
  3. Добавить команду записи номера блока
  4. При сохранении ДГ с блока или СД-01, если Рабочая площадь поршня неизвестна - предложить оператору ввести вручную xxx.x см2 (по умолчанию 42.2 см2)
  5. При сохранении ДГ в имени директории указывать и номер блока и имя
  6. Добавить настройку шаблона имени ДГ(<имя>_<номер>_<год>_<месяц>_<день>_<время>) и директории сохранения (<имя>_<номер>) в шаблоне можно использовать любые из перечисленных элементов в любом порядке.
  7. Добавить опциональную возможность сохранять ДГ в поддиректории c названием <текущая дата>

Артём, Андрей, проверьте ничего не забыл?

Last edited 6 years ago by san (previous) (diff)

comment:28 by andrei, 6 years ago

Как посмотреть дату/время? R понажимать?
Артем, насколько актуально? Сейчас готовятся 4 блока УГП-04, может их отгрузим с серийным ПО? Чтобы не затягивать сроки.

comment:29 by Art_M, 6 years ago

По схеме нажать кнопку R 6 или 7 раз, в зависимости куда попадет настройка рабочей площади площадь поршня.

Решение актуально, давайте постараемся на них реализовать новую прошивку:

  1. блоки пойдут как раз туда, где просили это.
  2. опробуем прошивку не на новых приводах, а на "ремонтных".

Мы же об этой прошивке заговорили около или даже более года назад... Хотели выпустить ее к прошлой партии станций СУГП, потом сообщили, что она недоделанная, сырая и доделывание займет много времени. Чтобы не затягивать сроки мы допустили поставку СУГП со старой прошивкой, и если опять переносить сроки, то такими темпами мы ее никогда не закончим.

comment:30 by san, 6 years ago

недоделанная, сырая

В общем то, она доделанная и сухая, и сдана больше полгода назад, просто задача была поставлена по другому.

comment:31 by Art_M, 6 years ago

Пардоньте, господа, это палка в мой огород? Получается, что я сам виновен в затягивании сроков?

  1. Хочу поинтересоваться когда сдана, кому сдана прошивка? Почему меня при этом не было?

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

  1. Что значит задача была поставлена по другому?

Ребята, сейчас специально полез открыл письмо еще от 18 июля 2017 года, разосланное мной на andrei@…, san@…, vlad@…. Позвольте процитировать тут:

Пожелания к меню прошивки блока УГП

1) Через кнопку R листаются все настройки поочередно вкруг, включая время (без даты) и площадь цилиндра.

2) На площади кнопкой R входим в режим редактирования площади (здесь очевидная ошибка, что речь о кнопке ВВОД)

3) На часах кнопкой R входим в подменю редактирования времени (здесь очевидная ошибка, что речь о кнопке ВВОД)

4) В текущем состоянии блока отображать не часы и дату, а текущее давление

5) Заводской номер отображать, так же как и прошивку по долгому нажатию на R в ручном режиме. (пусть 5 секунд зав.номер, потом номер прошивки)

Пожелания к СД и UGP_Config

Ввести в UGP_Config рядом с кнопкой считать с СД поле со значеним по умолчанию. Т.е. если динамки имеют переписанные с Блока УГП значения площади, то используются эти значения, если нет, то значение из поля в UGP_Config.

Скажите, много ли добавилось задач к прошивке блока УГП за последние 9 с лишним месяцев или нет? Я вижу только пожелание к ПО UGP_Config по наименованию папок и подпапок, кто не верит, проверьте сами!

Ребята, давайте не будем искать крайнего, давайте просто возьмем, чуть-чуть напряжемся, и наконец сделаем это!

Last edited 6 years ago by Art_M (previous) (diff)

comment:32 by andrei, 6 years ago

Крайнего не ищем. Отмашка от всех инстанций есть, прошивка пишется.

in reply to:  31 comment:33 by san, 6 years ago

Replying to Art_M:

палка

сам первый начал :-D сырыми обзываться

04.2017 выпущена прошивка 4.09/4.69 по предыдущему согласованому т.з.

Пожелания к меню прошивки блока УГП

Ну вот наконец, пожелания превратились в т.з. и работаем

Last edited 6 years ago by san (previous) (diff)

comment:34 by san, 6 years ago

вместо часов на "экран по умолчанию" выводить текущее давление.
выводить в основном меню Рабочую площадь поршня добавить возможность скорректировать кнопками значение площади поршня.
выводить номер блока УГП(аналогично версии ПО УГП)

Реализовано в v.4.10/4.70

comment:35 by san, 5 years ago

Resolution: готово
Status: assignedclosed

УГП:
вместо часов на "экран по умолчанию" выводить текущее давление.
выводить в основном меню Рабочую площадь поршня добавить возможность скорректировать кнопками значение площади поршня.
выводить номер блока УГП(аналогично версии ПО УГП)

Реализовано в УГП-04 v.4.10/4.70

СД-01:
Кроме имени считывать ещё и номер блока
Считывать рабочую площадь поршня

Реализовано в СД-01 v.1.11

Ugp_config:
Добавить в таблицу номер блока
Добавить в таблицу Рабочую площадь поршня с возможностью редактирования
Добавить команду записи номера блока
При сохранении ДГ с блока или СД-01, если Рабочая площадь поршня неизвестна - предложить оператору ввести вручную xxx.x см2 (по умолчанию 42.2 см2)
При сохранении ДГ в имени директории указывать и номер блока и имя
Добавить настройку шаблона имени ДГ(<имя>_<номер>_<год>_<месяц>_<день>_<время>) и директории сохранения (<имя>_<номер>) в шаблоне можно использовать любые из перечисленных элементов в любом порядке.
Добавить опциональную возможность сохранять ДГ в поддиректории c названием <текущая дата>

Реализовано в Ugp_Config v2.28

comment:36 by san, 15 months ago

milestone: 1 очередь

Milestone deleted

Note: See TracTickets for help on using tickets.