Commits · a87a94d4e9defcf7e5d0a6729e4b4b13bdde8f99 · machinery / pyMKcam

24 Mar, 2012 1 commit
- Removed old commented out code and converted PolygonExtractor.py to new Point tuple style. · 73cb1bb0
  Whitham D. Reeve II authored 13 years ago
  
  73cb1bb0
05 Mar, 2012 1 commit

Replace Point class with a 3 position tuple and a collection of helper functions. · 3ca79c3c

Whitham D. Reeve II authored 13 years ago

PyCam has been spending about 75% of it's time creating point objects. I decided to replace
the pycam Point object with a 3 position tuple, and the pycam Vector object with a 4 position tuple.

All of the point object methods were rewritten to accept and emit tuples. The transform_by_matrix method
was rewritten to accept both 3 and 4 position tuples with behavior dependent on that size. The
normalized method is the only method required to emit the same kind of object that it is called on, and
this has been taken care of with the tuple version.

What does this mean?
All instances of Point(x,y,z) have been converted into (x,y,z)
All instances of Vector(x,y,z) have been converted into (x,y,z,'v')
The notation to access the x,y,z of the Point objects has been been changed
from p.x,p.y,p.z to p[0],p[1],p[2]
The notation for the point math functions has been completely changed.
Instead of p1.sub(p2) it has been converted to psub(p1,p2)
Instead of p1.sub(p2).mul(0.5) it has been converted to pmul(psub(p1,p2), 0.5)

It is very important to point out that the tuple is an immutable type.
You can not change it once you create it.
t[0] = calculated_x_value # syntax error
You have to replace the reference (t) to the old tuple with a new reference to a new tuple.
t = (calculated_x_value, t[1], t[2]) # works

There was a particularly hairy mutable/immutable barrier in the next() generator present in each class that
inherits TransformableContainer. The TransformableContainer.transform_by_matrix function uses these
generators to collapse polygons and lines and triangles into points where a shift is performed on each.
Now that point is immutable, you can not change the value emitted by the generator.
Geometry/__init__.py transform_by_marix and the associated next() generator in each sub class has been
rewritten to transfer the attr used to store the reference to the tuple.
Geometry/__init__.py transform_by_matrix now sets these attributes which effectively gets around this
limitation.

There could be some old point object code in any of the files not contained in this commit.
It is impossible to know without running the code path and/or careful analysis of each file.

Some list comprehensions that convert a list of point objects into a 3 position tuple have been removed.

Whitham D. Reeve II

3ca79c3c

21 Jan, 2012 1 commit
- Rearrange to a more standard python package layout · 5aad95d9
  Paul Bonser authored 13 years ago
  
  5aad95d9
15 Jan, 2012 1 commit

added "__slots__" definitions to memory-relevant classes · 32855e03

sumpfralle authored 13 years ago

* this decreases memory consumption significantly
* see http://docs.python.org/release/2.5.2/ref/slots.html


git-svn-id: https://pycam.svn.sourceforge.net/svnroot/pycam/trunk@1241 bbaffbd6-741e-11dd-a85d-61de82d9cad9

32855e03

19 Jul, 2011 1 commit

mirgrated the path generators to a more uniform interface · a19f544c

sumpfralle authored 13 years ago

added a motion-grid generator for engraving
separated the toolpath visualization

git-svn-id: https://pycam.svn.sourceforge.net/svnroot/pycam/trunk@1125 bbaffbd6-741e-11dd-a85d-61de82d9cad9

a19f544c

07 Feb, 2011 1 commit

more style fixes · dfbffd2b