Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable Checksum check? #42

Open
TheMKDev opened this issue Aug 4, 2019 · 5 comments
Open

Disable Checksum check? #42

TheMKDev opened this issue Aug 4, 2019 · 5 comments

Comments

@TheMKDev
Copy link

TheMKDev commented Aug 4, 2019

Is there a way to disable the check?

@Eric-Miao
Copy link

No, we also encounter this problem. LOL

@RonAmihai
Copy link

RonAmihai commented May 2, 2020

I've succeeded to "achieve" this by disabling the raising of the checksum error.

raise ValueError('Invalid checksum')

But it seems to cause side effects sometimes like raising struct.error: unpack requires a buffer of 4 bytes here:

received, = struct.unpack(self.fmt, bytes(data[:self.size]))

I've took the ugliness ever further, surrounded the struct.unpack with try/except and ignored the error in case it rises.
Of course it doesn't end well from the data's perspective, and if the process actually reaches this point the data will be almost completely corrupted (event though the I/O logged as 'Finished').

So I'm trying to find what causes the size alignment issues in the case of ignored checksum error.

@Tetragromaton
Copy link

Same at android app, can it be toggled off ?

@romanz
Copy link
Owner

romanz commented Jul 3, 2020

It can be done, but the main issue (as @RonAmihai mentioned) is the loss of byte alignment.
Given enough noise, the receiver will lose the synchronization and won't be able to decode the rest of the message.
Would you like to have the option to recover the initial part of the message, before the checksum error?

@erikson1970
Copy link

Nice work @romanz - I added optional flag to enable/disable stop faults in my latest commit on my fork. I am going to add Reed Solomon FEC next. May take a few days as the day job is intruding.

Repository owner deleted a comment from OmerTurann Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants