Yep - Spotted that myself on Sunday. Thanks.

Also took the advice about removing the extraneous lines form the 
install.py file - much better now. (I copied this from one of the example 
files pretty much without understanding the relation ship between these 
lines and the code in the main file to add in the config options. I think I 
have this now but I didn't see anything about this in the docs. Perhaps 
this could be made a bit clearer??)

I think I'm nearly ready to release this but I'd still like a comment on 
hwo to best handle the new version (V3.7.0)

Susan

On Tuesday, February 21, 2017 at 12:27:29 PM UTC+11, dr__bob wrote:
>
> Hi Susan,
>
> this is a very minor bug, but I found it while looking through my log 
> files.  There's a missing "%" on an strftime string.  Yup, I did say minor! 
> Here's the diff to HP1000.py:
>
> *** HP1000.py 2017-02-05 02:15:01.000000000 -0800
>
> --- /usr/share/weewx/user/HP1000.py 2017-02-20 16:52:10.466575082 -0800
>
> ***************
>
> *** 237,241 ****
>
>   
>
>                       loginf('Established contact at %s' %
>
> !                            datetime.datetime.now().strftime('%d/%m/%y 
> H:%M:%S'))
>
>   
>
>                       # Now read the SETUP packet to find the current 
> value units
>
> --- 237,241 ----
>
>   
>
>                       loginf('Established contact at %s' %
>
> !                            datetime.datetime.now().strftime('%d/%m/%y 
> %H:%M:%S'))
>
>   
>
>                       # Now read the SETUP packet to find the current 
> value units
>
> Bob
>
> PS: the subject of this thread should probably be changed to something 
> more meaningful/useful.
>
>

Reply via email to