#8386: enhanced version of iet
--------------------------------------+-------------------------------------
       Reporter:  vdelecroix          |         Owner:  vdelecroix             
           Type:  defect              |        Status:  needs_review           
       Priority:  major               |     Milestone:  sage-5.10              
      Component:  combinatorics       |    Resolution:                         
       Keywords:  iet, combinatorics  |   Work issues:  save the new datatype ?
Report Upstream:  N/A                 |     Reviewers:  Frédéric Chapoton      
        Authors:  Vincent Delecroix   |     Merged in:                         
   Dependencies:  #12643, #13677      |      Stopgaps:                         
--------------------------------------+-------------------------------------
Changes (by chapoton):

  * work_issues:  startup problem => save the new datatype ?


Old description:

> 1) The library for iet is moved from sage.combinat.iet to
> sage.dynamics.interval_exchanges which seems more natural.
>
> 2) The old version for iet uses two kinds of datatype for reduced and
> labeled versions. The new implementation makes them coherent.
>
> 3) As a by product of 2), the following behavior is corrected
> {{{
> sage: p = iet.Permutation('a b','b a')
> sage: q = iet.Permutation('b a','a b')
> sage: p == q
> True
> }}}
>
> Apply only [attachment:trac_8386-enhanced_iet.v3.patch]

New description:

 1) The library for iet is moved from sage.combinat.iet to
 sage.dynamics.interval_exchanges which seems more natural.

 2) The old version for iet uses two kinds of datatype for reduced and
 labeled versions. The new implementation makes them coherent.

 3) As a by product of 2), the following behavior is corrected
 {{{
 sage: p = iet.Permutation('a b','b a')
 sage: q = iet.Permutation('b a','a b')
 sage: p == q
 True
 }}}

 Apply only [attachment:trac_8386_just_moving-fc.patch]

--

Comment:

 Let me try to do something for this ticket.

 Here is a new patch, that '''only''' moves the files from "combinat/iet"
 to a new folder "dynamics".

 This has been done by starting with the code in 5.10.beta3. So there is
 something lost, which is the refactoring of datatype.

 But it pass all tests.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8386#comment:21>
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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to