Drop `readline()` for `read()`; Remove wait for next frame1;

boredomwontgetus commented 2 years ago

Instead of using readline() we use read() to read the byte-stream in chunks of 100bytes. This requires the interval to be carefully set. It should not be >= 5. Tested successfully with intervals from 1s to 4s. All work, but i do recommend to stick with 1s or 2s interval (serial read timeout).
Changed the way we test for a complete segment: We can now tell when the end of a segment is reached by comparing the number of bytes we want to read to the number of bytes we actually got. This enables us to check if we received a full segment by looking for the known end_byte to actually sit at end of the segment.

I would be happy if you could test this PR a bit and let me know your findings and thoughts.

Thank you very much.

boredomwontgetus commented 2 years ago

thank you! i see you also merged 375d0d4d98c8caaac02b571d955823444f580fd1. this is is awesome.

tirolerstefan commented 2 years ago

I cannot test MQTT, so please take over the support of this piece of code :-).

tirolerstefan / kaifa