On Friday, January 19, 2018 at 7:26:17 AM UTC-8, Bill Morrow wrote:
>
> See https://github.com/morrowwm/weewxMQTT
>
>
Really cool stuff.....

There are really zero docs currently, but I'm uncertain what architecture 
you're assuming we have for an installation of weewx using this, so 
apologies if I read the code wrong.....

   - port number seems hard-coded to 1883, if the broker is in the cloud 
   someplace that is unwise.  Probably should be configurable.
   - it seems to expect a user+pass for the broker, what if you're running 
   mosquitto in the clear on localhost without accounts needed
   - you use 'station type' which I already have set to Vantage of course. 
    Can you have multiple station types in a weewx installation ?  Can this be 
   added to a vanilla weewx system that is connected to a weather station 
   already ?
   - ultimately, would it be better to do this as an extension that the 
   extension installer could install ?

Lastly it would also be helpful to have unique keys for the key=value pairs 
in your [whatever] blocks so that we can script configuring our boxes with 
stuff like ansible for testing, something like:

# [wxMesh]
# wxMesh_host = localhost # MQTT broker hostname
# wxMesh_topic = weather/+ # topic
# wxMesh_driver = user.wxMesh Really cool stuff - more and more of us are 
interested in integrating MQTT sensors into our weewx setups, so keep going 
!!!!

Reply via email to