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

Error "Payload 'b10109000001003c0020' failed CRC and checksum" #101

Closed
lesgrebe opened this issue Mar 13, 2024 · 20 comments
Closed

Error "Payload 'b10109000001003c0020' failed CRC and checksum" #101

lesgrebe opened this issue Mar 13, 2024 · 20 comments
Assignees
Labels
bug Something isn't working device issue Issue is due to device limitation msmart-ng Issue is with underlying msmart-ng library

Comments

@lesgrebe
Copy link

lesgrebe commented Mar 13, 2024

Hello,
first of all, thank you for this great integration!

Since update from 2024.2.3 to 2024.3.1 everytime after Hass restart and also when HA is running an error occurs in the logger and constantly spamming the logger. I´ve tried it 2 times and in every case the error is shown in the log after HA restart an while HA is running. The integration (control the device) works without any problems.

Device in use:
Midea real cool 35 (mobile aircondition)

Error-Log (Screenshot):
image

Error-Log (Text):
Dieser Fehler wurde von einer benutzerdefinierten Integration verursacht

Logger: msmart.device.AC.device
Quelle: custom_components/midea_ac/coordinator.py:38
Integration: Midea Smart AC (Dokumentation, Probleme)
Erstmals aufgetreten: 05:29:58 (1056 Vorkommnisse)
Zuletzt protokolliert: 09:00:44

Payload 'b10109000001003c0020' failed CRC and checksum. Received: 0x20, Expected: 0xBE or 0x8.
Payload 'b10109000001003c0000' failed CRC and checksum. Received: 0x0, Expected: 0xBE or 0x8.
Payload 'b1010900000100300000' failed CRC and checksum. Received: 0x0, Expected: 0xF3 or 0x14.

System in use (Raspberry Pi 4 8GB)
image

Maybe its an fault on my side but I didn´t change anything in my environment.
Would be great if there is an bugfix in an future update - thx. a lot!

Regards
Michi

@mill1000 mill1000 self-assigned this Mar 13, 2024
@mill1000 mill1000 added bug Something isn't working msmart-ng Issue is with underlying msmart-ng library labels Mar 13, 2024
@mill1000
Copy link
Owner

Hi. Thanks for using the integration and sorry about the trouble its causing.

If you don't mind, can you enable debug logging and post the full logs here?

After that, you can try downgrading to 2024.2.3 via HACS to resolve these warnings.

@lesgrebe
Copy link
Author

Hi,

No Problem, give me some time to upload the full logfile.

Thanks for your effort!!

Regards
Michi

@lesgrebe
Copy link
Author

lesgrebe commented Mar 13, 2024

Here the full logfile after enable debug logging. Hope this is helpful. Let me know if I can support you in any way...

Log-File:
home-assistant_midea_ac_2024-03-13T16-12-12.429Z.log

@mill1000
Copy link
Owner

Thanks!

My best guess is it's some device quirk/bug specific to the "new" protocol. The other packets have valid CRCs.

I figure the fix will be detecting when a device sends bad CRCs and disabling the check.

@mill1000
Copy link
Owner

I've got a fixed lined up for you in mill1000/midea-msmart#114

Would you be comfortable testing this? It would require updating the msmart-ng package on your HA instance/container.

@lesgrebe
Copy link
Author

Hmmm...thanks, but the problem is that I do not know how to update my container 😬. Would you be so kind and tell me how?

@mill1000
Copy link
Owner

I'll make it easy for you. I've merged the fix into the main branch.

You can just use HACs to install the main branch instead of a specific release
image

@lesgrebe
Copy link
Author

lesgrebe commented Mar 15, 2024

Hi,
thanks for making my life easier ;)

I´ve installed the "main" branch and did some tests (turn on/off, changed modes fan_only/cool), changed fan speed and temperature) an no errors are logged. When I restartet HA an CRC related error occured a few minutes after restart but it could be that this error is not related to HA restart because it needed some time (4min after restart) until the error occurred?!?

home-assistant_2024-03-15T08-01-25.235Z.log

It is not repeatable, sometimes after restart HA the error occurs and sometimes not. Unfortunately I forgot to enable debug messages. I will enable debug logging and stay on this "main" branch. If any error occurs i will report them...

Thx. a lot!

@lesgrebe
Copy link
Author

Hi again,
now i have an CRC error logged with debug logging on. The error occurs during using the device (perform some tests).

home-assistant_midea_ac_2024-03-15T08-22-16.597Z.log

The CRC error does not constantly spamming the logger. Hope this logfile helps for further investigation.
Thx. for your effort!!!!

@lesgrebe
Copy link
Author

Hi,

also got this error - Unfortunately debug logging was not enabled...

Error from Logger:

Dieser Fehler wurde von einer benutzerdefinierten Integration verursacht

Logger: msmart.device.AC.device
Quelle: custom_components/midea_ac/coordinator.py:38
Integration: Midea Smart AC (Dokumentation, Probleme)
Erstmals aufgetreten: 12:03:36 (1 Vorkommnisse)
Zuletzt protokolliert: 12:03:36

Payload 'b10109000001003c0000' failed CRC and checksum. Received: 0x0, Expected: 0xBE or 0x8.

Regards
Michi

@theshaww
Copy link

Same here. I just enabled debug logging.

@mill1000
Copy link
Owner

Thanks for the logs, at least there's a little improvement.

I guess the solution will need to be a little more involved.

@lesgrebe
Copy link
Author

Let me know if you need anything (testing, debug logs,...). As mentioned above, I will stay on main branch till a newer update is available.

Thank you!

mill1000 added a commit that referenced this issue Mar 15, 2024
Another attempt to solve #101
mill1000 added a commit that referenced this issue Mar 15, 2024
Another attempt to solve #101
@mill1000
Copy link
Owner

Alright just push another update to main that I hope will resolve those final CRC errors.

@lesgrebe
Copy link
Author

Okay merci! I redownload the main branch again and report the results.

@lesgrebe
Copy link
Author

Hi,

Good news, no errors since last "main" branch update. Several checks (HA restart, device use) performed 👍🏽. For me, you can mark this as closed and I will stay on this main branch till a newer version is available.

Now summer can come 😏

Thank you very much!!

@mill1000
Copy link
Owner

That's great to hear! Could you do me one favor and grab another debug log of the device running for about a minute?

There was one additional minor quirk I addressed and would like to confirm it's fixed

@lesgrebe
Copy link
Author

For sure, you will get this debug log in about 5-10 minutes.

@lesgrebe
Copy link
Author

I´ve turned on the device in AUTO MODE and change to FAN_ONLY and change some other parameters like temperature, swing mode on/off, swing position, fan speed high/auto and back again till I turned the device off.

Debug-Log:
home-assistant_midea_ac_2024-03-16T07-03-19.629Z.log

Let me know if i can do more for you!

@lesgrebe lesgrebe changed the title Error "Payload 'b10109000001003c0020' failed CRC and checksum" after HA restart Error "Payload 'b10109000001003c0020' failed CRC and checksum" Mar 16, 2024
@mill1000
Copy link
Owner

mill1000 commented Mar 17, 2024

Released in 2024.3.2

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working device issue Issue is due to device limitation msmart-ng Issue is with underlying msmart-ng library
Projects
None yet
Development

No branches or pull requests

3 participants