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

[puff]

Я сам пугаюсь. На картинке выше настроены четыре канала, из которых первый, третий и четвертый можно редактировать , а второй - не активен..

Sent from my XT1058 using Tapatalk

[AndreyB]

Видимо я вижу доступный для редактирования 8ой канал, а ты не видишь?

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

[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

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

[AndreyB]

Да, всё именно так!

[puff]

оно тоже есть в пулреквесте!

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

[AndreyB]

оно тоже есть в пулреквесте!

И это плохо! Потому что получается у нас два разных изменения смешаны вместе, и это не правильно!

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

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

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

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

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

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

[puff]

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

Sent from my XT1058 using Tapatalk

[AndreyB]

Переменной tickOdoDistance просто должно не быть - из-за этой переменной изменение engineConfiguration->vehicleSpeedCoef не будет учитываться пока не перезагрузишь. Вместо переменной либо константа, либо постоянно умножать vehicleSpeedCoef на константу.

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

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

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

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

[puff]

Переменной tickOdoDistance просто должно не быть - из-за этой переменной изменение engineConfiguration->vehicleSpeedCoef не будет учитываться пока не перезагрузишь. Вместо переменной либо константа, либо постоянно умножать vehicleSpeedCoef на константу.

вот это меня вообще нисколько не смущает - в отличие от енджин-тюнов эту штуку настраиваешь всего лишь единожды. зато ресурс чипа отъедается условно не на 0,5%, а на 0,001%

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

а как еще перевести километры в час в метры в секунду? у тебя есть лучшая формула пересчета исходного коэффициента? вообще я так записал, потому что думал, так понятнее.

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

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

[puff]

ну что, тогда обращаюсь к @andreika и @maxi (у них с пониманием того, что требуется, почему-то не возникло).

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

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

[puff]

хопля!
последний номер лишен всякой логики!

[AndreyB]

Вот это радость!

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

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

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

[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? зачем оно понадобилось и почему не было сделано раньше?

[puff]

рано радовался.
перезагрузил консоль без сброса платы - все ок переподключилось (за исключением этой fatal error)
а потом попробовал перезагрузить плату - и фиг, повторить успех пока не удалось.

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

BinaryProtocolServer on 2390

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

[puff]

а в ТС этого нету, но есть другое:

[AndreyB]

тно msg,FATAL error: PA0 req by brake pedal switch used by adc input,
надо в конфиге мотора отключать этот brake pedal (кстати, зачем он понадобился?)

да, при миграции прошивки нужно будет отключать - в дефолтах уже отключенно.

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

[AndreyB]

а в ТС этого нету, но есть другое:

Ага, это я над https://github.com/rusefi/rusefi/issues/4 работаю и сломал. Спасибо - починил!

[puff]

а как через тюнерстудию указать, что pc10/pc11 работают на скорости 9600?
и вообще, как сейчас идет работа через эти пины? нужно еще какие-то перемычки включать, usb порт отключать? или сразу цепляемся и работаем?

[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

[puff]

и можно ли текстовом режиме законнектится условным телнетом на порт тюнерстудии/консоли и вручную в текстовом режиме поменять отдельные байты в настройках (ту же скорость обмена данными, номер мотора)? было бы очень удобно иметь такую возможность...

[AndreyB]

http://rusefi.com/wiki/index.php?title=Manual:Software:User/en#Communication_mode немного стёр

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

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

[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 (93.59 KiB) Viewed 16090 times

Консоль закрыл, на плате ресет нажал, коннекчусь к блютусу (смотрел его только что логанализатором - 9600 бодрейт).
Тычу туда 't'
а в ответ - вот такие штуки (по одному символу на каждое нажатие):

Снимок экрана 2017-05-19 в 19.30.57.png (14.98 KiB) Viewed 16090 times

[puff]

чертовщина
попробовал сделать screen /dev/cu.usbmodem321 - он сразу стал вываливать текстом всякую фигню типа как в девконсоли
сбросился, подключился девконсолюь - та же байда, что и раньше.
подключился еще раз через screen - чистый экран. нажал t - получил в ответ 14105 6044 3 ts_p_alive

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

[puff]

и вообще, если девконсоль и ТС использует тот же порт и протокол - как у них может быть разная скорость?
мне не очень понятно: усб интерфейс - это порт, который никак не связан с тем уартом, что на pc10-pc11?

[AndreyB]

Ты прав, там до сих пор каша скорости и правда у порта. исторически была идея разной скорости в зависимости от того, в каком режиме порт. Это всё нужно немного чистить

[puff]

перепрошился заново. установил верхнюю скорость в 9600. сбросил плату. тюнерстудия законнектилась обратно ок.
подключился с блютуса - он выводил всякую фигню, но вот на l и m по-моему стал показывать
MShift v0.02
я хочу всего-навсего девконсолью подключиться...

[AndreyB]

startConsoleInTextMode=false

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

[puff]

а в тюнерстудии нету ничего про sd карту?
о! проверил - заработала сразу, просто вставил ее в порт!
а что означает в настройках Mass Storage и опции Auto, Always, Never, 3?

[puff]

и все же. что происходит с UINT8 когда оно было 240 и к ней прибавили еще 15
а то вот тут такое
https://www.linux.org.ru/forum/talks/13428281

[Maxi]

и все же. что происходит с UINT8 когда оно было 240 и к ней прибавили еще 15
а то вот тут такое
https://www.linux.org.ru/forum/talks/13428281

"эти идиоты добрались до gcc и сломали его"
ой ты божечки - да всем насрать! У нормальный пацанов в прожетах юзается 3-я версия GCC не обновляемая годами и охваченная 100500 тестами с жестко зафиксированными уровнями оптимизациями. А че там пионеры говкода делают в новых ветках gcc и какие от этого грабли вырастают - это лишь их проблемы.