search

captain-coredump / weatherflow-udp

WeatherFlow Personal Weather Station driver for weewx, via UDP broadcast packets
GNU General Public License v3.0
53 stars 18 forks source link

convert to python3 #9

Closed vinceskahan closed 1 year ago

vinceskahan commented 4 years ago

Here are my trivial patches to make your driver work with python3 on Debian 10.0 (Raspbian) on a pi4. Works fine for me here in several months of use. I didn't update any docs or anything, nor do more than a minimal checkout on python2, but I wanted to send the PR before I forget.

FWIW, I also set my shebang line to say python3 in my pi4, but I'm not certain whether that's needed or not. This one's tested using 4.0a7 and python 3.7.3, with weewx installed via "python3 setup.py install", which seems to set everything to use python3 nicely.

captain-coredump commented 4 years ago

I manually merged these changes into my local copy of 1.04 that I have not committed yet, and am running it against the pre-packaged Python 3 stuff in Raspbian Buster on a 4GB RAM Pi 4B and WeeWX 4.0.0b2. The first run bailed after a few minutes, so I'm going to run the tests longer.

What I did to force Python3 was go to /usr/bin and change the /usr/bin/python symlink to point at python3 and not python2. There are arguments on the web against doing this, but I'm only testing for now and plan to wipe the SD card in a few weeks.....