theavey/ParaTemp

Python 2/3 compatibility

theavey opened this issue · 2 comments

Some of this code works (only?) with Python 2, other parts with Python 3. That's not good.

For my code, it should be fairly simple to make it work across versions, but some of this is because of the compatibility of imported code.
This should be documented well so it's clear to users what works with which Python.

MDAnalysis now officially supports Python 3.4+, so there's nothing holding me back now, I think.

Possibly GromacsWrapper, but I don't recall how much that is used. Might be worth getting rid of it as a dependency depending on how up to date it is.

Well it seems that this works with Python 3.6 already (d49c7b6) based on Travis-CI testing. None of the dependencies threw errors on installation and all tests passed. This is not extensive testing of the Python 3 compatibility, but it looks like it should be good.