Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

calculus -- bug in argument ordering for formal functions #1502

Closed
williamstein opened this issue Dec 14, 2007 · 5 comments
Closed

calculus -- bug in argument ordering for formal functions #1502

williamstein opened this issue Dec 14, 2007 · 5 comments
Assignees
@mwhansen
Labels
c: calculus p: major / 3 t: bug
Milestone
sage-2.9

Comments

@williamstein
Copy link
Contributor

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)

Component: calculus

Issue created by migration from https://trac.sagemath.org/ticket/1502
@williamstein williamstein added this to the sage-2.10 milestone Dec 14, 2007
@williamstein williamstein self-assigned this Dec 14, 2007
@mwhansen
Copy link
Contributor

Attachment: 1502.patch.gz

@mwhansen
Copy link
Contributor

comment:1

Apply after #553 .

@mwhansen mwhansen modified the milestones: sage-2.10, sage-2.9 Dec 14, 2007
@mwhansen mwhansen assigned mwhansen and unassigned williamstein Dec 14, 2007
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Dec 15, 2007

comment:2

Was reviewed this positively in IRC during BD7.

Cheers,

Michael

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Dec 15, 2007

comment:3

Merged in 2.9.rc0.

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Dec 15, 2007

comment:4

Merged in 2.9.rc0.

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Dec 15, 2007
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mwhansen mwhansen

Labels
c: calculus p: major / 3 t: bug
Projects
None yet
Milestone
sage-2.9
Development

No branches or pull requests
2 participants
@mwhansen @williamstein