#11685: Pari finite field extension: element created by list not recognised as 
zero
----------------------------------------+-----------------------------------
   Reporter:  johanbosman               |          Owner:  AlexGhitza           
       
       Type:  defect                    |         Status:  positive_review      
       
   Priority:  major                     |      Milestone:  sage-4.7.2           
       
  Component:  algebra                   |       Keywords:  finite fields, pari  
       
Work_issues:                            |       Upstream:  N/A                  
       
   Reviewer:  Simon King, Johan Bosman  |         Author:  Johan Bosman, Jeroen 
Demeyer
     Merged:                            |   Dependencies:                       
       
----------------------------------------+-----------------------------------
Changes (by SimonKing):

  * status:  needs_review => positive_review


Old description:

> In sage-4.7.1:
> {{{
> sage: K.<a> = GF(3^11)
> sage: K([])
> 0
> sage: K([]) == 0
> False
> }}}

New description:

 In sage-4.7.1:
 {{{
 sage: K.<a> = GF(3^11)
 sage: K([])
 0
 sage: K([]) == 0
 False
 }}}

 Apply

  * [attachment:11685.patch]
  * [attachment:11685_referee.patch]

--

Comment:

 The tests pass, hence, it was not necessary to have a special case for
 zero if `value_from_pari==True`.

 I have attached a referee patch, that documents the arguments and adds a
 few examples. I hope my words are clear and concise enough, as the
 possible input can be very different.

 If you think that my referee patch needs work, then feel free to complain.

 Apply 11685.patch 11685_referee.patch

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