Point class v2.0

[Nabushika]

Made point class use a position tuple, so can use 2, 3, or more dimensions. Should be a drop in replacement for the old point.

Changes

• .gitignore - ignore GUI config
• map.py - changed incorrect not tuple to tuple is None
• structures.py - using pos tuple for generality, plus x, y, z properties for backwards compatibility.
  • Had to override __new__ for torch.Tensor allocation
  • Change to_tensor/from_tensor to support arbitrary dimensions
  • Overrode __lt__ - doesn't make much sense, but is necessary for heaps (need items to be orderable) and Python supports lexicographic ordering or tuples
  • __repr__/__init__ specialise for 2D/x,y - this was mainly to pass the tests as-is.
  • Added tests

Future possible things to do/questions to reviewers

• Add operators to point, so code doesn't need to do diff = Point(dst.x - src.x, dst.y - src.y)
• Maybe allow taking z as a keyword argument?

Things to review

• Does this code require breaking changes?
• Is all code clean and readable?
• Is code tested?
• Are all changes relevant?

[judicaelclair]

• Definitely need to homogenise __repr__ - same style regardless of the dimension (a future todo).
• test_ne_pos only checks for changes in y-coordinate, but changes in x / z aren't tested.

[Nabushika]

• Definitely need to homogenise __repr__ - same style regardless of the dimension (a future todo).
• test_ne_pos only checks for changes in y-coordinate, but changes in x / z aren't tested.

Both should be fixed in the latest commit -- __repr__ style is now consistent and reflected in tests.