#14516: Refactoring of crystals for speedup
----------------------------------------------+-----------------------------
       Reporter:  tscrim                      |         Owner:  sage-combinat  
           Type:  enhancement                 |        Status:  positive_review
       Priority:  major                       |     Milestone:  sage-5.11      
      Component:  combinatorics               |    Resolution:                 
       Keywords:  crystals speedup, days49    |   Work issues:                 
Report Upstream:  N/A                         |     Reviewers:  Anne Schilling 
        Authors:  Travis Scrimshaw            |     Merged in:                 
   Dependencies:  #2023 #14402 #14413 #14143  |      Stopgaps:                 
----------------------------------------------+-----------------------------

Old description:

> In order to speed up many of the crystals computations, I'm proposing the
> following:
>
> - Cythonize `CrystalOfLetters`
> - Make `index_set()` a cached method. Subsequently this requires it to be
> returned as a tuple instead of a list
> - Make different letters classes, one for integers (classical types), one
> for tuples (exceptional/spin)
> - Store by caching result of `_element_constructor_()` the elements of
> `CrystalOfLetters`.
>
> See #14686 for a followup.

New description:

 In order to speed up many of the crystals computations, I'm proposing the
 following:

 - Cythonize `CrystalOfLetters`
 - Make `index_set()` a cached method. Subsequently this requires it to be
 returned as a tuple instead of a list
 - Make different letters classes, one for integers (classical types), one
 for tuples (exceptional/spin)
 - Store by caching result of `_element_constructor_()` the elements of
 `CrystalOfLetters`.

 See #14686 for a followup.

 Apply:

 * [attachment:trac_14516-crystals_speedup-ts.2.patch]

--

Comment (by tscrim):

 Forgot to qrefresh.

 Apply: trac_14516-crystals_speedup-ts.2.patch

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