#14076: Update ChomP to latest upstream version (compilation failure)
--------------------------------------+-------------------------------------
       Reporter:  cnassau             |         Owner:  tbd         
           Type:  defect              |        Status:  needs_review
       Priority:  major               |     Milestone:  sage-5.11   
      Component:  packages: optional  |    Resolution:              
       Keywords:                      |   Work issues:              
Report Upstream:  N/A                 |     Reviewers:              
        Authors:  Volker Braun        |     Merged in:              
   Dependencies:  #14595              |      Stopgaps:              
--------------------------------------+-------------------------------------

Comment (by vbraun):

 Making simplicial complexes immutable isn't something that can be done
 gradually, so code that relies on them being mutable will necessarily
 break.

 Having said that, I never was very fond of the `add/remove_face`. To me,
 that implies that you have already a higher-dimensional cell to which you
 are adding a boundary cell. I would prefer `add_simplex` (for simplicial
 complexes) or maybe just `add_cell` (same interface for all cell
 complexes).

 I agree that components of the torsion generator are reversed, will have a
 look at that.

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


Reply via email to