simon-git: polyhedra (master): Simon Tatham

[Commits to Tartarus hosted VCS]

TL;DR:
  2abe59f Clean up relics of a removed diagnostic.
  3a3a41c Make drawnet.py work without --adjacent.
  7bc1105 Factor common code out into a library module.
  f1885f0 Simplify and rename isometry.py.
  cb20de3 cleanpoly.py: add a verbose option, off by default.

Repository:     https://git.tartarus.org/simon/polyhedra.git
On the web:     https://git.tartarus.org/?p=simon/polyhedra.git
Branch updated: master
Committer:      Simon Tatham <anakin at pobox.com>
Date:           2019-07-09 19:30:02

commit 2abe59fd88da3ca6c9ae5a1468bd550f7c2d5441
web diff https://git.tartarus.org/?p=simon/polyhedra.git;a=commitdiff;h=2abe59fd88da3ca6c9ae5a1468bd550f7c2d5441;hp=8bdb8fe12a3396bf3ad88f828ef76faacf66c661
Author: Simon Tatham <anakin at pobox.com>
Date:   Tue Jul 9 18:09:59 2019 +0100

    Clean up relics of a removed diagnostic.
    
    While debugging the Knuth convex-hull algorithm in zonohedron.py, I
    wrote some code that built up a list at the end of each iteration and
    then printed it. Once it was working, I removed the print statement,
    but not the now-useless code that actually built the list.

 zonohedron.py | 7 -------
 1 file changed, 7 deletions(-)

commit 3a3a41c3336fc7125961cdbfb504437fce2825dc
web diff https://git.tartarus.org/?p=simon/polyhedra.git;a=commitdiff;h=3a3a41c3336fc7125961cdbfb504437fce2825dc;hp=2abe59fd88da3ca6c9ae5a1468bd550f7c2d5441
Author: Simon Tatham <anakin at pobox.com>
Date:   Tue Jul 9 18:11:35 2019 +0100

    Make drawnet.py work without --adjacent.
    
    While doing the argparse rewrite, I implemented that flag last, and
    tested it with as the last thing I did, forgetting that I hadn't
    installed an empty list as the default option value, so now it would
    no longer work _without_ that option.

 drawnet.py | 1 +
 1 file changed, 1 insertion(+)

commit 7bc1105641c170267652075cee0a153bfd22d94d
web diff https://git.tartarus.org/?p=simon/polyhedra.git;a=commitdiff;h=7bc1105641c170267652075cee0a153bfd22d94d;hp=3a3a41c3336fc7125961cdbfb504437fce2825dc
Author: Simon Tatham <anakin at pobox.com>
Date:   Tue Jul 9 18:12:40 2019 +0100

    Factor common code out into a library module.
    
    This includes general vector-handling routines; code to load and save
    lists of points and my polyhedron description files; and now I have a
    general-purpose library module, crosspoint() can live in it too.

 canvas.py            |  37 +-----
 cleanpoly.py         |  49 ++------
 crosspoint.py        |  75 ------------
 describe.py          |  37 +-----
 drawnet.py           |  38 +-----
 drawpoints.py        |  21 +---
 drawpoly.py          |  36 +-----
 dual.py              |  59 ++--------
 flashpoly.py         |  37 +-----
 isometry.py          |  55 +++------
 misc/dihedral.py     | 123 ++++++++-----------
 mkpoints.py          |  20 ++--
 nfaces.py            |  40 +++----
 nvertices.py         |  59 ++--------
 polylib.py           | 326 +++++++++++++++++++++++++++++++++++++++++++++++++++
 povpoly.py           |  56 ++-------
 regular/mkregular.py | 170 ++++++++-------------------
 untangle.py          |  37 +-----
 zonohedron.py        |  69 +++--------
 19 files changed, 563 insertions(+), 781 deletions(-)

commit f1885f05942982f61cff5d4f1a8d96ef378dd505
web diff https://git.tartarus.org/?p=simon/polyhedra.git;a=commitdiff;h=f1885f05942982f61cff5d4f1a8d96ef378dd505;hp=7bc1105641c170267652075cee0a153bfd22d94d
Author: Simon Tatham <anakin at pobox.com>
Date:   Tue Jul 9 19:17:35 2019 +0100

    Simplify and rename isometry.py.
    
    Now that it's using the centralised facility to recalculate face
    normals, there's no longer any reason why it should have to enforce
    that its input matrix is orthogonal. The only reason it had to do that
    at all was because it was transforming the input face normals through
    the same matrix as the vertices. But now it can apply any old matrix
    to just the vertices, and let polylib sort out the normals.
    
    Also, I've reinstated the ability to translate the polyhedron, which
    was accidentally lost in the argparse transition, although argparse
    requires me to do it with a different syntax.

 isometry.py  | 94 ------------------------------------------------------------
 transform.py | 70 ++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 70 insertions(+), 94 deletions(-)

commit cb20de3cb2f9875eaf46df5c8f655c29d0395217
web diff https://git.tartarus.org/?p=simon/polyhedra.git;a=commitdiff;h=cb20de3cb2f9875eaf46df5c8f655c29d0395217;hp=f1885f05942982f61cff5d4f1a8d96ef378dd505
Author: Simon Tatham <anakin at pobox.com>
Date:   Tue Jul 9 19:23:19 2019 +0100

    cleanpoly.py: add a verbose option, off by default.
    
    Now I don't get bothered by the details of what it's been doing,
    unless I actually ask for them.

 cleanpoly.py | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)