#8987: `
----------------------------------+-----------------------------------------
   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 vbraun):

 I agree that there is some rationale for calling individual cones to be
 faces of the fan, but that is not the standard terminology and I think it
 would be confusing. So I would go with 3)
   * A cone is a strictly convex polyhedral cone
   * Faces are sub-cones of cones
   * A fan is a collection of cones, closed under "taking faces".
 I think the rationale here is that (the support of) the fan is not
 necessarily convex, so its easier to keep fans and cones separate.

 But i'm not opposed to keeping `Fan.ambient()` to refer to either a cone
 or a fan (or anything that behaves suitable). To safely use duck-typing it
 would be nice to write down exactly what you expect `ambient` to provide.
 I guess we want the following to be available, but not necessarily
 anything else:
   * `ambient().nrays()`
   * `ambient().rays()`
   * `ambient().ray_set()`
   * `ambient().ray_indices()`

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8987#comment:31>
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