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.

Sign up for GitHub

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

Jump to bottom

python3 compatibility #15

Open

andfranklin wants to merge 2 commits into davidwhogg:master from andfranklin:master

andfranklin commented Oct 23, 2020

cube.py and simple_cube.py now work in python3.

andfranklin added 2 commits

October 23, 2020 12:05


          Workaround/simplification of code in simple_cube.py.

679bc26

This change makes simple_cube.py work in python 3.
It is necessary since the class scope is inaccessible from
functions, list comprehensions or generator expressions
enclosed in that scope; they act as if that scope does not exist.

see: https://stackoverflow.com/questions/13905741/accessing-class-variables-from-a-list-comprehension-in-the-class-definition

This code was tested and worked in 3.8.4 & 2.7.14


          Adjusted printing in cube.py.

23d1b93

The code now works in python2 and python3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet