Made4RC / HoTTServer

Arduino library to send telemetry data to Graupner HoTT-recievers
10 stars 6 forks source link

Synchronisation can not work the way it is implemented #4

Open icebreaker-ch opened 3 months ago

icebreaker-ch commented 3 months ago

The synchronisation in the processRequest method can not work the way it is implemented. The program is waiting for 2 bytes. The first is expected to be the request mode and the second the requestID. So it is expected, to receive the first byte first after switch on the receiver like this:

0x80 0x8E 0x80 0x8D 0x80 0x8C 0x80 0x89 0x80 0x8A
^         ^         ^         ^         ^ 
^sync points

But if the bytes are received like this (missed first byte during startup):

0x8E 0x80 0x8D 0x80 0x8C 0x80 0x89 0x80 0x8A 0x80
^         ^         ^         ^         ^ 
no sync possible

The program will never sync to the communication.

Made4RC commented 3 months ago

Thank you for your investigation and reply.

I do not use this code anymore. I‘d appreciate if someone would take over and maintain it for further use.

— Norbert Kolb

Am 01.09.2024 um 13:10 schrieb icebreaker @.***>:

 The synchronisation in the processRequest method can not work the way it is implemented. The program is waiting for 2 bytes. The first is expected to be the request mode and the second the requestID. So it is expected, to receive the first byte first after switch on the receiver like this:

0x80 0x8E 0x80 0x8D 0x80 0x8C 0x80 0x89 0x80 0x8A ^ ^ ^ ^ ^ ^sync points But if the bytes are received like this (missed first byte during startup):

0x8E 0x80 0x8D 0x80 0x8C 0x80 0x89 0x80 0x8A 0x80 ^ ^ ^ ^ ^ no sync possible

The program will never sync to the communication.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.

sp3swj commented 2 months ago

Using Radiomaster ELRS ER6 and this code ( arduino Nano) I can send correctly only

server.setAltitude(temp1 += 1); and some fake increment value

On my Radiomaster MT12 on LCD i have this value propper displayed and increased :-)

Other value not works - or I to less test.