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

Acurite Sensor 00592TX signals stopped with release 22.11 #2457

Closed
NorthernMan54 opened this issue Mar 27, 2023 · 2 comments
Closed

Acurite Sensor 00592TX signals stopped with release 22.11 #2457

NorthernMan54 opened this issue Mar 27, 2023 · 2 comments

Comments

@NorthernMan54
Copy link
Contributor

@rct @DAVe3283 I recently updated my rtl_433 installation to release 22.11, and found that after the update some of my Acurite Tower sensor devices stopped reporting. I have double clicked on the issue, and found that the pr #1953 and the signal validation checks based on humidity introduced in the PR are the trigger for the problem.

When running analysis on the signal I'm seeing this in the log

acurite_txr_callback: row 1 bits 57, bytes 7, extra bits 1, msg type 0x04
acurite_tower_decode: 0x1765 Ch A : Impossible humidity: 127 %rH
acurite_txr_callback: stats: row 1, msg type 0x04, bytes 7, decoded 0, error -4
acurite_txr_callback: row 2 bits 57, bytes 7, extra bits 1, msg type 0x04
acurite_tower_decode: 0x1765 Ch A : Impossible humidity: 127 %rH
acurite_txr_callback: stats: row 2, msg type 0x04, bytes 7, decoded 0, error -4
acurite_txr_callback: row 3 bits 57, bytes 7, extra bits 1, msg type 0x04
acurite_tower_decode: 0x1765 Ch A : Impossible humidity: 127 %rH
acurite_txr_callback: stats: row 3, msg type 0x04, bytes 7, decoded 0, error -4

And after reviewing the code found that this line is the issue -

if (humidity < 0 || humidity > 100) {

Acurite 00592TX - The problematic sensor ( it does not have a humidity sensor )
IMG_7593

Acurite 0592TXR - Another sensor I had received with the Acurite tower, this one supports humidity.
IMG_7592

In my setup I did change the problematic line to if ((humidity < 0 || humidity > 100) && humidity != 127 ) { and the previous behaviour was restored.

@zuckschwerdt
Copy link
Collaborator

Thanks, Support for 592TX added.

@NorthernMan54
Copy link
Contributor Author

Tks very much for this, I have updated my installation and can report that it is working fine.

andrewjw pushed a commit to andrewjw/rtl_433 that referenced this issue Sep 29, 2023
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

2 participants