matthewwall / weewx-wh23xx

weewx driver for fine offset wh23xx hardware (tycon tp2700)
GNU General Public License v3.0
10 stars 9 forks source link

fix for python3 #12

Open ra1u opened 4 years ago

ra1u commented 4 years ago

It is tested on WH2315 tested with python3 weewx 4.1.1 on debian 10 and with python2 weewx 3.9.0 on debian 9 where it also works.

On python3 it has some issues with logging but I have no idea what is going on as it seem to be out of this library, but rather in weewx or my misuse.

Other that that it works for me.

olwaldi commented 3 years ago

I'm using your branch for my WH4000 weather station. To get rid of debugging messages within /var/log/debug I've chosen a brute force approch: I've mapped the function logmsg onto pass

def logmsg(level, msg):
#    syslog.syslog(level, 'wh23xx: %s' % msg)
    pass

Maybe that can serve as a temporary workaround.

And thanks for your work to use wh23xx within python 3. Without it I couldn't upgrade.

fankyy commented 2 years ago

Is this included in the installation files in the meantime ?