Опечатки в 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

[capzap]

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

[Владимир Ситников]

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

Это без проблем.

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

[lara197a]

Это без проблем.

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

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

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

[Владимир Ситников]

Выложив недостатки таргета вы ни указали версию и что за ПЛК.
В целом ПЛК 110 стабильный прибор и у пользователей проблем не вызывает.
Какой смысл выкладывать на форуме выдержки кода,

Очень простой смысл: чтобы ОВЕН'овцы прочитали, приняли в работу, высказали мнение и т.п.

Ну и 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" параметров, но если уж добавили, то ради чего-то?
Комментария что-ли мало?

Вопрос о правильно-не правильно
в данном случае не корректен, т. к. неизвестен замысел и цель автора.
А реализация тем более, как удобнее было писавшему.

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

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

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

Опять путаете.

Смотрите пункт 1.1 правил форума:

1.1. Форум предназначен для общения по теме АСУ ТП и по продукции ОВЕН

Была бы JIRA для отслеживания проблем/вопросов -- завёл бы туда тикет.
Если бы target'ы были на github -- завёл бы pull request с исправлением опечаток.


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

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

Не пойму ход мысли. Имеется ввиду "доработкой target'ов убили изделие"? Или "кривому изделию target файлы как припарка"? Ещё что-то?

[Sergey666]

Это без проблем.

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

Какой умненький мальчик...
полез , куда ... нормальные люди не лезут , чето наковырял , опять-же спорно , ведь работает . Обычно таких умных мальчиков бьют в челюсть , а потом ногой в живот ... и после этого умный мальчик иногда становится обычным человеком .
Да ПЛК110 тормоз , хуже ПЛК100 и что ? Не в мнимых очепятках в таргете дело .
Подайте что-ли резюме в Овен , может соратником Филоненко станете , ... если жубы жмут

[Василий Кашуба]

Какой умненький мальчик...
полез , куда ... нормальные люди не лезут , чето наковырял , опять-же спорно , ведь работает . Обычно таких умных мальчиков бьют в челюсть , а потом ногой в живот ... и после этого умный мальчик иногда становится обычным человеком .
Да ПЛК110 тормоз , хуже ПЛК100 и что ? Не в мнимых очепятках в таргете дело .
Подайте что-ли резюме в Овен , может соратником Филоненко станете , ... если жубы жмут

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

[rovki]

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

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

[Вольд]

Какой умненький мальчик...
полез , куда ... нормальные люди не лезут , чето наковырял , опять-же спорно , ведь работает . Обычно таких умных мальчиков бьют в челюсть , а потом ногой в живот ... и после этого умный мальчик иногда становится обычным человеком .
Да ПЛК110 тормоз , хуже ПЛК100 и что ? Не в мнимых очепятках в таргете дело .
Подайте что-ли резюме в Овен , может соратником Филоненко станете , ... если жубы жмут

Опять поддатый на форум заявился.

[rovki]

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