#13726: The semimonomial group
-------------------------------------------------+-------------------------
       Reporter:  tfeulner                       |        Owner:  joyner
           Type:  enhancement                    |       Status:
       Priority:  major                          |  needs_review
      Component:  group theory                   |    Milestone:  sage-5.13
       Keywords:  (semi-)monomial group,         |   Resolution:
  semilinear action, isometry group              |    Merged in:
        Authors:  Thomas Feulner                 |    Reviewers:
Report Upstream:  N/A                            |  Work issues:
         Branch:                                 |       Commit:
   Dependencies:                                 |     Stopgaps:
-------------------------------------------------+-------------------------

Comment (by vbraun):

 Small nitpicks:
   * In general it is better to use lazy import to improve the Sage startup
 time:
     {{{
 lazy_import('sage.groups.semimonomial_group.semimonomial_group',
             'SemimonomialGroup')
     }}}
     in `sage/groups/all.py`. Then the actual import is deferred until you
 use it.
   * incidentally, the change to `sage/groups/all.py` doesn't apply cleanly
 on sage-5.13.beta0 and needs to be rediffed anyways.
   * Add also `semimonomial_group_element` to the developer manual.

 The code looks great, thank you for your hard work!

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