Jan/WLED_MM_Infinity
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

AR audio sync: improved sequence check for audio packet handling

Browse Source 
currently just a comment so I don't forget about it :-) needs some testing but very sure this will work.
This commit is contained in:
Frank
2025-11-18 23:57:44 +01:00
committed by GitHub
parent 0a0ca506eb
commit 219355d047
1 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
usermods/audioreactive/audio_reactive.h
View File

@@ -1754,7 +1754,15 @@ class AudioReactive : public Usermod {
// add info for UI // add info for UI
if ((receivedPacket.frameCounter > 0) && (lastFrameCounter > 0)) receivedFormat = 3; // v2+ if ((receivedPacket.frameCounter > 0) && (lastFrameCounter > 0)) receivedFormat = 3; // v2+
else receivedFormat = 2; // v2 else receivedFormat = 2; // v2
// check sequence
// Simpler 8-bit rollover-safe sequence check:
// (int8_t)(cur - prev) > 0 => cur is ahead of prev by 1..127
// 0 => duplicate,
// < 0 => older
// bool sequenceOK = !audioSyncSequence || receivedPacket.frameCounter == 0 || // always accept legacy "0"
// ((int8_t)(receivedPacket.frameCounter - lastFrameCounter) > 0);
// check sequence
bool sequenceOK = false; bool sequenceOK = false;
if(receivedPacket.frameCounter > lastFrameCounter) sequenceOK = true; // sequence OK if(receivedPacket.frameCounter > lastFrameCounter) sequenceOK = true; // sequence OK
if((lastFrameCounter < 12) && (receivedPacket.frameCounter > 248)) sequenceOK = false; // prevent sequence "roll-back" due to late packets (1->254) if((lastFrameCounter < 12) && (receivedPacket.frameCounter > 248)) sequenceOK = false; // prevent sequence "roll-back" due to late packets (1->254)