Re: the added loginf, I wanted to let this version run for several days. If it is stable up to then I plan to remove the unnecessary loginf. Congrats on the new grandkid! And thanks again for creating the HP1000 driver.
On Monday, April 8, 2019 at 10:48:19 PM UTC-5, vk3...@gmail.com wrote: > > I'm not sure what this means but if it works then great. > > On Monday, April 8, 2019 at 8:59:13 PM UTC+10, Ron wrote: >> >> Oh, in my repo I also changed the formatting of the installation steps in >> the README file. Githubs markdown was smushing them all onto a single line >> and eating the portions of the commands saying to insert the HP1000 >> directory. > > > You have uncovered a bug in my code that has been there all along with > regard to the 'sleep' command. As I mentioned above, there is a > 'sleep'command in the genLoopPackates function that stops the weather > station being accessed too frequently. in *that* call to 'sleep' I *do* > prefix it with 'time.' but in all other cases I don't. I guess it shows > that I hadn't checked all of the error code paths well enough!!! > > The only word of warning I'd make is that anyone using your code will get > the additional 'loginf' messages as it is reading in the history > packets.(BTW, in line 785 of your file, the 'loginf' is after the 'raise' > so probably won;t get actioned.) > > Thanks for the assistance on this - I'll update my repository over the > next few days as time permits (my daughter is about to make me a 'Grannie' > again). > > Susan > -- 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 weewx-user+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
