#6637: standardize the interface to TransitiveIdeal and friends
-------------------------------------+-------------------------------------
       Reporter:  nthiery            |        Owner:  mhansen
           Type:  enhancement        |       Status:  needs_info
       Priority:  major              |    Milestone:  sage-6.3
      Component:  combinatorics      |   Resolution:
       Keywords:  backtrack,         |    Merged in:
  enumerated set, transitive         |    Reviewers:  Travis Scrimshaw
  closure, days57                    |  Work issues:
        Authors:  Sébastien Labbé    |       Commit:
Report Upstream:  N/A                |  dd72bfc1227605362e9baf847a623ddd6bd297f9
         Branch:                     |     Stopgaps:
  public/ticket/6637                 |
   Dependencies:  #14052             |
-------------------------------------+-------------------------------------

Comment (by slabbe):

 Replying to [comment:51 tscrim]:
 > Looking at your commit, I don't see any benefit in using the metaclass.

 Yes exactly. I needed to play with it to understand what you meant.

 > However I'm not opposed to the renaming, so I've just pushed that.

 Well, the renaming was just an easy way for me to check that {{{sage -t
 recursively_enumerated_set.pyx}}} was ok after I realised that
 `__classcall_private__` was ignored by `cdef` class. It was not a
 suggestion, but it would not be the first renamed function:

 {{{
 sage: import_statements(sum)
 from sage.misc.functional import symbolic_sum
 }}}

 So, we go with commit ​dd72bfc instead of ​3191690 ?

--
Ticket URL: <http://trac.sagemath.org/ticket/6637#comment:52>
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/d/optout.

Reply via email to