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.

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

A wrong (easy) limit #10868

Closed
sagetrac-jvarona mannequin opened this issue Mar 3, 2011 · 13 comments
Closed

A wrong (easy) limit #10868

sagetrac-jvarona mannequin opened this issue Mar 3, 2011 · 13 comments

Comments

@sagetrac-jvarona
Copy link
Mannequin

sagetrac-jvarona mannequin commented Mar 3, 2011

If we define f(n)=2+1/factorial(n) the answer of sage for limit(1/f(n),n=oo) is 1, that is clearly wrong. However, limit(f(n),n=oo) gives the answer 2, that is correct.

I have seen this problem in (at least) sage 4.6 and sage 4.6.2, in several kind of computers.

Depends on #12094

Upstream: Fixed upstream, in a later stable release.

CC: @orlitzky

Component: calculus

Author: Michael Orlitzky

Reviewer: Aly Deines

Merged: sage-5.0.beta3

Issue created by migration from https://trac.sagemath.org/ticket/10868

@sagetrac-jvarona sagetrac-jvarona mannequin added this to the sage-5.0 milestone Mar 3, 2011
@sagetrac-jvarona sagetrac-jvarona mannequin assigned burcin Mar 3, 2011
@kcrisman
Copy link
Member

kcrisman commented Mar 3, 2011

Changed author from jvarona to none

@kcrisman
Copy link
Member

kcrisman commented Mar 3, 2011

Upstream: Reported upstream. Little or no feedback.

@kcrisman
Copy link
Member

kcrisman commented Mar 3, 2011

comment:1

This is in Maxima.

sage: maxima_console()
;;; Loading #P"/Applications/MathApps/sage/local/lib/ecl/SB-BSD-SOCKETS.fas"
;;; Loading #P"/Applications/MathApps/sage/local/lib/ecl/SOCKETS.fas"
;;; Loading #P"/Applications/MathApps/sage/local/lib/ecl/DEFSYSTEM.fas"
;;; Loading #P"/Applications/MathApps/sage/local/lib/ecl/cmp.fas"
;;; Loading #P"/Applications/MathApps/sage/local/lib/ecl/sysfun.lsp"
Maxima 5.22.1 http://maxima.sourceforge.net
using Lisp ECL 10.4.1
Distributed under the GNU Public License. See the file COPYING.
Dedicated to the memory of William Schelter.
The function bug_report() provides bug reporting information.
(%i3) limit(2+1/factorial(n),n,inf);
(%o3)                                  2
(%i4) limit(1/(2+1/factorial(n)),n,inf);
(%o4)                                  1

This is still present in the latest Maxima. See this report.

@kcrisman
Copy link
Member

comment:2

This report suggests it is fixed in the latest upstream. This would need a patch to verify, once we would update Maxima to that release.

@kcrisman
Copy link
Member

Changed upstream from Reported upstream. Little or no feedback. to Fixed upstream, but not in a stable release.

@orlitzky
Copy link
Contributor

Dependencies: #12094

@orlitzky
Copy link
Contributor

Changed upstream from Fixed upstream, but not in a stable release. to Fixed upstream, in a later stable release.

@orlitzky
Copy link
Contributor

comment:3

This is fixed in stable maxima 5.24.0:

sage: f(n)=2+1/factorial(n)
sage: limit(1/f(n),n=oo)
1/2
sage: limit(f(n),n=oo)
2
sage: maxima.version()
'5.24.0'

@orlitzky
Copy link
Contributor

Attachment: sage-trac_10868.patch.gz

Add a doctest for this result. Apply on top of #11483 and #11591 to avoid fuzz.

@orlitzky
Copy link
Contributor

Author: Michael Orlitzky

@orlitzky
Copy link
Contributor

comment:4

Maxima 5.24 has a positive review now, so here's a doctest. The file maxima_lib.py gets some new doctests in #11483 and #11591 so to avoid fuzz, apply on top of those (we don't require either of them for the result, though).

@adeines
Copy link
Mannequin

adeines mannequin commented Feb 2, 2012

Reviewer: Aly Deines

@jdemeyer
Copy link

jdemeyer commented Feb 7, 2012

Merged: sage-5.0.beta3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants