Added support for Python 3.x

mdolab / pyofm

Python wrapper for OpenFOAM meshes

Other

7 stars 4 forks source link

Added support for Python 3.x #2

Closed friedenhe closed 4 years ago

friedenhe commented 4 years ago

Purpose

Added support for Python 3.x

Type of change

New feature (non-breaking change which adds functionality)

Testing

The DAFoam reg tests that use this repo passed for both Python 3.x and Python 2.7

Checklist

Put an x in the boxes that apply.

[x] I have run unit and regression tests which pass locally with my changes
[ ] I have added new tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation

friedenhe commented 4 years ago

Yes, it should be just python 3 fixes, but for some reason, it shows two commits... I guess it should be fine.

friedenhe commented 4 years ago

Yes, that is only needed for .gz files with Python 3.x, and it is also compatible with Python 2.7. For ASCII files, the byte option is not needed and works for both 2.7 and 3.x.