On Tuesday, January 3, 2017 at 5:47:04 PM UTC-5, lionel.sylvie wrote:

This is the same corrections in the weewx.conf between versions.
> Where I can find the commits?
>

the change to rain multiplier happened on 11 jun 2015:

https://github.com/weewx/weewx/commit/0bb7db1edc806caebe68357bc315ccf8674eea52

so it has been part of the ws23xx driver for some time now.

the divide-by-ten happens only to data from the logger, so if you are using 
record_generation=hardware you will definitely see a difference, but if you 
are using record_generation=software you will see a difference only when 
weewx has to do catchup.

 

>
>  
>>
>>> PS: In the WS23xx.py driver it's necessary to have a delay in the delay 
>>> read definition :
>>> This sleep time allows to solve the hazardous or wrong reading data from 
>>> the station. It's due to communication between the station and the outside 
>>> sensors in the same time.
>>>
>>
>> have you been using this delay since weewx 2.7?  are you running it now 
>> with weewx 3.6?  what happens when you do not use the delay?
>>
>> Yes I already used this delay in the WS23xx.py in the 2.7 and added it in 
> the 3.5 in june and now in the 3.6. This solves the spikes issues in the 
> temperature measurement and calculation which arrive randomly. I had 
> mentioned this issue in December 2014 in the LaCrosse WS2300 & WS2350 
> support post. 
>

adding a slight delay seems like an innocuous change, especially if it 
reduces or eliminates spikes.  on the other hand, magic numbers for serial 
communications make for some pretty stinky code.

i added the delay at commit 61b866a

ws23xx driver version is now 0.26rc1

m

-- 
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].
For more options, visit https://groups.google.com/d/optout.

Reply via email to