#10817: implementation of the generalized associahedron as a polyhedral complex
----------------------------------------------------+-----------------------
   Reporter:  stumpc5                               |          Owner:           
      
       Type:  enhancement                           |         Status:  
positive_review
   Priority:  major                                 |      Milestone:  sage-5.0 
      
  Component:  combinatorics                         |       Keywords:  
associahedra   
Work_issues:                                        |       Upstream:  N/A      
      
   Reviewer:  Frédéric Chapoton, Nicolas M. Thiéry  |         Author:  
Christian Stump
     Merged:                                        |   Dependencies:           
      
----------------------------------------------------+-----------------------
Changes (by nthiery):

  * status:  needs_review => positive_review


Comment:

 Replying to [comment:25 stumpc5]:
 > Replying to [comment:24 nthiery]:
 > > Replying to [comment:23 stumpc5]:
 > > Then the result should be sorted according to their string
 > > representation, and string comparison should be platform independent.
 >
 > Done

 Thanks! Positive review, assuming the tests pass.

 > how can I know that string representations should be platform
 independent and some others aren't?

 Because:

 - I am amost certain that Python compares strings lexicographically (it
 does for e.g. tuples; see http://docs.python.org/library/stdtypes.html;
 it's certainly specified somewhere for strings too)
 - The Inequality class implements _repr_ independently of the platform
 - On the other hand, this class does not implement __cmp__, so its
 behavior is unspecified. And likely to be platform dependent.

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