On Sunday, February 14, 2021 at 8:10:44 PM UTC+1 michael.k...@gmx.at wrote:

> seems to work!
>

Great news! Thanks for letting me know! :)
 

> I made two local changes: 
>
1.) I removed the 
>
>     @property
>     def archive_interval(self):
>         return 60
> And changed the logging of "Import from UDP" to debug level. The driver 
> works now for me, the last thing on the wishlist would bei to let 
> genStartupRecords only generate one entry per archive_interval. I might do 
> that myself if I find some time.
>

I'll incorporate these changes yes :)
 

> Good work! And thank you for the driver enhancements!
>

You're welcome. Did you find out why the REST part didn't work initially? 
>From your earlier explanation, I don't really see why it wouldn't work for 
you?

A bit surprised you didn't take up an issue that I had expected you'd 
mention: The skin you use seems to actively use the wind packages that come 
much more often in the UDP messages, but which I have removed, at least 
when it comes to the default sensor maps. And if you want to use these 
nonetheless, by using an explicit sensor map, you will get problems when 
using the REST interface as these are not really compatible as it is.

I thought we didn't loose too much functionality by doing that in the 
driver, but I now understand that there actually is a use case for that 
stuff as well.

I have some ideas how to address that for a future version.

Cheers,
Jan-Jaap

-- 
You received this message because you are subscribed to the Google Groups 
"weewx-development" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to weewx-development+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/weewx-development/8db6ace3-eda7-4cc7-9742-4e2bd1648613n%40googlegroups.com.

Reply via email to