Страница 1 из 4 123 ... ПоследняяПоследняя
Показано с 1 по 10 из 31

Тема: Опечатки в target файле для ПЛК110

  1. #1

    По умолчанию Опечатки в target файле для ПЛК110

    Копался в target файлах для ПЛК110 (смотрел 2.10 версию target файлов для PLC110_32M) -- нашёл следующие ошибки. Есть шанс, что target'ы обновят?


    Ещё интересно стало: target файлы вручную составляют? Или какой-то инструмент-таки есть?
    Тут, скорее, просто интересно, т.к. крайне похоже, что текущие файлы PLC110_32.cfg, owen.cfg и т.п. -- "рукотворные".
    Если вручную, то снимаю шляпу перед тем, кто в блокноте такое составил -- довольно мало ошибок.

    Логические:
    Код:
    [Module.ID140]
    Name=Float variable (Listen)
    ...
    Param6=Name='Float type',Section=float_type,Access=RW,Visible=True, Default=0
    
    [Parameter.FrameOriented]
    Id=1005
    Type=INT
    Class=Symbolic
    Symbol1=ASCII
    Value1=0
    Symbol2=RTU
    Value2=1
    Default=0
    Должно быть Default=1 в обоих случаях. Default же ссылается не на значение (не на value), а на индекс у Symbol. Т.е. Default==1 будет означать Symbol1.

    Таких мест довольно много:
    Код:
    $ grep Default=0 * | grep Param
    hardware.cfg:Param6=Name='Frame oriented',Section=FrameOriented,Access=RW,Visible=True,Default=0
    hardware.cfg:Param7=Name='Framing time ms',Section=Timeout,Access=RW,Visible=True,Default=0
    hardware.cfg:Param6=Name='Frame oriented',Section=FrameOriented,Access=RW,Visible=True,Default=0
    hardware.cfg:Param7=Name='Framing time ms',Section=Timeout,Access=RW,Visible=True,Default=0
    hardware.cfg:Param6=Name='Frame oriented',Section=FrameOriented,Access=RW,Visible=True,Default=0
    hardware.cfg:Param7=Name='Framing time ms',Section=Timeout,Access=RW,Visible=True,Default=0
    hardware.cfg:Param7=Name='Framing time ms',Section=Timeout,Access=RW,Visible=True,Default=0
    hardware.cfg:Param1=Name='First input',Section=IOMask,Default=0
    hardware.cfg:Param1=Name='First input',Section=IOMaskHiSpeed,Default=0
    hardware.cfg:Param1=Name='Register Adress',Section=ModbusAddress,Access=RW,Visible=True,Default=0
    hardware.cfg:Param1=Name='Register Adress',Section=ModbusAddress,Access=RW,Visible=True,Default=0
    owen.cfg:Param6=Name='Float type',Section=float_type,Access=RW,Visible=True, Default=0
    owen.cfg:Param6=Name='Float type',Section=float_type,Access=RW,Visible=True, Default=0
    owen.cfg:Param6=Name='Less time field',Section=less_time_field,Access=RW,Visible=True, Default=0
    owen.cfg:Param6=Name='Float type',Section=float_type,Access=RW,Visible=True, Default=0
    owen.cfg:Param6=Name='Float type',Section=float_type,Access=RW,Visible=True, Default=0
    owen.cfg:Param6=Name='Less time field',Section=less_time_field,Access=RW,Visible=True, Default=0
    owen.cfg:Param6=Name='Float type',Section=float_type,Access=RW,Visible=True, Default=0
    owen.cfg:Param6=Name='Float type',Section=float_type,Access=RW,Visible=True, Default=0
    owen.cfg:Param6=Name='Less time field',Section=less_time_field,Access=RW,Visible=True, Default=0
    owen.cfg:Param6=Name='Float type',Section=float_type,Access=RW,Visible=True, Default=0
    owen.cfg:Param6=Name='Float type',Section=float_type,Access=RW,Visible=True, Default=0
    owen.cfg:Param6=Name='Less time field',Section=less_time_field,Access=RW,Visible=True, Default=0

    Discrete outputs 10 bit:
    Код:
    [Module.ID131]
    Name= Discrete outputs - 10
    Id=131
    ...
    Channel1=Section=ByteOutput,Comment='8 discrete outputs'
    Channel2=Section=ByteOutput,Comment='2 discrete outputs'
    Для channel2 специально используется тот же тип ByteOutput? Может, стоило сделать ByteOutput2, чтобы не приходилось гадать какие из 8-и бит реально используются, а какие нет?
    Изменение типа канала потребует обновления прошивки? Или нет?


    При добавлении Modbus (Master) по умолчанию выбирается "Debug RS-232". Это реально частый случай?
    Может, стоит изменить значение по умолчанию на RS-485-1?


    Довольно много опечаток:
    Код:
    [Parameter.CommunicationSpeed]
    Id=1000
    Type=INT
    Class=Symbolic
    Symbol1=11520
    TTHelp=Speed of commections in bits/sec
    115200 же. Нолик потерялся
    commections -> connection

    Код:
    [Module.Statistic_ID002]
    Name=Statistic
    Id=002
    Channel6=Section=WordInput,Comment='Free processor resourse mks in 1 cycle'
    resourse -> resource
    mks -> us. И много где ещё используется "mks", хотя подразумевается μs. мкс -- это русское сокращение. А по-английски будет μs.

    Код:
    [Module.ID571]
    Name=PWM
    Id=571
    Channel1=Section=WordOutput,Comment='PWM 1 power'
    Channel2=Section=WordOutput,Comment='PWM 2 power'
    Channel3=Section=WordOutput,Comment='PWM 3 power'
    Channel4=Section=WordOutput,Comment='PWM 4 power'
    Channel5=Section=DWordOutput, Comment='PWM 1 period'
    Channel6=Section=DWordOutput, Comment='PWM 1 period'
    Channel7=Section=DWordOutput, Comment='PWM 1 period'
    Channel8=Section=DWordOutput, Comment='PWM 1 period'
    Должно быть "PWM 1 period", "PWM 2 period", "3 period" и т.п.


    Preset singl register -> Preset single register
    Последний раз редактировалось Владимир Ситников; 07.03.2016 в 12:54. Причина: указал версию target'а

  2. #2
    Пользователь Аватар для capzap
    Регистрация
    25.02.2011
    Адрес
    Киров
    Сообщений
    10,224

    По умолчанию

    а какие проблемы, поправили да запустили плк с новым таргетом, протестировали и выложили сюда, вот мол рабочая версия, лучше старой
    Bad programmers worry about the code. Good programmers worry about data structures and their relationships

    среди успешных людей я не встречала нытиков
    Барбара Коркоран

  3. #3

    По умолчанию

    Цитата Сообщение от capzap Посмотреть сообщение
    а какие проблемы, поправили да запустили плк с новым таргетом, протестировали и выложили сюда, вот мол рабочая версия, лучше старой
    Это без проблем.

    Вопрос лишь в том, что на эту тему думает ОВЕН.
    Может, они скажут, что они не возьмут эти изменения: "нужно тестировать, а тестировщики на ближайшую пятилетку заняты", или "во всю идёт переход ПЛК110 на КДС3, и КДС2 таргеты уже не нужны", или ещё что-нибудь.

  4. #4
    Пользователь Аватар для rovki
    Регистрация
    03.01.2010
    Адрес
    Чехов
    Сообщений
    12,113

    По умолчанию

    + почет и уважение ,а не просто соринки в чужом глазу .
    Главные тестировщики - пользователи имхо за ними слово
    электронщик до мозга костей и не только

  5. #5
    Пользователь
    Регистрация
    11.01.2009
    Адрес
    Кострома
    Сообщений
    3,144

    По умолчанию

    Цитата Сообщение от vladimirisitnikov Посмотреть сообщение
    Это без проблем.

    Вопрос лишь в том, что на эту тему думает ОВЕН.
    Может, они скажут, что они не возьмут эти изменения: "нужно тестировать, а тестировщики на ближайшую пятилетку заняты", или "во всю идёт переход ПЛК110 на КДС3, и КДС2 таргеты уже не нужны", или ещё что-нибудь.
    Выложив недостатки таргета вы ни указали версию и что за ПЛК.
    В целом ПЛК 110 стабильный прибор и у пользователей проблем не вызывает.
    Какой смысл выкладывать на форуме выдержки кода,
    часть из которых по вашему мнению не целесообразо написана,
    но это уж на усмотрение писавшего. Вопрос о правильно-не правильно
    в данном случае не корректен, т. к. неизвестен замысел и цель автора.
    А реализация тем более, как удобнее было писавшему.
    Главное результат- "прибор стабилен".
    Ну и 100% пользователей это вообще не нужно.
    С таким же успехом можно писать в администрацию к примеру московской области.

    Тот же сименс к примеру тоже имеет много неочень удачных изделий.
    взять к примеру коммуникационные модули ET200 3 серии.
    (сейчас 5-я). Так вот все эти миллионы изделий через год эксплуатации
    оказались на помойке.
    У S300 тоже цсд файлы дорабатывались и версии прошивок менялись..

  6. #6
    Пользователь
    Регистрация
    11.01.2009
    Адрес
    Кострома
    Сообщений
    3,144

    По умолчанию

    Цитата Сообщение от rovki Посмотреть сообщение
    + почет и уважение ,а не просто соринки в чужом глазу .
    Главные тестировщики - пользователи имхо за ними слово
    Анатолий, на основании чего вывод?
    много непонятного текста?
    вы же в этом не специалист

  7. #7
    Пользователь Аватар для rovki
    Регистрация
    03.01.2010
    Адрес
    Чехов
    Сообщений
    12,113

    По умолчанию

    Цитата Сообщение от lara197a Посмотреть сообщение
    Анатолий, на основании чего вывод?
    много непонятного текста?
    вы же в этом не специалист
    Вывод прост и вы его уже сами сделали .Нормально работает ,а почерк это черта автора.Поиск помарок (соринок) в чужом коде (глаз) это то ,о чем я и сказал .Но если человек (врач) поможет вытащить соринку из глаза ,то ему хвала ,а иначе это по другому называется ,имхо.Для этого не надо быть крутым специалистом ,достаточно что писал на ассемблере довольно большие проекты в свое время.
    электронщик до мозга костей и не только

  8. #8

    По умолчанию

    Цитата Сообщение от lara197a Посмотреть сообщение
    Выложив недостатки таргета вы ни указали версию и что за ПЛК.
    В целом ПЛК 110 стабильный прибор и у пользователей проблем не вызывает.
    Какой смысл выкладывать на форуме выдержки кода,
    Очень простой смысл: чтобы ОВЕН'овцы прочитали, приняли в работу, высказали мнение и т.п.

    Цитата Сообщение от lara197a Посмотреть сообщение
    Ну и 100% пользователей это вообще не нужно.
    Тут неправда ваша.
    Во-первых, ошибки (в первую очередь, логические) мешают мне загружать эти target'ы. Если что, я их не КДС'ом загружаю.
    Во-вторых, явные опечатки режут глаз. Из правил форума, видно, что ОВЕН ценит "культурное литературное наследие русского языка". Логично предположить, что и опечатки в английском не специально сделали.
    В-третьих, "некорректно написаный target файл" может внезапно перестать работать при очередном обновлении КДС. Одна версия КДС "проглатывает белиберду", а следующая уже может и перестать.

    В четвёртых, логические ошибки могут приводить к вообще необъяснимым эффектам.
    Например (это из 3.09 PLC110_32M_v2):
    Код:
    [Module.DEBUG_USART]
    Name=Debug RS-232
    Param3=Name='Data bits',Section=BitLength,Access=R0,Visible=True
    Внимание, вопрос: можно ли менять значение параметра "Data bits" на закладке Modbus (Master) -> Debug RS-232 -> Module Parameters?
    Если что, то там написано Access=R<ноль>, т.е. вместо буквы O там цифра 0.


    Скачал target'ы 3.09 для PLC110_32_v2. Да, немного исправлено (например, 11520 поправили на 115200), а многое осталось.
    Те же самые mks в полный рост.

    Ещё удивительно в v2 версии то, что почти у каждого модуля появился параметр "Name", который допускает редактирование:
    Код:
    Param1=Name='Name',Section=ChannellName,Access=RW,Visible=True,Default='Discrete inputs'
    Честно говоря, вообще непонятно зачем везде надобавляли этих "Name" параметров, но если уж добавили, то ради чего-то?
    Комментария что-ли мало?

    Цитата Сообщение от lara197a Посмотреть сообщение
    Вопрос о правильно-не правильно
    в данном случае не корректен, т. к. неизвестен замысел и цель автора.
    А реализация тем более, как удобнее было писавшему.
    Не путайте тёплое с мягким. Если автор имел реальный замысел или цель использовать цифру 0 вместо буквы O при написании Access=RO, в таких случаях принято в комментариях указывать "да, тут должно быть RO, но из-за того-то и того-то пишем нолик".
    Нигде таких комментариев нет -- значит, никакого замысла и не было. Просто тот, кто составлял target, опечатался и всего делов.

    Если конкретная ошибка не воспроизвелась у вас, то это не значит, что её нет.


    Цитата Сообщение от lara197a Посмотреть сообщение
    С таким же успехом можно писать в администрацию к примеру московской области.
    Опять путаете.

    Смотрите пункт 1.1 правил форума:
    1.1. Форум предназначен для общения по теме АСУ ТП и по продукции ОВЕН
    Была бы JIRA для отслеживания проблем/вопросов -- завёл бы туда тикет.
    Если бы target'ы были на github -- завёл бы pull request с исправлением опечаток.


    "администрацию к примеру московской области" != "официальный канал ОВЕН по связям с общественностью"

    Так вот все эти миллионы изделий через год эксплуатации
    оказались на помойке.
    У S300 тоже цсд файлы дорабатывались и версии прошивок менялись..
    Не пойму ход мысли. Имеется ввиду "доработкой target'ов убили изделие"? Или "кривому изделию target файлы как припарка"? Ещё что-то?

  9. #9

    По умолчанию

    В правильном направлении идете, vladimirisitnikov. В разработках "ОВЕН" есть чего почистить.
    Последний раз редактировалось Вольд; 07.03.2016 в 14:14.

  10. #10
    Пользователь
    Регистрация
    11.01.2009
    Адрес
    Кострома
    Сообщений
    3,144

    По умолчанию

    vladimirisitnikov
    в контактах есть прямые адреса ответственных лиц Овен.
    Логичнее писать прямо им.

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

Страница 1 из 4 123 ... ПоследняяПоследняя

Похожие темы

  1. Ответов: 4
    Последнее сообщение: 04.09.2015, 18:47
  2. Target 2.10 или 2.11
    от Boris_K в разделе ПЛК1хх
    Ответов: 1
    Последнее сообщение: 27.06.2014, 15:28
  3. target файл ПЛК110-30-L
    от Артем91 в разделе ПЛК1хх
    Ответов: 1
    Последнее сообщение: 19.01.2014, 14:02
  4. перевод строки в файле
    от Назаров Александр в разделе ПЛК1хх
    Ответов: 2
    Последнее сообщение: 29.03.2010, 10:08

Ваши права

  • Вы не можете создавать новые темы
  • Вы не можете отвечать в темах
  • Вы не можете прикреплять вложения
  • Вы не можете редактировать свои сообщения
  •