Page 33 of 39

Re: Всякие глупые вопросы, терзающие меня

Posted: Sat May 13, 2017 10:52 am
by puff
Я сам пугаюсь. На картинке выше настроены четыре канала, из которых первый, третий и четвертый можно редактировать , а второй - не активен..

Sent from my XT1058 using Tapatalk

Re: Всякие глупые вопросы, терзающие меня

Posted: Sat May 13, 2017 11:29 am
by AndreyB
Видимо я вижу доступный для редактирования 8ой канал, а ты не видишь?

very limited telepathic abilities - please post logs & tunes where appropriate

Re: Всякие глупые вопросы, терзающие меня

Posted: Sat May 13, 2017 12:32 pm
by puff
кажется, дошло. очевидно, это связано с
engineConfiguration->specs.firingOrder = FO_1_8_4_3_6_5_7_2;
и он сответственно пуляет в таком цикле:
ignitionPins[0] -> ignitionPins[7] -> ignitionPins[3] - > ignitionPins[2]

и похоже, что надо переиначить, вместо:

Code: Select all

  boardConfiguration->ignitionPins[0] = GPIOE_8; // Frankenstein: low side - out #8
    boardConfiguration->ignitionPins[1] = GPIOE_10; // Frankenstein: low side - out #8
    boardConfiguration->ignitionPins[2] = GPIOE_12; // Frankenstein: low side - out #8
    boardConfiguration->ignitionPins[3] = GPIOE_14; // Frankenstein: low side - out #8
сделать

Code: Select all

  boardConfiguration->ignitionPins[0] = GPIOE_8; // Frankenstein: low side - out #8
    boardConfiguration->ignitionPins[7] = GPIOE_10; // Frankenstein: low side - out #8
    boardConfiguration->ignitionPins[3] = GPIOE_12; // Frankenstein: low side - out #8
    boardConfiguration->ignitionPins[2] = GPIOE_14; // Frankenstein: low side - out #8
как-то так.
плюс, в нужно степпере поправить пины - видимо, зимой я правил у себя локально (степпер же работал!) - а на сервере не обновил.

Re: Всякие глупые вопросы, терзающие меня

Posted: Tue May 16, 2017 12:37 pm
by AndreyB
Да, всё именно так!

Re: Всякие глупые вопросы, терзающие меня

Posted: Tue May 16, 2017 2:02 pm
by puff
оно тоже есть в пулреквесте!

правда, тут освежил в памяти - из0-за ограничений дискавери не нужно бы использовать GPIOE_0 и GPIOE_1. (а они сейчас в файле конфига на сервере)
если от GPIOE_1 можно вовсе откааться (он на лампочку чекэнджин, а она по canbus теперь управляется, правда, наверное, надо интегрировать это в код), то GPIOE_0 - это бензонасос в моем файле конфигурации - его бы нужно снести на следующий доступный пин...

Re: Всякие глупые вопросы, терзающие меня

Posted: Tue May 16, 2017 10:56 pm
by AndreyB
puff wrote:оно тоже есть в пулреквесте!
И это плохо! Потому что получается у нас два разных изменения смешаны вместе, и это не правильно!

Закоммитил изменённую версию конфига - так что теперь наверное будет конфликт.

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

Во-первых, форматирование кода - ну это будем считать придирки.

Я не понимаю назначения переменных - с такими именами я переменные принять не могу

mazdaCANOdoDelayCounter - почему CAN? почему delay counter? счётчик задержки? Это всё очень как-то шайтанисто и слишком частных случай, можно он пока просто поживёт в твоём форке?

По уму я согласен, что какой-то одометр быть должен - ну тогда он должен сохранять в ПЗУ? Не вижу кода сохранения. Иначе это просто счётчик с момента старта ЭБУ? Какой в этом смысл?

Re: Всякие глупые вопросы, терзающие меня

Posted: Tue May 16, 2017 11:16 pm
by puff
Оно сохраняется в ПЗУ приборки :-) canododelaycounter нужен, чтобы отправлять этот пакет не так часто, как все остальные (в логах с живой машины так и вовсе один пакет с температурой и одо приходился на десяток пакетов со скоростью и оборотами).
У меня этот код отлично пашет с разными коэффициентами, и если оно не отжирает процессорных ресурсов - не вижу причин его не закоммитить.

Sent from my XT1058 using Tapatalk

Re: Всякие глупые вопросы, терзающие меня

