#12542: Fix issues with converting symbolic expressions to polynomials
------------------------------------------+---------------------------------
   Reporter:  mhansen                     |          Owner:  burcin     
       Type:  defect                      |         Status:  needs_work 
   Priority:  minor                       |      Milestone:  sage-5.0   
  Component:  symbolics                   |       Keywords:             
Work_issues:  Document method parameters  |       Upstream:  N/A        
   Reviewer:  Michael Orlitzky            |         Author:  Mike Hansen
     Merged:                              |   Dependencies:             
------------------------------------------+---------------------------------
Changes (by mjo):

 * cc: mjo (added)
  * reviewer:  => Michael Orlitzky
  * status:  needs_review => needs_work
  * work_issues:  => Document method parameters


Comment:

 expression.pyx:

 Can you add documentation for the new `ring` parameter (and maybe the
 existing `base_ring`, too)? What's the difference between `ring` and
 `base_ring`?

 constants.py:

 Same thing, we should document the parameter. It looks like it's just some
 ring, but not necessarily the global `SR`? In that case, we should use a
 different name to avoid confusion.

 Can we include a doctest for your second example in the description?

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