Skip to content

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