Багрепорт и пожелания

Любые вопросы, относящиеся к приложению FORScan
Shtep
Posts: 6
Joined: Fri Nov 14, 2014 12:38 pm
Vehicle: Mazda6 2.0 2008

Багрепорт и пожелания

Post by Shtep »

V 2.1.22b for Windows

1. AFR_ACT принимает значение 1 или 2 ( такое не горит, это практически одно топливо)
2. EQ_RAT11 больше похоже на AFR (на ХХ 14.8 ЛЯМБДЫ гореть точно не будет)
3. В заголовке в CSV русские буквы в неправильной кодировке, не ДОС и не ВИН...
4. Хорошо бы иметь возможность встроенной трансляции параметров из текста в цифру, вместо On - 1, вместо Off - 0. Для того же экуедита приходится руками заменять, что не есть удобно.
5. Лог писать хорошо бы с момента получения полного комплекта всех выбранных перемешек - "елку" на старте приходится каждый раз руками удалять.
6. В настройку хорошо бы вынести выбор разделителя для CSV (; или ,)
7. Периодически происходит остановка опроса примерно на 0,5-1 сек. Толи программа, толи адаптер. ( Но на двух разных повторяется)

Будем смотреть дальше.
FORScan
Site Admin
Posts: 2970
Joined: Fri Jun 13, 2014 2:21 am

Re: Багрепорт и пожелания

Post by FORScan »

Shtep wrote: 1. AFR_ACT принимает значение 1 или 2 ( такое не горит, это практически одно топливо)
В данном PID используется некорректная формула - результат округляется до целой части. Т.е. например 1.5 округляется до 2. Мы это исправим.
Shtep wrote: 2. EQ_RAT11 больше похоже на AFR (на ХХ 14.8 ЛЯМБДЫ гореть точно не будет)
Для разных моделей интерпретация PID может отличаться, поэтому нужно разбираться. Присылайте, пожалуйста, логи на почту (инструкция на странице Контакты)
Shtep wrote: 3. В заголовке в CSV русские буквы в неправильной кодировке, не ДОС и не ВИН...
Используется кодировка UTF-8, которая уже несколько лет как стандарт для мульти-язычных приложений. FORScan поддерживает не только русский, но также и польский, и венгерский ,и немецкий и т.д., у которых свои собственные ДОС и ВИН.
4. Хорошо бы иметь возможность встроенной трансляции параметров из текста в цифру, вместо On - 1, вместо Off - 0. Для того же экуедита приходится руками заменять, что не есть удобно.
Просьба чуть подробнее описать пример использования - не совсем понятно зачем это надо.
5. Лог писать хорошо бы с момента получения полного комплекта всех выбранных перемешек - "елку" на старте приходится каждый раз руками удалять.
Не поняли, о чем речь.
6. В настройку хорошо бы вынести выбор разделителя для CSV (; или ,)
Запятые использовать не рекомендуется, т.к. в некоторых языках (кстати, в русском тоже) запятая используется как разделитель между целой и дробной частями.
Shtep wrote: 7. Периодически происходит остановка опроса примерно на 0,5-1 сек. Толи программа, толи адаптер. ( Но на двух разных повторяется)
Не знаем, что может быть. Нужно смотреть внутренние логи (с сессией, в которых проблема была). См. комментарий к п.2.
Shtep
Posts: 6
Joined: Fri Nov 14, 2014 12:38 pm
Vehicle: Mazda6 2.0 2008

Re: Багрепорт и пожелания

Post by Shtep »

Писал писал и все пропало.
Кратко. логи скину как до компа доберусь.
п.3. не весь софт понимает utf. По крайней мере экуэдит не разумеет. А это на данный момент один из самых удобных и используемых инструментов анализа логов.
п.4. при построении графиков, использовать значения 0,1,2 - без проблем, а вот Open loop CL - Drive e.t.c. довольно проблематично. Приходится руками заменять на что-то цифровое. Опять же, кроме экселя есть масса инструметов для анализа :-)
п.5. -первые строки лога выглядят как первая строка - одно значение и прочерки, вторая - два значения и прочерки и т.д. и так пока все. ПИДы не опросит. в итоге пока все не опросит - лог полупустой. Вобщем то проблем нет, но коряво.
п.6. Я тоже так раньше думал. Разделитель задается в языковых настройках. А вот анализаторы логов часто не имеют настроек. В итоге нам пришлось в своем софте принять реалии, а не гнуть их под себя. Реалии - кодировка win и разделитель - как в языке задано. По факту 90% - десятичная точка, а разделитель в csv - запятая. так что настройка не лишняя.
п.7 - ок. сделаем.
Shtep
Posts: 6
Joined: Fri Nov 14, 2014 12:38 pm
Vehicle: Mazda6 2.0 2008

Re: Багрепорт и пожелания

Post by Shtep »

Про "елку" в CSV (п.5):

0;1;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-
0;1;786;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-
35;1;786;Off;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-
74;1;786;Off;Off;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-
117;1;786;Off;Off;76;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-
152;1;785;Off;Off;76;0.00;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-;-

Про UTF - эксель под ХР крякозябры тоже рисует, и хочет видеть в качестве разделителя запятую. ;-)

Про просмотр логов - на картинке выделил:
1. Пропуски по связи, с чем связаны - пока непонятно. выглядит как остановка, при этом потери связи и ошибок нет. Изучаю вопрос дальше, попробую на USB шнурке проверить.
2. Отображение крякозябр
3. На примере параметра BOO - так трактуются текстовые значения Off и On. Про Open Loop - и говорить не приходится ;-)
Attachments
Безымянный.PNG
Безымянный.PNG (108.84 KiB) Viewed 12718 times
Shtep
Posts: 6
Joined: Fri Nov 14, 2014 12:38 pm
Vehicle: Mazda6 2.0 2008

