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

Add support for parsing WH57 (lightning) data for the EcowittClient #72

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

claudobahn
Copy link

@claudobahn claudobahn commented Jul 8, 2020

Note that the wview_extended schema provides support for some of the lightning data, now, and my pull request for adding default unit assignments (weewx/weewx#581) has been merged. I've got a second PR pending for correcting the accumulator's extraction behavior (weewx/weewx#582).

bin/user/interceptor.py Outdated Show resolved Hide resolved
@claudobahn
Copy link
Author

@matthewwall I'm hoping for some guidance from you for pulling in lightning_distance. Basically I'd like to check lightning_time to see if it falls within the current accumulator window and if it does, pull in the distance (which still needs to be converted from km to miles), otherwise discard that value.

Are you aware of how to do that with the existing weewx API?

@floydsteam
Copy link

Hello together, seems like this PR draft was opened more than a year ago.
How's the progress? I just got my own sensor today and couldn't get it working with the interceptor.
I run my WeeWX in a docker (felddy/weewx-docker) and it is working fine for the standard sensors and an additional one.
I figured out the sensor data for the WH57 is not mapped and therefore it can't be passed to WeeWX.
There are many users that have the same issue and probably wait for a solution.
So is it planned to make it working? It seems like this PR could close the gap.
At least the count and distance of lightnings can be send to the WeeWX database.

@fra-venti
Copy link

I managed to get the WH57 work with the interceptor driver

@fra202088
Copy link

#99 I've have been using it for two months, and it seems to work properly

Hello together, seems like this PR draft was opened more than a year ago. How's the progress? I just got my own sensor today and couldn't get it working with the interceptor. I run my WeeWX in a docker (felddy/weewx-docker) and it is working fine for the standard sensors and an additional one. I figured out the sensor data for the WH57 is not mapped and therefore it can't be passed to WeeWX. There are many users that have the same issue and probably wait for a solution. So is it planned to make it working? It seems like this PR could close the gap. At least the count and distance of lightnings can be send to the WeeWX database.

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.

4 participants