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

Added support to Ecowitt WH46 (device 262) #3010

Merged
merged 3 commits into from
Aug 2, 2024

Conversation

joanma747
Copy link
Contributor

I have bought a Ecowitt WH46: https://www.ecowitt.com/shop/goodsDetail/296

I see that is not initially supported in RTL_433 but WH45 is: #2099, #2101

I have reused this line:
rtl_433 -c 0 -R 0 -X "n=DP250,m=FSK_PCM,s=56,l=56,r=1500,preamble=0x2dd4"
to get some samples of the data extracted and comparing with the WH45 it seems very similar, so I created a new device and tried the compiled application for a day. It seems to work just fine.

I hope you are willing to incorporate support to this sensor based on my implementation.

Thanks

@zuckschwerdt
Copy link
Collaborator

Seems really good. Change all tabs to (4) spaces please :)

@joanma747
Copy link
Contributor Author

I was not aware of the style rules. Sorry, Tabs and trailing removed.

@zuckschwerdt zuckschwerdt merged commit 3821498 into merbanan:master Aug 2, 2024
8 checks passed
@ProfBoc75
Copy link
Collaborator

FYI: README.md has been updated accordingly

@joanma747
Copy link
Contributor Author

Nice! Thank you for considering my piece of work.

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

Successfully merging this pull request may close these issues.

3 participants