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

Clarify the proxy mode #26

Closed
3 tasks done
toxic0berliner opened this issue Aug 29, 2024 · 2 comments · Fixed by #29
Closed
3 tasks done

Clarify the proxy mode #26

toxic0berliner opened this issue Aug 29, 2024 · 2 comments · Fixed by #29
Labels
enhancement New feature or request

Comments

@toxic0berliner
Copy link

Checklist

  • I have filled out the template to the best of my ability.
  • This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
  • This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.

I did setup the integration.
I believe that it will accept both WeatherUnderground and WeatherCloud and display them in home-assistant regardless of the "proxy" checkbox during the setup, but I might be wrong.

If not the case, please disregard the proposed solution below, more clarification is required maybe elsewhere :D

Describe the solution you'd like

Clarify on the setup screen that "proxy WU" or "proxy WC" simply enables the forwarding to the cloud of the recieved metrics, leaving it unchecked will still display the values in HA

Describe alternatives you've considered

N/A

Additional context

N/A

@lhw
Copy link
Owner

lhw commented Aug 30, 2024

Thats actually a fair point. I wanted to introduce some changes to the configflow anyway. Reconfiguration was part of that. I'll try to make it more readable and precise.

But yes your assumption was correct. It will always listen for all supported APIs eitherway. The proxying is just an extra feature one can enable to forward the metrics to their intended data sink.

@toxic0berliner
Copy link
Author

That being said the integration is very usable and working fine just fine for me thanks a lot for the hard work!
I just realized though that with this system I'm subject to the publish rate of the device and not able to increase it. First time for me with a weather station so I'll check over a few weeks most probably havinghigher time resolution is simply overkill, let's see 😉
I think this integration is here to stay in my home setup, almost sad I didn't have to try out SDR to intercept the signals from the sensor to the base, sounded kinda fun 😂

@lhw lhw added the enhancement New feature or request label Nov 30, 2024
@lhw lhw closed this as completed in #29 Dec 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants