#16219: Implement a catalog for algebras
-------------------------------------+-------------------------------------
       Reporter:  tscrim             |        Owner:  tscrim
           Type:  enhancement        |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.4
      Component:  interfaces         |   Resolution:
       Keywords:  algebras catalog   |    Merged in:
        Authors:  Travis Scrimshaw   |    Reviewers:  Nicolas M. ThiƩry
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  public/algebras/catalog-16219      |  61d6bd1dd3be8ef99ac8c835fd94890e68bef235
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by tscrim):

 Replying to [comment:5 nthiery]:
 > I have just been through the diff, and overall it looks good.

 Thanks for taking a look at this.

 > It seems a bit redundant to both import the algebras and link to their
 doc. We should find a better way to handle this automatically at some
 point. But that's for a later ticket.

 Concur.

 > The documentation improvements elsewhere in the algebra module is nice.
 Yet's it's orthogonal to the main purpose of the ticket. Next time, please
 make it a separate ticket.

 When I added the `quaternion_algebra.py` to the doc, it causes docbuild
 failures. The cleanup was just as I was going through reading the file in
 order for it to build.

 > Why not lazy importing everything?

 I lazy imported things that were lazily imported into the global
 namespace, so it wouldn't trigger an import when doing tab completion on
 `algebras.` (as what happens for groups).

 There's a more trickier question of what algebras could we remove from the
 global namespace (it's not nearly as clear as it was for crystals to me),
 but I think that can be done on a followup.

 > Up to this last point, I believe it's good to go (assuming all tests
 pass).

 Was this suppose to be set as needs_info? I don't quite understand why
 this was set to needs_work.

 > Ah, also: at some point, we will want to point to all the Hopf algebras
 from combinat. But it's probably easier to do this once #16256 is done.

 Yep, and it should probably be a subcatalog called `combinatorial_hopf`
 (or maybe just `.hopf` and include all Hopf algebras). Anyways, stuff for
 later and I'll take a look at #16256 soon.

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