Python3 version? - Githubissues

Woutrrr / Omnik-Data-Logger

Data logger for Omnik Solar Inverters

GNU General Public License v3.0

40 stars 42 forks source link

Python3 version? #13

Open edsub opened 10 years ago

edsub commented 10 years ago

Hi, I have tried to convert your code to Python3 (as I have some dataloggers running that are in Python3), but I seem to get stuck in bit where the juggling with the string/bytes/hex values takes place. (the InverterMsg.generate_string logic)

Anyone who has accompoished this allready?

slwessels commented 7 years ago

Hi, I'm interested in getting the same done. Did you make any progress on this?

clmcavaney commented 4 years ago

Righto - I think I might have got this working. I will clean up my code a little and then submit a pull request.

LaszloBalder commented 3 years ago

Hi, I'm interested in your python3 solution for the InverterMsg implementation, can you make it public?

thnx in advance.

ervee commented 3 years ago

I didn't try, but: https://github.com/clmcavaney/Omnik-Data-Logger/tree/python3-change

jdgmeester commented 4 months ago

Hi, I have tried to convert your code to Python3 (as I have some dataloggers running that are in Python3), but I seem to get stuck in bit where the juggling with the string/bytes/hex values takes place. (the InverterMsg.generate_string logic)

Anyone who has accompoished this allready?

I have exactly the same:

ConfigParser was renamed to configparser (change in OmnikExport.py)
Print requires parentheses now (change in InverterMsg.py and ConsoleOutput.py)
the decode feature is not accepting the hex value anymore. I have moved to the webbased scraper for now, but the CLI script has way more data available.

clmcavaney commented 4 months ago

I didn't try, but: https://github.com/clmcavaney/Omnik-Data-Logger/tree/python3-change

?

I did submit a pull request (https://github.com/Woutrrr/Omnik-Data-Logger/pull/44) but I don't think it was merged (yet)