Opened 6 years ago

Closed 6 years ago

#1502 closed defect (fixed)

[with patch, with positive review] calculus -- bug in argument ordering for formal functions

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

Description

This is wrong:

sage: f = function('Gamma', var('z'), var('w')); f
Gamma(z, w)
sage: f(2)
Gamma(z, 2)
sage: f(2,5)
Gamma(5, 2)

It should be

sage: f = function('Gamma', var('z'), var('w')); f
Gamma(z, w)
sage: f(2)
Gamma(2, w)
sage: f(2,5)
Gamma(2, 5)

Note that this works:

sage: f(z,w) = function('Gamma'); f
(z, w) |--> Gamma(z, w)
sage: f(2)
Gamma(2, w)
sage: f(2,5)
Gamma(2, 5)

Attachments (1)

1502.patch (4.3 KB) - added by mhansen 6 years ago.

Download all attachments as: .zip

Change History (5)

Changed 6 years ago by mhansen

comment:1 Changed 6 years ago by mhansen

  • Milestone changed from sage-2.10 to sage-2.9
  • Owner changed from was to mhansen
  • Status changed from new to assigned
  • Summary changed from calculus -- bug in argument ordering for formal functions to [with patch] calculus -- bug in argument ordering for formal functions

Apply after #553 .

comment:2 Changed 6 years ago by mabshoff

  • Summary changed from [with patch] calculus -- bug in argument ordering for formal functions to [with patch, with positive review] calculus -- bug in argument ordering for formal functions

Was reviewed this positively in IRC during BD7.

Cheers,

Michael

comment:3 Changed 6 years ago by mabshoff

Merged in 2.9.rc0.

comment:4 Changed 6 years ago by mabshoff

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

Merged in 2.9.rc0.

Note: See TracTickets for help on using tickets.