#12650: Perform safe simplifications in Expression.simplify()
------------------------------------+---------------------------------------
       Reporter:  mjo               |         Owner:  mjo         
           Type:  enhancement       |        Status:  needs_review
       Priority:  major             |     Milestone:  sage-5.0    
      Component:  symbolics         |    Resolution:              
       Keywords:                    |   Work issues:              
Report Upstream:  N/A               |     Reviewers:  David Roe   
        Authors:  Michael Orlitzky  |     Merged in:              
   Dependencies:                    |      Stopgaps:              
------------------------------------+---------------------------------------

Comment (by kcrisman):

 I don't think this is the right fix.
  * Breaks behavior for no reason
  * Breaks with our recent trajectory of ''adding'' access to Maxima stuff
 and granularity
  * Makes users have to do much more than needed for the super-basic
 simplifications that Maxima did
 Now, maybe we should do more in `simplify`.  But it seems sort of
 redundant to ''remove'' `radcan` from `simplify_full` and then ''add''
 everything else here anyway.

 I think that it might make sense to look over the Maxima documentation and
 ''code'' very carefully and determine if something like the factorial
 simplification truly is the same in all cases, like it would be with
 `expand`.  Otherwise it seems unwise to make this change.

 I'm especially mystified as to what the difference between `simplify` and
 `simplify_full` should be under this regime.

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