Re: Багрепорт и пожелания

Post by Shtep »

И еще странную шляпу заметил Выделил - это явно не единичный сбой - данные плавно уходят.
Attachments
Безымянный2.PNG
Безымянный2.PNG (52.51 KiB) Viewed 12718 times
FORScan
Site Admin
Posts: 2970
Joined: Fri Jun 13, 2014 2:21 am

Re: Багрепорт и пожелания

Post by FORScan »

Shtep wrote: п.3. не весь софт понимает utf. По крайней мере экуэдит не разумеет. А это на данный момент один из самых удобных и используемых инструментов анализа логов.
FORScan хранит все ресурсы в UTF, поэтому придется делать специально конвертацию, причем в две кодировки для каждого языка, т.е. поддерживать на данный момент почти полтора десятка кодировок (а в будущем кол-во языков будет расти). Потом все это поддерживать, таскать за собой с платформы на платформу и все это только для того, что бы удовлетворить парсер логов, который не поддерживает UTF. Вообщем, можем предложить альтернативный способ решения проблемы - создать специальные скрипты для обработки файлов между FORScan и вашим парсером, который будет делать все необходимые действия по преобразованию. Для перекодировки можно использовать утилиту iconv - очень мощный и удобный инструмент. Если нужно, можем помочь с написанием скриптов.
Shtep wrote: п.4. при построении графиков, использовать значения 0,1,2 - без проблем, а вот Open loop CL - Drive e.t.c. довольно проблематично. Приходится руками заменять на что-то цифровое. Опять же, кроме экселя есть масса инструметов для анализа :-)
Мы планировали в будущей версии добавить для всех PID т.н. Raw конвертер, т.е. возможность выводить PID в том виде, в каком он пришел от модуля, без применения каких либо преобразований. Это должно решить эту проблему.
Shtep wrote: п.5. -первые строки лога выглядят как первая строка - одно значение и прочерки, вторая - два значения и прочерки и т.д. и так пока все. ПИДы не опросит. в итоге пока все не опросит - лог полупустой. Вобщем то проблем нет, но коряво.
Понятно. Такую опцию (не выводить строку, если не собраны данные со всех PID) сделать можно, но считаем это мелкой мелочью. В список задач поставим с маленьким приоритетом. Как временную меру, можно рекомендовать использование промежуточных скриптов (см. выше).
Shtep wrote: п.6. Я тоже так раньше думал. Разделитель задается в языковых настройках. А вот анализаторы логов часто не имеют настроек. В итоге нам пришлось в своем софте принять реалии, а не гнуть их под себя. Реалии - кодировка win и разделитель - как в языке задано. По факту 90% - десятичная точка, а разделитель в csv - запятая. так что настройка не лишняя.
По разделителю CSV - согласны, возможность его задавать необходима. Постараемся добавить в одной из будущих версий.
Shtep wrote: 1. Пропуски по связи, с чем связаны - пока непонятно. выглядит как остановка, при этом потери связи и ошибок нет
А как выглядит CSV файл для таких "пропусков"?
Shtep wrote: И еще странную шляпу заметил Выделил - это явно не единичный сбой - данные плавно уходят.
Такое бывало на старых версиях FORScan, но уже подобных жалоб не было. В данном случае нужно смотреть не только CSV, но и логи FORScan что бы найти проблемное место.
Shtep
Posts: 6
Joined: Fri Nov 14, 2014 12:38 pm
Vehicle: Mazda6 2.0 2008

Re: Багрепорт и пожелания

Post by Shtep »

С логами проблему решил - просто нарисовал себе конвертор. На самом деле там посути 2 строки добавить - из UTF в строку и из строки в нужную кодовую страницу, и настройку конечной кодировки. Ну и заодно оттранслил все в цифры. С пропусками - csv выглядит как обычно. просто "дырка" во времени. По провалу температуры - погляжу еще, но это точно какая то лажа. Впрочем обороты в 70-80 тоже не падают, там синхронизация оборотов с 200, небось только работает... но ведь как плавненько... А вот сырые данные и возможность настройки их декодирования - это будет гуд. Кому то цикловой расход привычней в гр. сек, а кому то в кг. час... кому то проще с лямбдой оперировать, а кому то с AFR...
FORScan
Site Admin
Posts: 2970
Joined: Fri Jun 13, 2014 2:21 am

Re: Багрепорт и пожелания

Post by FORScan »

Shtep wrote: С пропусками - csv выглядит как обычно. просто "дырка" во времени. По провалу температуры - погляжу еще, но это точно какая то лажа
По описанию эти две проблемы действительно похожи на проблему FORScan, с ней нужно разбираться. Но для этого, повторимся, нужны логи и вывод (в виде CSV - нормально). По опыту, поиск подобных проблем занятие очень длинное и неблагодарное, сильно зависящее от конкретных условий, поэтому браться за это без логов даже смысла нет.
Shtep
Posts: 6
Joined: Fri Nov 14, 2014 12:38 pm
Vehicle: Mazda6 2.0 2008

Re: Багрепорт и пожелания

Post by Shtep »

Логи в почте. Пересчет AFR поправили?
FORScan
Site Admin
Posts: 2970
Joined: Fri Jun 13, 2014 2:21 am

Re: Багрепорт и пожелания

Post by FORScan »

Shtep wrote:Логи в почте. Пересчет AFR поправили?
На почту ответили. Пересчет AFR поправили, но к сожалению в версию 2.1.23 (выданную на прошлой неделе) это изменение попасть не успело. Если очень надо, можем сделать отдельную тестовую сборку с этим изменением.
Post Reply