#9933: BooleanPolynomialRing not recognizing leading term of elements
-----------------------------------+----------------------------------------
   Reporter:  mariah               |       Owner:  malb        
       Type:  defect               |      Status:  needs_review
   Priority:  minor                |   Milestone:  sage-4.6    
  Component:  commutative algebra  |    Keywords:              
     Author:  Martin Albrecht      |    Upstream:  N/A         
   Reviewer:                       |      Merged:              
Work_issues:                       |  
-----------------------------------+----------------------------------------
Changes (by newvalueoldvalue):

  * status:  new => needs_review
  * author:  Mariah Lenox => Martin Albrecht


Comment:

 The attached patch fixes the reported issue. In the process I also fixed a
 very annoying performance issue

 {{{
 #!python
 # Old
 sage: B = BooleanPolynomialRing(200,'x')
 sage: %timeit B("x0")
 25 loops, best of 3: 17.1 ms per loop

 # New
 sage: B = BooleanPolynomialRing(200,'x')
 sage: %timeit B("x0")
 625 loops, best of 3: 11.6 µs per loop

 # "optimal"
 sage: gd = B.gens_dict()
 sage: %timeit gd['x0']
 625 loops, best of 3: 94.2 ns per loop
 }}}

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9933#comment:1>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to