#18194: Speedup of calculation of Macdonald H and Ht bases
-------------------------------------+-------------------------------------
       Reporter:  zabrocki           |        Owner:
           Type:  enhancement        |       Status:  needs_work
       Priority:  minor              |    Milestone:  sage-6.6
      Component:  combinatorics      |   Resolution:
       Keywords:  sf, days67, sage-  |    Merged in:
  combinat                           |    Reviewers:  Travis Scrimshaw
        Authors:  Mike Zabrocki      |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  9196d339a8adafb0c011d0f0ddb4172fbcbdd136
  public/combinat/mac_speedup-18194  |     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------
Changes (by tscrim):

 * commit:  a70a5314dbc203c66a08f1f070b739112921a146 =>
     9196d339a8adafb0c011d0f0ddb4172fbcbdd136
 * branch:  public/combinat/zabrocki/mac_speedup =>
     public/combinat/mac_speedup-18194
 * reviewer:   => Travis Scrimshaw


Comment:

 I did some cleanup of the docstrings and imports. However there are some
 doctest failures:
 {{{
 ----------------------------------------------------------------------
 sage -t ../../tests/book_schilling_zabrocki_kschur_primer.py  # 1 doctest
 failed
 sage -t ../../structure/parent.pyx  # 1 doctest failed
 ----------------------------------------------------------------------
 }}}
 The second one is slightly worrysome in that the coercion path has gotten
 longer, but it is not so bad. The first one is much more serious as I (and
 the patchbot) get a division by zero error on this test:
 {{{
 sage: Sym = SymmetricFunctions(FractionField(QQ['q']))
 sage: Mac = Sym.macdonald(t=0)
 sage: H = Mac.H()
 sage: s = Sym.schur()
 sage: for la in Partitions(3):
 ....:    print "H",la, "=", s(H(la))
 }}}
 I suspect your computational assumptions are not always being met and lead
 to this error.
 ----
 New commits:
 
||[http://git.sagemath.org/sage.git/commit/?id=71c9ae0bd14cffdcff827a8c0490dbc482f29b2e
 71c9ae0]||{{{Merge branch 'public/combinat/zabrocki/mac_speedup' of
 trac.sagemath.org:sage into public/combinat/zabrocki/mac_speedup}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=9196d339a8adafb0c011d0f0ddb4172fbcbdd136
 9196d33]||{{{Some cleanup of the docstrings and line lengths.}}}||

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