#8987: Add support for rational polyhedral fans
----------------------------------+-----------------------------------------
   Reporter:  novoselt            |       Owner:  mhampton  
       Type:  enhancement         |      Status:  needs_work
   Priority:  major               |   Milestone:  sage-4.4.4
  Component:  geometry            |    Keywords:            
     Author:  Andrey Novoseltsev  |    Upstream:  N/A       
   Reviewer:  Volker Braun        |      Merged:            
Work_issues:                      |  
----------------------------------+-----------------------------------------

Comment(by novoselt):

 1) Yes, I thought about renaming it or making it hidden since
 `containing_cones` and `cone_containing` are so similar yet give quite
 different result. Thanks for pointing out that it is unused anymore at
 all. In fact, I have repeated its code in `cone_containing` when I needed
 its functionality, so I guess it was not something extremely natural
 anyway. Will be gone.

 2) There was a typo, will be fixed and your example goes into doctests!

 I also like your suggestion for `_repr_` strings, except that I would like
 to insert `-`:
 {{{
 sage: cone_face
 1-d face of 2-d cone in 3-d lattice N
 }}}
 I think it makes it a little easier to see the actual dimension and in the
 case of "1d" there is less confusion with "ld".

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8987#comment:42>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to