#11685: Pari finite field extension: element created by list not recognised as 
zero
----------------------------------------+-----------------------------------
   Reporter:  johanbosman               |          Owner:  AlexGhitza           
       
       Type:  defect                    |         Status:  needs_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:                       
       
----------------------------------------+-----------------------------------

Comment(by jdemeyer):

 Replying to [comment:23 SimonKing]:
 > I wouldn't say that your patch is simpler, because you first convert
 into a list of prime field elements and then convert into pari; a double
 conversion, so it seems to me, tends to be fragile.

 I would rather argue the opposite: I break the conversion down to two
 simpler independent steps.

 It is probably true that my code is slower, but it all depends on what you
 want.  Pick two:
  - Fast code
  - Simple code
  - Code with many features

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