Опечатки в target файле для ПЛК110 [Архив] - Форум АСУ ТП. ОВЕН форум

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

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

06.03.2016, 23:32

Копался в 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=Tr ue,Default=0
hardware.cfg:Param1=Name='Register Adress',Section=ModbusAddress,Access=RW,Visible=Tr ue,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=T rue, 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=T rue, 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=T rue, 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=T rue, 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

07.03.2016, 07:07

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

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

07.03.2016, 08:58

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

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

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

rovki

07.03.2016, 08:59

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

lara197a

07.03.2016, 11:18

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

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

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

lara197a

07.03.2016, 11:18

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

Анатолий, на основании чего вывод?
много непонятного текста?
вы же в этом не специалист

rovki

07.03.2016, 12:49

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

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

07.03.2016, 13:39

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

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

Ну и 100% пользователей это вообще не нужно.
Тут неправда ваша.
Во-первых, ошибки (в первую очередь, логические) мешают мне загружать эти target'ы. Если что, я их не КДС'ом загружаю (http://www.owen.ru/forum/showthread.php?t=23013).
Во-вторых, явные опечатки режут глаз. Из правил форума, видно, что ОВЕН ценит "культурное литературное наследие русского языка". Логично предположить, что и опечатки в английском не специально сделали.
В-третьих, "некорректно написаный 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 файлы как припарка"? Ещё что-то?

Вольд

07.03.2016, 14:11

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

lara197a

07.03.2016, 16:50

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

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

Sergey666

08.03.2016, 00:56

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

08.03.2016, 10:17

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

rovki

08.03.2016, 10:33

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

ASo

08.03.2016, 10:46

Потому, что я бы выложил прямо на форуме подправленный файл. И он пойдет в работу.
Для справки - я выкладывал подправленный *.inf для драйвера СПК10*, устраняющий проблему в XP после некоторых SP и HF. Почемуто мою правку по словам администрации включили в новый дистриб, а как времянку - рекомендовали пользоваться с форума.

Просто людей, желающих тупо постебаться, хотя и по делу - не любят.

rovki

08.03.2016, 10:59

ASo

08.03.2016, 11:04

При чем тут я и согласие???
Ровно тоже было предложено в посте №2.

Недоволен - критикуй, критикуешь - предлагай, предлагаешь - делай, делаешь - отвечай (с) Сталин И.В.

Вольд

08.03.2016, 11:04

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

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

rovki

08.03.2016, 11:09

При чем тут я и согласие???
Ровно тоже было предложено в посте №2.

Недоволен - критикуй, критикуешь - предлагай, предлагаешь - делай, делаешь - отвечай (с) Сталин И.В.
Так это я о своем согласии с вашим мнением ,что очень редко бывает .

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

08.03.2016, 11:36

Потому, что я бы выложил прямо на форуме подправленный файл. И он пойдет в работу.

Есть такая штука, лицензионное соглашение называется.
Оно либо допускает, либо не допускает модификацию и выкладывание.
Специально про него серия "Человекайпадоножка".

У target файлов лицензионного соглашения вообще нет. Русская рулетка в чистом виде.
Была бы лицензия обозначена явно -- можно было бы заходить с козырей и выкладывать исправленный файл.

Нет лицензии -- ждём ОВЕН.

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

ASo

08.03.2016, 11:44

Была бы лицензия обозначена явно -- можно было бы заходить с козырей и выкладывать исправленный файл.

Нет лицензии -- ждём ОВЕН.
Если бы была бы лицензия - то "копание в файле" научно называемое реинжинирингом прямо запрещено лицензией и ГК.
Поскольку лицензии нет - правьте и выкладывайте.

Телепаты в отпуске. Если вы просто постебаться -- не вопрос. Но, таких, как верно замечено, не любят.
Понимаете ли в чем дело... Здесь тусуются практики. Ремесленники, если хотите.
Поэтому их критерий - работает - не лезь. Ну, да, очепатки, работе не мешает - пофигу.
Вы, как я понимаю, хотите создать альтернативное IDE КДС 2. Только вот Ваша работа нужна хорошо если 5% всех пользователей КДС 2 (это оценка сверху) а из местных завсегдатаев - не нужна никому. Поэтому такие комментарии.
Тут действительно, ничего личного, только бизнес в хорошем смысле этой фразы.

Вольд

08.03.2016, 11:56

Поэтому их критерий - работает - не лезь. Ну, да, очепатки, работе не мешает - пофигу..

С чего вы взяли что не мешает. Не понятных глюков в продуктах "ОВЕН" хватает.

vladimirisitnikov, не надо слушать скептиков, нытиков и прочих. Копайте.

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

08.03.2016, 12:09

Если бы была бы лицензия - то "копание в файле" научно называемое реинжинирингом прямо запрещено лицензией и ГК.

Когда последний раз ГК открывали? Лицензии, как уже убедились, пока нет. Да и "копание в файле" она не может запретить, т.к. оно разрешено в ГК.

ГК РФ Статья 1280. Право пользователя программы для ЭВМ и базы данных
1. Лицо, правомерно владеющее экземпляром программы для ЭВМ или экземпляром базы данных (пользователь), вправе без разрешения автора или иного правообладателя и без выплаты дополнительного вознаграждения:
1) осуществлять действия, необходимые для функционирования программы для ЭВМ или базы данных (в том числе в ходе использования в соответствии с их назначением), включая запись и хранение в памяти ЭВМ (одной ЭВМ или одного пользователя сети), внесение в программу для ЭВМ или базу данных изменений исключительно в целях их функционирования на технических средствах пользователя, исправление явных ошибок, если иное не предусмотрено договором с правообладателем;
(пп. 1 в ред. Федерального закона от 12.03.2014 N 35-ФЗ)
(см. текст в предыдущей редакции)
2. Лицо, правомерно владеющее экземпляром программы для ЭВМ, вправе без согласия правообладателя и без выплаты дополнительного вознаграждения изучать, исследовать или испытывать функционирование такой программы в целях определения идей и принципов, лежащих в основе любого элемента программы для ЭВМ, путем осуществления действий, предусмотренных подпунктом 1 пункта 1 настоящей статьи.

Переводя с русского на русский, ГК явно разрешает reverse engineering.
Более того, даже если "в лицензии будет сказано, что запрещено изучение", то такой пункт считается ничтожным, т.к. он противоречит ГК.

Другое дело, что прав "выкладывать на форумах" модифицированное ПО ГК не даёт. Как минимум для этого и нужна лицензия.

ASo

08.03.2016, 12:12

если иное не предусмотрено договором с правообладателем == лицензия, как публичный договор с правообладателем.

Spawn

09.03.2016, 12:35

Оно либо допускает, либо не допускает модификацию и выкладывание.

Х-м...можно, конечно, заморочиться и указать все (или часть, тут уже по Вашему желанию), на Ваш взгляд, некорректные места, с комментариями что необходимо изменить, опять же на Ваш взгляд, в том или ином случае. Этим можно убить сразу 3-х зайцев: 1 - Вы не вносите изменения; 2 - тут, конечно, смотря что понимать под словом "выкладывания", оригинально файла, файла с внесенными изменениями, или только, измененных или нет, фрагментов файла; 3 - все, кто пожелают вносить те или иные изменения, пускай делают это сами.

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

17.03.2016, 10:42

Прошла неделя, support@owen.ru молчит, на вопрос "какая лицензия у target файлов, и можно ли их выложить на github" не отвечают.

Малышев Олег

17.03.2016, 11:07

Когда скачивали таргет-файлы - было окно с соглашением?

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

17.03.2016, 11:14

Когда скачивали таргет-файлы - было окно с соглашением?

1) Не было. Качал отсюда: http://www.owen.ru/catalog/programmiruemij_logicheskij_kontroller_oven_plk110/87310138
2) Лицензия в составе самого zip тоже не помешает

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

22.03.2016, 10:44

Прошло 2 недели, support@owen.ru молчит, на вопрос "какая лицензия у target файлов, и можно ли их выложить на github" не отвечают.

ASo

22.03.2016, 10:57

Люди делом заняты, им не до баловства.

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

22.03.2016, 12:50

Люди делом заняты, им не до баловства.

Не вам судить баловство это или нет. Пусть представитель ОВЕН хоть что-то скажет.

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

И я не просто про о коне в вакууме вопрос задаю. У меня реально существующий ПЛК110 М02.

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

22.03.2016, 16:02

По вопросам
1) Допускается ли создание "производных произведений" на основе
"target файлов с сайта owen.ru"? (модификация файлов, исправление
опечаток)

2) Допускается ли распространение этих самых "производных
произведений"? (выкладывание на форуме owen.ru, выкладывание на
github.com)

получен такой ответ:

1) таргет файл предоставляется абсолютно бесплатно. можете использовать неограниченно много раз
2) Да, конечно можно распространять
С уважением, инженер группы технической поддержки Дударев Евгений Геннадиевич

На этом спасибо. Надеюсь, в очередных выпусках target.zip'ов в составе будет license.txt