#13312: Polyhedral bugfixes and improvements
-------------------------------------+-------------------------------------
Reporter: mhampton | Owner: mhampton
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.1
Component: geometry | Resolution:
Keywords: polyhedra, polar | Merged in:
Authors: Marshall Hampton, | Reviewers:
Volker Braun | Work issues:
Report Upstream: N/A | Commit:
Branch: | 9e2fe957e16c53fe39df2d58cb739b0dc416bfd6
u/vbraun/polyhedral_neg_polar | Stopgaps:
Dependencies: #11763 |
-------------------------------------+-------------------------------------
Comment (by ncohen):
Yooooooooooooo !!
I agree with this patch, though I don't understand one bit : why this
change in `.polar` ?
{{{
#!python
- verts = [list(v.vector() - self.center()) for v in
self.vertex_generator()]
+ verts = [list(self.center() - v.vector()) for v in
self.vertex_generator()]
}}}
If it is just to have your example of code work, shouldn't you change the
constructor instead ?
By the way, running `polytopes.n_cube(3).show()` just gives me a black
Jmol window. I mean, empty. It's like the world is missing a cube.
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/13312#comment:17>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.
