No, all get_site_dict does is check that the listed config options exist and are not the install defaults. The real issue is that _ambient_dict is used to pass keyword arguments to class AmbientThread init, apiKey is not a parameter in the init signature so hence the error. One approach will be to simply add an apiKey=None parameter to the AmbientClass init signature, it won't be used but who knows where WU may end up, maybe it could be thought of as a bit of future proofing/planning. Another approach would be to have a separate [Wunderfixer] stanza in weewx.conf, seems a waste to me and it makes sense to have the WU parameters together in the one place. Maybe leave it as a command line parameter only for wunderfixer, to me that seems almost the same as having a separate [Wunderfixer] stanza. Maybe introduce whatever approach in 4.0 and leave it as a wunderfixer command line parameter for the (expected) 3.9.2 release. Probably one for guidance from Tom.
Gary -- You received this message because you are subscribed to the Google Groups "weewx-user" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/weewx-user/68bfe21d-d0a2-4c55-9e71-2d65a69eea03%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
