Changes between Initial Version and Version 7 of Ticket #20259


Ignore:
Timestamp:
03/31/16 12:44:01 (4 years ago)
Author:
jdemeyer
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #20259

    • Property Status changed from new to needs_work
    • Property Commit changed from to 496646879d95208d240ea521340f41029cf9bdfc
    • Property Dependencies changed from to #20279, #20287
    • Property Branch changed from to u/jdemeyer/polyhedron_restricted_automorphism_group____allow_returning_the_affine_matrix_group
  • Ticket #20259 – Description

    initial v7  
    11Currently, the method `Polyhedron.restricted_automorphism_group()` returns an abstract permutation on the vertices. In this ticket, we optionally allow returning a concrete affine matrix group on the ambient space.
     2
     3We also fix this bug:
     4{{{
     5sage: len(polytopes.dodecahedron().restricted_automorphism_group())
     6---------------------------------------------------------------------------
     7UnboundLocalError                         Traceback (most recent call last)
     8<ipython-input-6-d332241c23a2> in <module>()
     9----> 1 len(polytopes.dodecahedron().restricted_automorphism_group())
     10
     11/usr/local/src/sage-config/src/sage/misc/cachefunc.pyx in sage.misc.cachefunc.CachedMethodCallerNoArgs.__call__ (build/cythonized/sage/misc/cachefunc.c:12404)()
     12   2233         if self.cache is None:
     13   2234             f = self.f
     14-> 2235             self.cache = f(self._instance)
     15   2236         return self.cache
     16   2237
     17
     18/usr/local/src/sage-config/local/lib/python2.7/site-packages/sage/geometry/polyhedron/base.pyc in restricted_automorphism_group(self)
     19   4619                 v_i = v_list[i]
     20   4620                 v_j = v_list[j]
     21-> 4621                 c_ij = rational_approximation( v_i * Qinv * v_j )
     22   4622                 G.add_edge(i+1,j+1, edge_label(i,j,c_ij))
     23   4623
     24
     25UnboundLocalError: local variable 'rational_approximation' referenced before assignment
     26}}}