Posted: Tue May 16, 2017 11:54 pm
by AndreyB
Переменной tickOdoDistance просто должно не быть - из-за этой переменной изменение engineConfiguration->vehicleSpeedCoef не будет учитываться пока не перезагрузишь. Вместо переменной либо константа, либо постоянно умножать vehicleSpeedCoef на константу.

Я не понимаю, что означает 0.277 - пожалуйста объясни 0.277.

У меня ощущение, что всё это можно заново переписать на базе существующего счётчика engine->engineState.vssDebugEventCounter

Если это будет работать на базе engine->engineState.vssDebugEventCounter, то я его переименую без слова debug

Давай начнём новый PR и не будем в него ничего кроме одометра пихать - конфиги пихать роверные не будем и CLT лампочки тоже. Один PR должен изменять только одну функциональность.

Re: Всякие глупые вопросы, терзающие меня

Posted: Wed May 17, 2017 7:25 am
by puff
russian wrote:Переменной tickOdoDistance просто должно не быть - из-за этой переменной изменение engineConfiguration->vehicleSpeedCoef не будет учитываться пока не перезагрузишь. Вместо переменной либо константа, либо постоянно умножать vehicleSpeedCoef на константу.
вот это меня вообще нисколько не смущает - в отличие от енджин-тюнов эту штуку настраиваешь всего лишь единожды. зато ресурс чипа отъедается условно не на 0,5%, а на 0,001%
Я не понимаю, что означает 0.277 - пожалуйста объясни 0.277.
а как еще перевести километры в час в метры в секунду? у тебя есть лучшая формула пересчета исходного коэффициента? вообще я так записал, потому что думал, так понятнее.
У меня ощущение, что всё это можно заново переписать на базе существующего счётчика engine->engineState.vssDebugEventCounter
Если это будет работать на базе engine->engineState.vssDebugEventCounter, то я его переименую без слова debug
я вообще хз что это за переменная и зачем она, трогать не стал. если она не нужна - ну убери ее.
счетчик считает тики датчика, а нам нужно считать метры.

Re: Всякие глупые вопросы, терзающие меня

Posted: Wed May 17, 2017 12:38 pm
by puff
ну что, тогда обращаюсь к @andreika и @maxi (у них с пониманием того, что требуется, почему-то не возникло).

пул-реквест закрыт с просьбой:
Please re-implement using vssDebugEventCounter

я не представляю, как это сделать (очевидно, еще и сохранив функциональность этого каунтера)

Re: Всякие глупые вопросы, терзающие меня

Posted: Wed May 17, 2017 9:40 pm
by puff
хопля!
последний номер лишен всякой логики!

Re: Всякие глупые вопросы, терзающие меня

Posted: Wed May 17, 2017 9:51 pm
by AndreyB
Вот это радость!

https://github.com/rusefi/rusefi/tree/master/firmware/tunerstudio/docs

Один файлик переименовал без пробелов и получилось сделать readme.MD

Теперь надо PR переименовывать все и все добавлять в readme.MD :)

Re: Всякие глупые вопросы, терзающие меня

Posted: Thu May 18, 2017 11:05 am
by puff
только что скачал-прошил - консоль, пропустив через себя прежние ошибки, секунд через десять заработала!
но msg,FATAL error: PA0 req by brake pedal switch used by adc input,
надо в конфиге мотора отключать этот brake pedal (кстати, зачем он понадобился?)
вот сокращенный лог:

Code: Select all

SerialConnector: connecting
scheduleOpening
scheduleOpening>openPort
postMessage SerialManager: Opening port: /dev/tty.usbmodem321
Opening /dev/tty.usbmodem321 @ 115200
PortHolder: Sleeping a bit
0 bytes in input buffer
Waiting for 2 byte(s): switch to binary
Got a7e - switched to binary protocol
Waiting for 2 byte(s): switch to binary

//и потом это a7e раз 15 подряд повторяется зачем-то
//потом повторяется  - до 16000 офсета
Reading from controller...
Sending packet 52 00 00 00 00 90 01 
Waiting for 2 byte(s): load image offset=0 header
Waiting for 405 byte(s): load image offset=0 body
Got configuration from controller.
Sending [set date 1495114732]
Sending packet 4F 
Waiting for 2 byte(s): output channels header
Waiting for 361 byte(s): output channels body
Sending packet 47 
Waiting for 2 byte(s): text header
Waiting for 6473 byte(s): text body
Looking for OPEN_SR5_0.1

