Ticket #644 (closed defect: fixed)

Opened 6 years ago

Last modified 5 years ago

[with patch] (sin + cos)(1) does not work

Reported by: robertwb Owned by: mhansen
Priority: major Milestone: sage-2.8.15
Component: calculus Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description (last modified by was) (diff)

Both sin and cos are functions of one (undetermined) variable, and can be called, but when one performs arithmetic on them this changes.

Maybe there should be a CallableSymbolicExpressionRing? with an unnamed variable that coerces into any CallableSymbolicExpressionRing? with a specified variable name? 

sage: f = sin
sage: g = cos
sage: f(1)
sin(1)
sage: g(1)
cos(1)
sage: h = f+g
sage: h(1)
sin + cos # should be sin(1)+cos(1)
sage: f = 3*sin
sage: f(1)
3*sin # should be 3*sin(1)

Attachments

644.patch Download (4.7 KB) - added by mhansen 5 years ago.

Change History

comment:1 Changed 6 years ago by robertwb

Better formatting: 

sage: f = sin
sage: g = cos
sage: f(1)
sin(1)
sage: g(1)
cos(1)
sage: h = f+g
sage: h(1)
sin + cos         # should be sin(1)+cos(1), or at least throw an error
sage: f = 3*sin
sage: f(1)
3*sin             # should be 3*sin(1)

comment:2 Changed 6 years ago by was

  • Description modified (diff)
  • Milestone set to sage-2.9

comment:3 Changed 6 years ago by robertwb

Also should have 

sage: f(x) = x^2
sage: f + sin
x |--> sin(x) + x^2

comment:5 Changed 5 years ago by mhansen

  • Owner changed from was to mhansen
  • Status changed from new to assigned
  • Summary changed from (sin + cos)(1) does not work to [with patch] (sin + cos)(1) does not work
  • Milestone changed from sage-2.9.1 to sage-2.8.15

This patch should be applied after #644.

Changed 5 years ago by mhansen

comment:6 Changed 5 years ago by mabshoff

  • Status changed from assigned to closed
  • Resolution set to fixed

Merged in 2.8.15.alpha1.

Note: See TracTickets for help on using tickets.