#7950: factoring broken in 0 variable polynomial ring
-----------------------------------+----------------------------------------
   Reporter:  burcin               |       Owner:  malb         
       Type:  defect               |      Status:  needs_work   
   Priority:  major                |   Milestone:  sage-4.3.1   
  Component:  commutative algebra  |    Keywords:               
Work_issues:                       |      Author:  Burcin Erocal
   Upstream:  N/A                  |    Reviewer:               
     Merged:                       |  
-----------------------------------+----------------------------------------
Changes (by was):

  * status:  needs_review => needs_work


Comment:

 1. Factoring of 0 should raise an error like it does over ZZ, but doesn't
 right now:
 {{{
 sage: P = PolynomialRing(ZZ,0,'')
 sage: P(10).factor()
 10
 sage: P(0).factor()
 0
 sage: factor(0)
 ---------------------------------------------------------------------------
 ArithmeticError                           Traceback (most recent call
 last)
 }}}

 2. The element 10 in the polynomial ring "ZZ[]" in 0-variables is actually
 *not* a unit.  So it is wrong that it is put in the "unit" slot of the
 factorization.   Notice how factoring 10 works:
 {{{
 sage: R.<x> = ZZ[]
 sage: (10*x).factor()
 2 * 5 * x
 sage: list((10*x).factor())
 [(2, 1), (5, 1), (x, 1)]
 }}}
 In particular, the 10 is *not* treated incorrectly as a unit.

 So I think this patch needs work.

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