#10136: `log(x).operator() == log` should return `True`
-------------------------------------+-------------------------------
       Reporter:  burcin             |        Owner:  burcin
           Type:  defect             |       Status:  positive_review
       Priority:  major              |    Milestone:  sage-6.2
      Component:  symbolics          |   Resolution:
       Keywords:  log, ln, function  |    Merged in:
        Authors:  Burcin Erocal      |    Reviewers:  Ralf Stephan
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------
Changes (by rws):

 * keywords:   => log, ln, function
 * reviewer:   => Ralf Stephan
 * status:  needs_review => positive_review


Comment:

 @ppurka: `log`, like `exp`, never had an "INPUT" description; the `hold`
 parameter is introduced informally in both cases; please open a separate
 ticket.

 @kcrisman: `log?` is fine, and identical to `ln?`

 Reviewer's patch addresses this patchbot failure:
 {{{
 File "src/sage/ext/fast_callable.pyx", line 1524, in
 sage.ext.fast_callable._expression_binop_helper.get_builtin_functions
 Failed example:
     sorted(list(builtins.values()))
 Expected:
     ['abs', 'abs', 'acos', 'acosh', 'add', 'asin', 'asinh', 'atan',
 'atanh', 'ceil', 'cos', 'cosh', 'cot', 'csc', 'div', 'exp', 'floor',
 'floordiv', 'inv', 'log', 'log', 'mul', 'neg', 'pow', 'sec', 'sin',
 'sinh', 'sqrt', 'sub', 'tan', 'tanh']
 Got:
     ['abs', 'abs', 'acos', 'acosh', 'add', 'asin', 'asinh', 'atan',
 'atanh', 'ceil', 'cos', 'cosh', 'cot', 'csc', 'div', 'exp', 'floor',
 'floordiv', 'inv', 'log', 'mul', 'neg', 'pow', 'sec', 'sin', 'sinh',
 'sqrt', 'sub', 'tan', 'tanh']
 }}}
 I hazard that this small change won't need another review, so I'll set
 positive.

--
Ticket URL: <http://trac.sagemath.org/ticket/10136#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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to