#11431: Conversion from Singular to Sage
-------------------------------+--------------------------------------------
   Reporter:  SimonKing        |          Owner:  was       
       Type:  defect           |         Status:  needs_work
   Priority:  major            |      Milestone:  sage-4.7.1
  Component:  interfaces       |       Keywords:            
Work_issues:                   |       Upstream:  N/A       
   Reviewer:  Martin Albrecht  |         Author:  Simon King
     Merged:                   |   Dependencies:  #11316    
-------------------------------+--------------------------------------------
Changes (by malb):

  * status:  needs_review => needs_work
  * reviewer:  => Martin Albrecht


Comment:

 * can we rename {{{sage_basering}}} to {{{sage_global_ring}}}. In Sage
 {{{base_ring}}} has a special meaning which is not meant in your patch. I
 think global ring captures it.
  * don't real fields also have a precision in Singular?
  * Previously, we enforced and assumed {{{not short}}}, but it's good
 you're taking it into account.
  * Can you also fix "If it is provided, then you *must* take care OFF
 it."?
  * shouldn't {{{TermOrder_from_Singular}}} be lower-case since it's a
 function? Or were you aiming for upper-case because it's a constructor?
  * In "Invalid termorder in Singular" shouldn't it be term order, i.e.
 with a space?

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