patrick/OpenDTU
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Fix: Only count RF RX packets when packets where sent

Browse Source 
This mainly occours after a reset of  the statistics that receive count is higher then transmit count
This commit is contained in:
Thomas Basler 2024-09-26 17:45:34 +02:00
parent 67cae68e83
commit 8e26ef4e2e
1 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
lib/Hoymiles/src/HoymilesRadio.cpp
View File

@ -67,7 +67,9 @@ void HoymilesRadio::handleReceivedPackage()
} else if (verifyResult == FRAGMENT_ALL_MISSING_TIMEOUT) { } else if (verifyResult == FRAGMENT_ALL_MISSING_TIMEOUT) {
Hoymiles.getMessageOutput()->println("Nothing received, resend count exeeded"); Hoymiles.getMessageOutput()->println("Nothing received, resend count exeeded");
// Statistics: Count RX Fail No Answer // Statistics: Count RX Fail No Answer
inv->RadioStats.RxFailNoAnswer++; if (inv->RadioStats.TxRequestData > 0) {
inv->RadioStats.RxFailNoAnswer++;
}
_commandQueue.pop(); _commandQueue.pop();
_busyFlag = false; _busyFlag = false;
@ -75,7 +77,9 @@ void HoymilesRadio::handleReceivedPackage()
} else if (verifyResult == FRAGMENT_RETRANSMIT_TIMEOUT) { } else if (verifyResult == FRAGMENT_RETRANSMIT_TIMEOUT) {
Hoymiles.getMessageOutput()->println("Retransmit timeout"); Hoymiles.getMessageOutput()->println("Retransmit timeout");
// Statistics: Count RX Fail Partial Answer // Statistics: Count RX Fail Partial Answer
inv->RadioStats.RxFailPartialAnswer++; if (inv->RadioStats.TxRequestData > 0) {
inv->RadioStats.RxFailPartialAnswer++;
}
_commandQueue.pop(); _commandQueue.pop();
_busyFlag = false; _busyFlag = false;
@ -83,7 +87,9 @@ void HoymilesRadio::handleReceivedPackage()
} else if (verifyResult == FRAGMENT_HANDLE_ERROR) { } else if (verifyResult == FRAGMENT_HANDLE_ERROR) {
Hoymiles.getMessageOutput()->println("Packet handling error"); Hoymiles.getMessageOutput()->println("Packet handling error");
// Statistics: Count RX Fail Corrupt Data // Statistics: Count RX Fail Corrupt Data
inv->RadioStats.RxFailCorruptData++; if (inv->RadioStats.TxRequestData > 0) {
inv->RadioStats.RxFailCorruptData++;
}
_commandQueue.pop(); _commandQueue.pop();
_busyFlag = false; _busyFlag = false;
@ -101,7 +107,9 @@ void HoymilesRadio::handleReceivedPackage()
// Successful received all packages // Successful received all packages
Hoymiles.getMessageOutput()->println("Success"); Hoymiles.getMessageOutput()->println("Success");
// Statistics: Count RX Success // Statistics: Count RX Success
inv->RadioStats.RxSuccess++; if (inv->RadioStats.TxRequestData > 0) {
inv->RadioStats.RxSuccess++;
}
_commandQueue.pop(); _commandQueue.pop();
_busyFlag = false; _busyFlag = false;