Skip to content
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

update project to python 36 #2

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ZeeWanderer
Copy link

@ZeeWanderer ZeeWanderer commented Sep 21, 2018

  • Added standard python .gitignore from https://github.com/github/gitignore;
    modified it to exclude .png and .bak files so repo would not be polluted by IDE or Editors byproducts and raycaster execution products;
    added .idea dir for PyCharm
  • Fixed commas in pointLight.py.
  • converted code using futurize and autopep8
  • fixed PIL includes
  • Fixed None compassion for hit.etry and hit.exit by substituting None with float('-inf');
    Modified condition statements;
  • Added helpers.isninf to avoid mistaking float('inf') for float('-inf')
  • Substituted old_div with corresponding python3 div operators in used code

Note: Requires Pillow which is a PIL fork;
Also, could you create a separate branch for python3x or something so I could merge it there instead of master?
This repo is a good piece of code so I would like to keep it up to date with modern python, maybe someone else will use it for something.

add standart .gitignore from https://github.com/github/gitignore
add .png to ignored formats.
Add .bak to gitignore
Fix commas in pointLight.py
Workaround for None comprasion and __div__ operator
None swithed to `float('-inf')` for hit.entry and hit.exit.
Added `helpers.isninf` so `float('inf')` would not be mistaken for float('-inf').
`old_div` substituted for corresponding python3 division operators in USED code.
import helpers changed to `from helpers import isninf` to lessen lenght of condition statements.
@ZeeWanderer ZeeWanderer changed the title Minor update. update project to python 36 Sep 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant