Ошибки компиляции для платы Arduino Due about svisual

[Tyill]

Здравствуйте.

MsTimer2::set(SV_CYCLEREC_MS, thrProc);
MsTimer2::start();

заменить на

Timer0.attachInterrupt(thrProc).start(SV_CYCLEREC_MS);

[Alex6305]

Спасибо, заменил. Теперь скетч загружается успешно. Но модуль в SVMonitor не появляется.
Может быть, ещё что-то нужно изменить при использовании другого таймера?

[Tyill]

Да вроде ничего больше не надо.
Попробуйте поотлаживать так: напишите в функцию thrProc где-нидь Serial.print("Hello"), и запустите в Arduino IDE, то есть без подключения к SVMonitor. Тем самым проверите что функция отправки данных вызывается внутри по таймеру.
Потом можно дальше копать.

[Alex6305]

Не работает.

Хотя я не уверен, что написал правильно.

[Tyill]

Правильно все.
Значит надо разбираться с DueTimer.

Попробуйте Timer1 сделать:
Timer1.attachInterrupt(thrProc).start(SV_CYCLEREC_MS);

Вот здесь у него есть примеры, посмотрите:
https://github.com/ivanseidel/DueTimer/tree/master/examples

В интернете еще поищите примеры с DueTimer.

[Tyill]

А. Нет. Неправильно.

Функция thrProc - это моя ф-я.

Ее на править в https://github.com/Tyill/SVisual/blob/master/src/SVArduino/sv_client.cpp#L88

[Alex6305]

Только вечером смогу проверить. Напишу, что получилось. сб, 29 февр. 2020 г., 7:33 Alexander Medvedev <[email protected]>:

…

А. Нет. Неправильно. Функция thrProc - это моя ф-я. Ее на править в https://github.com/Tyill/SVisual/blob/master/src/SVArduino/sv_client.cpp#L88 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#24?email_source=notifications&email_token=AORDCVX6MOTEZ4XIZVIRZ63RFCH2PA5CNFSM4K5OZGZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENLGYBY#issuecomment-592866311>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AORDCVV67NJW6BCUFW4DDKTRFCH2PANCNFSM4K5OZGZA> .

[Alex6305]

Попробовал Timer1 – ни чего не изменилось. А вот дальше:

Если

То

А если

То

Может быть, МИЛЛИсекунды в MsTimer2 и МИКРОсекунды в DueTimer?

[Tyill]

Да, точно, там микросекунды:
To call a function handler every 1000 microseconds:

Сделайте так тогда:
Timer1.attachInterrupt(thrProc).start(SV_CYCLEREC_MS * 1000);

[Alex6305]

Сделал: "Timer1.attachInterrupt(thrProc).start(SV_CYCLEREC_MS * 1000);".
В мониторе порта "Hello" печатает, но в SVMonitor модуль не появляется.
Где-то что-то ещё не так.

[Tyill]

Проверьте еще раз как сейчас выглядит ф-я thrProc, когда вы работаете с SVMonitor.
Должна выглядеть так:

void thrProc (){
   objCln.doSendCycle();
}

То есть без всяких Serial.print(...), которые вы добавляли для проверки.

Можете файлы мне отправить, которые вы поменяли в SVArduino, я их посмотрю еще.

[Alex6305]

Да, вот папка из c:\Program Files (x86)\Arduino\libraries\

SVArduino.zip

[Tyill]

Все правильно вроде. Должно работать все.

Можете попробовать в ArduinoIDE запустить, то есть без подключения к SVMonitor,
в Serial.print(...) ничего не надо писать.

Программа тестовая:

void setup() {

	bool ok = svisual::connectOfCOM("test");

}
int cnt = 0;
void loop() {

	svisual::addIntValue("dfv", cnt);

	++cnt;
	if (cnt > 10) cnt = 0;

	bool odd_cnt = cnt % 2;

    svisual::addBoolValue("bFW", odd_cnt);

    svisual::addBoolValue("bBW", !odd_cnt);

	delay(200);
}

Посмотрите монитора порта в ArduinoIDE, должен показывать данные.

[Alex6305]

В мониторе порта ArduinoIDE только "=".

