-
Notifications
You must be signed in to change notification settings - Fork 137
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Python 3 support #16
Comments
Hello, |
https://mehodin.com/i/9447dcac.png <- Getting that error tho, how to fix? |
There's a set of fixes necessary (imports, print-statements, fixing setup.py, handling bytes vs strings) to get memorpy to run properly under python3, but I have a version running. I'll make a PR in a day or two - but I've only tested it under Windows. As there seems to be no tests available I can't confirm that I haven't broken something outside of the functions I'm using myself. |
I've added a #20 as an initial set of fixes to make memorpy work under Python 3. Since my usage was limited to Windows this time, I haven't tested the patch with other OS-es or that it is in any way backwards compatible with Python 2. |
Hello,
are you planning on supporting Python 3, please?
If I wanted to try to make it work in Python 3, was there any particular problem with Python 3 that you've encountered (e.g. are there some dependencies which also don't support it)?
Thanks :)
The text was updated successfully, but these errors were encountered: