lhw / cloudweatherproxy

A HA integration that functions as proxy for weatherstations publishing to Weather Underground and Co
MIT License
4 stars 0 forks source link

Clarify the proxy mode #26

Open toxic0berliner opened 2 weeks ago

toxic0berliner commented 2 weeks ago

Checklist

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 commented 2 weeks ago

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 commented 2 weeks ago

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 😂