[Tyill]

Тоже самое сделайте, только с MsTimer, то есть обратно на MsTimer верните как было.
Посмотрим.

[Alex6305]

К сожалению MsTimer2 не работает с AVR контроллером.

#error Unsupported CPU type
#error MsTimer2 library only works on AVR architecture

[Alex6305]

То есть наоборот работает только с AVR, а Due ARM.

[Tyill]

Ладно тогда с этим MsTimer.

Я помню что писал в монитор, когда данные передавал.
Сейчас значит ничего не пишет.
Дальше надо значит отладочные сообщения писать в каждую функцию, которая должна вызываться, таким образом:

void thrProc(){
   Serial.print("thrProc")


void sv_client::doSendCyc(){
    Serial.print("sv_client::doSendCyc")


bool sv_client::sendData(){
   Serial.print("sv_client::sendData")

bool sv_client::sendDataOfCom(){
   Serial.print("sv_client::sendDataOfCom")

То есть надо понять где затык, потом из-за чего.

И запускать все это дело из под ArduinoIDE, то есть SVMonitor не подключать.

[Alex6305]

Отладочные сообщения добавил в функции. В результате в мониторе:
thrProcsv_client::doSendCycthrProcsv_client::doSendCycthrProcsv_client::doSendCycthrProcsv_client::doSendCycthrProcsv_client::doSendCycthrProcsv_client::doSendCycthrProcsv_client::doSendCycthrProcsv_client::doSendCycthrProcsv_client::doSendCyct

[Tyill]

вот видите, нет sv_client::sendData().

Уберите все отладоч сообщения, чтобы не мешали.
Дальше ставьте отладочные сообщения в ф-ии sv_client::doSendCyc(), например здесь:

 void sv_client::doSendCyc(){
	   
	  // connection support
	  if (!isConnect_)
		  isConnect_ = tcpConnect();
	
Serial.print("isConnect = ");Serial.print(isConnect); Serial.print("\n");

	  // check active value in current cycle
	  int prevCyc = curCycCnt_ - 1; if (prevCyc < 0) prevCyc = SV_PACKETSZ - 1;
	  int sz = values_.size(); valueRec* vr;
	  for (int i = 0; i < sz; ++i){

		  vr = (valueRec*)values_.at(i);

		  if (!vr->isActive){
			  vr->vals[curCycCnt_] = vr->vals[prevCyc];

			  if ((vr->type == valueType::tBool) && vr->isOnlyFront)
				  vr->vals[curCycCnt_].tBool = false;
		  }

		  vr->isActive = false;
	  }

	  int next = curCycCnt_ + 1;
 
Serial.print("next = "); Serial.print(next); Serial.print("\n");

	  // send data
	  if (next >= SV_PACKETSZ) {

Serial.print("beforeSendData"); Serial.print("\n");

		  isConnect_ = (isConnect_) ? sendData() : false;

		  curCycCnt_ = 0;
	  }
	  else 
		  ++curCycCnt_; 
	 
	}

[Alex6305]

Поскольку "sv_client.cpp:339:44: error: 'isConnect' was not declared in this scope",
Заменил "Serial.print("isConnect = ");Serial.print(isConnect); Serial.print("\n");"
на "Serial.print("isConnect_ = ");Serial.print(isConnect_); Serial.print("\n");".
В результате:

[Tyill]

После "Serial.print("next = "); Serial.print(next); Serial.print("\n");"
напишите еще:
Serial.print("SV_PACKETSZ= "); Serial.print(SV_PACKETSZ); Serial.print("\n");

Дальше. Сюда поставьте запись:

  bool sv_client::sendData(){
      Serial.print("sendData");
    bool ok = isCom_ ? sendDataOfCom() : sendDataOfEthernet();
      
	return ok; 
  }

И сюда:

bool sv_client::sendDataOfCom(){
      Serial.print("sv_client::sendDataOfCom");


}

[Alex6305]

Добавил записи, может быть, посмотрите файл, насколько правильно.

sv_client.zip

[Tyill]

Да все правильно. Дальше то что пишет после "next = 9"?
Вы понимаете что мы делаем?

Мы пытаемся понять, почему не заходит в функцию sendDataOfCom, соответственно не передает данные.

[Alex6305]

Дальше то что пишет после "next = 9"?

На этом останавливается.

Вы понимаете что мы делаем?

Пытаюсь вникнуть, но без Вашей помощи я с этим не разберусь.

[Alex6305]

Должно быть так:

isConnect_ = 1
next = 9
SV_PACKETSZ= 10
isConnect_ = 1
next = 10
SV_PACKETSZ= 10
beforeSendData
sendDatasv_client::sendDataOfCom=begin=

Но останавливается на:

isConnect_ = 1
next = 9
SV_PACKETSZ= 10
i

По аналогии с тестом "Hello", когда печатает только первые две буквы "He" и останавливается.
Может с этим таймером нужно как-то по-другому?..

[Tyill]

Давайте еще одну вещь проверим, сделайте так:

void sv_client::doSendCyc(){
	   
         Serial.print("sv_client::doSendCyc\n");

         return ;   

	  // connection support
	  if (!isConnect_)
	  .
          .
          .
}

Подсчитаем сколько раз выведет "sv_client::doSendCyc" прежде чем заткнется.
Если заткнется, значит таймер считает только до 10, хотя не должен так работать.

[Alex6305]

Не затыкается. Что-то другое.

[Tyill]

Замените:

// send data
	  if (next >= SV_PACKETSZ) {

Serial.print("beforeSendData"); Serial.print("\n");

		  isConnect_ = (isConnect_) ? sendData() : false;

		  curCycCnt_ = 0;
	  }
	  else 
		  ++curCycCnt_; 

на

// send data
	  if (next >= SV_PACKETSZ) {

Serial.print("beforeSendData"); Serial.print("\n");

		 if (isCom_)
                     sendDataOfCom();
                 else 
                     sendDataOfEthernet();

		  curCycCnt_ = 0;
	  }
	  else 
		  ++curCycCnt_; 

Дальше здесь, закомментируйте "Serial.flush();":

bool sv_client::sendDataOfCom(){

Serial.print("sv_client::sendDataOfCom");
      
	int sz = values_.size();
	
	if (sz == 0) return true;
	
	//Serial.flush();
		
	const char* start = "=begin=";
.
.
.

[Alex6305]

Вот что получается:

sv_client.zip

[Tyill]

вы это уберите, иначе не идет дальше:

void sv_client::doSendCyc(){
	   
         //Serial.print("sv_client::doSendCyc\n");

         //return ;   

[Alex6305]

Убрал.

[Tyill]

покажите файл sv_client.cpp

[Alex6305]

К сожалению, смогу только вечером. ср, 4 мар. 2020 г., 7:40 Alexander Medvedev <[email protected]>:

…

покажите файл sv_client.cpp — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#24?email_source=notifications&email_token=AORDCVWZKFINRXPISKX2OHTRFXLSLA5CNFSM4K5OZGZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENWKMKA#issuecomment-594322984>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AORDCVVOCWNXFDCOD7F2VNTRFXLSLANCNFSM4K5OZGZA> .

[Alex6305]

sv_client.zip

[Tyill]

sv_client.zip
пробуйте. что напишет - сюда.

Тяжелый случай какой-то. Мне пока кажется, что в размерности типов проблема, там есть места где я подразумеваю размеры, а надо было sizeof писать. Ну это ладно пока, смотрим дальше.

[Alex6305]

Вот что получается:

[Tyill]

sv_client.zip

идем дальше. что напишет - сюда.

[Alex6305]

Сейчас:

[Tyill]

sv_client.zip

дальше. что напишет - сюда.

[Alex6305]

Не пишет, только "=".
К сожалению, не было возможности проверить раньше.

[Tyill]

sv_client.zip

дальше.

[Alex6305]

[Tyill]

sv_client.zip

дальше.

[Alex6305]

[Tyill]

Замените

  bool sv_client::connectOfCOM(const char* moduleName, int speed){
  		
.
.
.
		Serial.begin(speed);   ->  Serial3.begin(speed); 

[Tyill]

Если не пойдет еще так попробуйте:

bool sv_client::connectOfCOM(const char* moduleName, int speed){
  		
.
.
.
		Serial.begin(speed); 
                SerialUSB.begin(speed); 

[Alex6305]

Первый вариант – ошибка компиляции.
А второй однократно печатает "=".
sv_client.zip

[Tyill]

sv_client.zip

пробуйте так.

[Alex6305]

То же самое, только "=".

[Tyill]

sv_client.zip

еще раз.

[Tyill]

sv_client.zip

поправил. предыд-й с ошибкой.

[Alex6305]

Не получается:

\sv_client.cpp:311:35: error: no matching function for call to 'UARTClass::write(svisual::valueType&, int)'
Serial.write(data->type, 4);

\sv_client.cpp:316:47: error: call of overloaded 'write(int&, int)' is ambiguous
Serial.write(data->vals[j].tInt, 4);

\sv_client.cpp:318:49: error: no matching function for call to 'UARTClass::write(float&, int)'
Serial.write(data->vals[j].tFloat, 4);

\sv_client.cpp:325:12: error: 'pAuxSD' was not declared in this scope
delete pAuxSD;

[Tyill]

sv_client.zip

так.

[Alex6305]

Эти ошибки остались:

\sv_client.cpp:311:35: error: no matching function for call to 'UARTClass::write(svisual::valueType&, int)'
Serial.write(data->type, 4);

\sv_client.cpp:316:47: error: call of overloaded 'write(int&, int)' is ambiguous
Serial.write(data->vals[j].tInt, 4);

\sv_client.cpp:318:49: error: no matching function for call to 'UARTClass::write(float&, int)'
Serial.write(data->vals[j].tFloat, 4);

[Tyill]

sv_client.zip

[Alex6305]

То же самое: "=".

[Alex6305]

Только: "=".

[Tyill]

Попробуйте увеличить размер внутреннего буфера, подробнее о нем тут
http://mypractic-forum.ru/viewtopic.php?t=11

поставьте 256 байт:
#define SERIAL_TX_BUFFER_SIZE 256
#define SERIAL_RX_BUFFER_SIZE 256

[Alex6305]

Поставил 256.

HardwareSerial.zip
sv_client.zip

[Alex6305]

Не могу оценить, насколько это проще, но может быть попробовать подойти с другой стороны, адаптировать DueTimer к SVMonitor?

[Tyill]

DueTimer не причем.
Здесь я думаю буфер переполняется (в который Serial.write пишет перед физической отправкой).

Попробуйте, еще пример сократить, только 1 переменную записывать:

#include <SVClient.h>
void setup() {

	bool ok = svisual::connectOfCOM("test");

}
int cnt = 0;
void loop() {

	svisual::addIntValue("dfv", cnt);

	++cnt;
	if (cnt > 10) cnt = 0;

	bool odd_cnt = cnt % 2;
 
	delay(200);
}

[Tyill]

.

[Alex6305]

С одной переменной заработало.

[Tyill]

ну вот и хорошо.
попробуйте буфер увеличить еще до 1024. И 3 переменные как в исходном примере записать.
(чтобы точно убедится что в нем дело)

[Alex6305]

Буфер увеличил, так как на скриншоте.

С тремя не работает, с двумя тоже.

[Tyill]

значит не в буфере дело, да и настройки общие поди для всех мк.
обратно верните настройки по буферу, которые были (выше скрин был с ними).

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

[Alex6305]

Настройки буфера вернул, память использует только 6%.

[Alex6305]

Может быть дело в разной разрядности Mega и Due?

[Tyill]

Посмотрите сколько напишет:

void setup() {

}
void loop() {

    Serial.print("int sz "); Serial.print(sizeof(int)); Serial.print("\n");
    Serial.print("long int sz "); Serial.print(sizeof(long int)); Serial.print("\n");
delay(200);

}

[Alex6305]

Пишет четыре.

[Tyill]

Посмотрел еще раз код.
Ну, не вижу где это (int32) могло повлиять. Везде правильно указываю размеры, когда вызываю Serial.write.

Так. Давайте попробуем отказаться от DuoTimer. Вручную если будете передавать данные, добавил для этого метод 'manualSendData':

void setup() {
	svisual::connectOfCOM("test");
}

int cnt = 0;
void loop() {

	svisual::addIntValue("dfv", cnt);

	++cnt;
	if (cnt > 10) cnt = 0;

	bool odd_cnt = cnt % 2;

    svisual::addBoolValue("bFW", odd_cnt);

    svisual::addBoolValue("bBW", !odd_cnt);


    svisual::manualSendData();

    delay(SV_CYCLEREC_MS);
}

sv_client.zip

[Alex6305]

Без таймеров наконец-то заработало, тест во всяком случае.
Попробую Ethernet, и уменьшить SV_CYCLEREC_MS, нужно ли при это изменять SV_PACKETSZ?

[Tyill]

Отлично.
Какой можно вывод сделать из всей этой эпопеи - пока думаю, что все таки dueTimer что-то там неправильно работает, или какое-то влияние на Serial оказывает.

SV_CYCLEREC_MS, нужно ли при это изменять SV_PACKETSZ

нет, они между собой не связаны.

[Alex6305]

БОЛЬШОЕ СПАСИБО ЗА ПОМОЩЬ.

Если не трудно, расскажите, пожалуйте более подробно о "SV_PACKETSZ" (размер пакета, как я понимаю).

В зависимости от чего нужно изменять и как он коррелируется с количеством переменных.

В моём, конкретном случае, это 12 целочисленных (10bit) и 18 булевых.

"SV_CYCLEREC_MS" при этом желательно - 1MS, 10 – приемлемо, ну а 100 край.

[Tyill]

SV_PACKETSZ зачем вообще нужен. Этот параметр задает количество значений переменной, которое за раз передается на сервер (SVMonitor). Это разгружает линию передачи и обработку, иначе бы пришлось передавать каждое значение.

SV_PACKETSZ может принимать значения от 1 до ... какого-то приемлемого значения. Чем больше, тем больший буфер будет в памяти МК для каждой переменной, например, SV_PACKETSZ = 10, значит буфер в МК для переменной 4 * 10 = 40 байт.

SV_CYCLEREC_MS может принимать значения от 10 мс.
То есть 10 мс это принятый мной минимум.
Потому что, если ниже ставить, архив записей на диске начинает занимать гигабайты.
Ну и посчитал что, нет смысла чаще писать, это уже сфера осциллографа.

[Tyill]

В вашем случае можно не трогать SV_CYCLEREC_MS, оставить по умолчанию (100мс).

А цикл опроса датчиков, сделать 1мс.

Получится так:

vaoid setup() {
	svisual::connectOfCOM("test");
}

int svSendCnt = 0;
int cnt = 0;
void loop() {
	
	++cnt;
	if (cnt > 10) cnt = 0;

	bool odd_cnt = cnt % 2;
  
    if ((svSendCnt % SV_CYCLEREC_MS) == 0) {

        svisual::addIntValue("dfv", cnt);

        svisual::addBoolValue("bFW", odd_cnt);

        svisual::addBoolValue("bBW", !odd_cnt);

       svisual::manualSendData();
   }

    ++svSendCnt;
    
    delay(1);
}

[Alex6305]

Понял. Спасибо. пн, 9 мар. 2020 г., 7:42 Alexander Medvedev <[email protected]>:

…

В вашем случае можно не трогать SV_CYCLEREC_MS, оставить по умолчанию (100мс). А цикл опроса датчиков, сделать 1мс. Получится так: vaoid setup() { svisual::connectOfCOM("test"); } int sendCnt = 0; int cnt = 0; void loop() { svisual::addIntValue("dfv", cnt); ++cnt; if (cnt > 10) cnt = 0; bool odd_cnt = cnt % 2; svisual::addBoolValue("bFW", odd_cnt); svisual::addBoolValue("bBW", !odd_cnt); if ((sendCnt % SV_CYCLEREC_MS) == 0) svisual::manualSendData(); ++sendCnt; delay(1); } — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#24?email_source=notifications&email_token=AORDCVRG2QBMPFUB7OGFVGLRGRXSVA5CNFSM4K5OZGZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOFUAVA#issuecomment-596328532>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AORDCVTOZRY4H6BLAWBQFVDRGRXSVANCNFSM4K5OZGZA> .