// и потом опять любимое: 
BinaryProtocolServer on 2390
Unexpected pending data: 6281 byte(s)
data:...
Sending packet 45 73 65 74 20 64 61 74 65 20 31 34 39 35 31 31 34 37 33 32 
Waiting for 2 byte(s): execute header
Waiting for 5 byte(s): execute body
Sending packet 4F 
Waiting for 2 byte(s): output channels header
Waiting for 361 byte(s): output channels body
Sending packet 47 
Waiting for 2 byte(s): text header
Waiting for 1814 byte(s): text body
Sending [ls /]
Unexpected pending data: 1686 byte(s)
data:
Sending packet 4F 
Waiting for 2 byte(s): output channels header
Waiting for 361 byte(s): output channels body
Sending packet 47 
Waiting for 2 byte(s): text header
Waiting for 1630 byte(s): text body
Unexpected pending data: 1502 byte(s)
data: 
Sending packet 45 6C 73 20 2F 
Waiting for 2 byte(s): execute header
Waiting for 5 byte(s): execute body
Sending packet 4F 
Waiting for 2 byte(s): output channels header
Waiting for 361 byte(s): output channels body
Sending packet 47 
Waiting for 2 byte(s): text header
Waiting for 1691 byte(s): text body
ConnectionWatchdog.reconnectTimer restarting
postMessage SerialManager: Restarting serial IO
postMessage SerialManager: Opening port: /dev/tty.usbmodem321
Opening /dev/tty.usbmodem321 @ 115200
PortHolder: Sleeping a bit


и опять сначала
а потом внезапно 

Stopping text pull
после этого еще три таких чанка фиговых, а после этого - все заработало ок!

вопрос: кто сделал этот stopping text pull? зачем оно понадобилось и почему не было сделано раньше?

Re: Всякие глупые вопросы, терзающие меня

Posted: Thu May 18, 2017 11:17 am
by puff
рано радовался.
перезагрузил консоль без сброса платы - все ок переподключилось (за исключением этой fatal error)
а потом попробовал перезагрузить плату - и фиг, повторить успех пока не удалось.

а что значит надпись

BinaryProtocolServer on 2390

цифры - это что такое? порт tcip? а если он у меня занят?

Re: Всякие глупые вопросы, терзающие меня

Posted: Thu May 18, 2017 11:32 am
by puff
а в ТС этого нету, но есть другое:

Re: Всякие глупые вопросы, терзающие меня

Posted: Thu May 18, 2017 12:02 pm
by AndreyB
puff wrote:тно msg,FATAL error: PA0 req by brake pedal switch used by adc input,
надо в конфиге мотора отключать этот brake pedal (кстати, зачем он понадобился?)
да, при миграции прошивки нужно будет отключать - в дефолтах уже отключенно.

нужно для гоночных логов, и теоретические для холостого хода

Re: Всякие глупые вопросы, терзающие меня

Posted: Thu May 18, 2017 12:06 pm
by AndreyB
puff wrote:а в ТС этого нету, но есть другое:
Ага, это я над https://github.com/rusefi/rusefi/issues/4 работаю и сломал. Спасибо - починил!

Re: Всякие глупые вопросы, терзающие меня

Posted: Fri May 19, 2017 2:21 pm
by puff
а как через тюнерстудию указать, что pc10/pc11 работают на скорости 9600?
и вообще, как сейчас идет работа через эти пины? нужно еще какие-то перемычки включать, usb порт отключать? или сразу цепляемся и работаем?

Re: Всякие глупые вопросы, терзающие меня

Posted: Fri May 19, 2017 2:25 pm
by puff
тут по-моему уже устаревшее?
http://rusefi.com/wiki/index.php?title=Manual:Software:User/en
Communication mode
Grounding PB1 flips the default communication port mapping
By default, microUSB is used for rusEfi console and PC10/PC11 UART is used for TunerStudio. If you put a jumper between GND and PB1, this would be the other way around.
microUSB channel is more sensitive to noise but it's faster
See also http://rusefi.com/wiki/index.php?title=Manual:Software:TunerStudio_connectivity

Re: Всякие глупые вопросы, терзающие меня

Posted: Fri May 19, 2017 3:26 pm
by puff
и можно ли текстовом режиме законнектится условным телнетом на порт тюнерстудии/консоли и вручную в текстовом режиме поменять отдельные байты в настройках (ту же скорость обмена данными, номер мотора)? было бы очень удобно иметь такую возможность...

Re: Всякие глупые вопросы, терзающие меня

