#8909: Coercion from Gap to cyclotomic fields; usage of GAP to improve 
computation
of invariant rings
-------------------------------------------+--------------------------------
   Reporter:  SimonKing                    |       Owner:  was                  
                  
       Type:  enhancement                  |      Status:  needs_review         
                  
   Priority:  major                        |   Milestone:  sage-4.5             
                  
  Component:  interfaces                   |    Keywords:  gap, cyclotomic 
fields, invariant rings
     Author:  Simon King                   |    Upstream:  N/A                  
                  
   Reviewer:  David Loeffler, Mike Hansen  |      Merged:                       
                  
Work_issues:                               |  
-------------------------------------------+--------------------------------
Changes (by SimonKing):

  * status:  needs_work => needs_review


Comment:

 Replying to [comment:5 SimonKing]:
 > Replying to [comment:4 mhansen]:
 > > There's a bare except clause at 6767 which should be fixed.
 >
 > By 'bare', you mean that it is not specified what error is raised? So,
 {{{except TypeError:}}} instead of {{{except:}}}? Well this would be easy
 enough to fix.
 >

 Under the assumption that I understood you correctly, I provided a second
 patch that specifies that we only want to catch a {{{TypeError}}}, and
 return to needs_review.

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