Posted: Fri May 19, 2017 3:33 pm
by AndreyB
http://rusefi.com/wiki/index.php?title=Manual:Software:User/en#Communication_mode немного стёр

скорость консоли добавил в rusefi.ini https://github.com/rusefi/rusefi/commit/dce616f48a314cabf5a5758f3f815dce678e185f

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

Re: Всякие глупые вопросы, терзающие меня

Posted: Fri May 19, 2017 4:31 pm
by puff
2) Once the USB TTL Serial device and drivers are installed, connect GND wire to GND on the brain board
connect RX wire of the TTL to pin PC10
connect TX wire of the TTL to pin PC11
3) Set up the new serial port @ 38400 with 8/N/1 no flow control using your favorite terminal
4) Hit button 't' on the keyboard. You should get something like this:
4073 Alive
у меня вот так в тюнерстудии
Снимок экрана 2017-05-19 в 19.26.26.png
Снимок экрана 2017-05-19 в 19.26.26.png (93.59 KiB) Viewed 17500 times
Консоль закрыл, на плате ресет нажал, коннекчусь к блютусу (смотрел его только что логанализатором - 9600 бодрейт).
Тычу туда 't'
а в ответ - вот такие штуки (по одному символу на каждое нажатие):
Снимок экрана 2017-05-19 в 19.30.57.png
Снимок экрана 2017-05-19 в 19.30.57.png (14.98 KiB) Viewed 17500 times

Re: Всякие глупые вопросы, терзающие меня

Posted: Fri May 19, 2017 4:42 pm
by puff
чертовщина
попробовал сделать screen /dev/cu.usbmodem321 - он сразу стал вываливать текстом всякую фигню типа как в девконсоли
сбросился, подключился девконсолюь - та же байда, что и раньше.
подключился еще раз через screen - чистый экран. нажал t - получил в ответ 14105 6044 3 ts_p_alive

но теперь тюнерстудия перестала работать!

Re: Всякие глупые вопросы, терзающие меня

Posted: Fri May 19, 2017 4:51 pm
by puff
и вообще, если девконсоль и ТС использует тот же порт и протокол - как у них может быть разная скорость?
мне не очень понятно: усб интерфейс - это порт, который никак не связан с тем уартом, что на pc10-pc11?

Re: Всякие глупые вопросы, терзающие меня

Posted: Fri May 19, 2017 4:59 pm
by AndreyB
Ты прав, там до сих пор каша :( скорости и правда у порта. исторически была идея разной скорости в зависимости от того, в каком режиме порт. Это всё нужно немного чистить :(

Re: Всякие глупые вопросы, терзающие меня

Posted: Fri May 19, 2017 5:25 pm
by puff
перепрошился заново. установил верхнюю скорость в 9600. сбросил плату. тюнерстудия законнектилась обратно ок.
подключился с блютуса - он выводил всякую фигню, но вот на l и m по-моему стал показывать
MShift v0.02
я хочу всего-навсего девконсолью подключиться...

Re: Всякие глупые вопросы, терзающие меня

Posted: Fri May 19, 2017 5:28 pm
by AndreyB
startConsoleInTextMode=false

ну нужно понимать, что ты поменяв эту опцию сам себе включил развлечения :)

Re: Всякие глупые вопросы, терзающие меня

Posted: Fri May 19, 2017 6:02 pm
by puff
а в тюнерстудии нету ничего про sd карту?
о! проверил - заработала сразу, просто вставил ее в порт!
а что означает в настройках Mass Storage и опции Auto, Always, Never, 3?

Re: Всякие глупые вопросы, терзающие меня

Posted: Fri May 19, 2017 7:20 pm
by puff
и все же. что происходит с UINT8 когда оно было 240 и к ней прибавили еще 15
а то вот тут такое
https://www.linux.org.ru/forum/talks/13428281

Re: Всякие глупые вопросы, терзающие меня

Posted: Fri May 19, 2017 8:12 pm
by Maxi
puff wrote:и все же. что происходит с UINT8 когда оно было 240 и к ней прибавили еще 15
а то вот тут такое
https://www.linux.org.ru/forum/talks/13428281
"эти идиоты добрались до gcc и сломали его"
ой ты божечки - да всем насрать! У нормальный пацанов в прожетах юзается 3-я версия GCC не обновляемая годами и охваченная 100500 тестами с жестко зафиксированными уровнями оптимизациями. А че там пионеры говкода делают в новых ветках gcc и какие от этого грабли вырастают - это лишь их проблемы.