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

incorrect summation #21801

Closed
kcrisman opened this issue Nov 2, 2016 · 13 comments
Closed

incorrect summation #21801

kcrisman opened this issue Nov 2, 2016 · 13 comments

Comments

@kcrisman
Copy link
Member

kcrisman commented Nov 2, 2016

Reported here:

sage: sum(1/((n+1)*(2*n-1)), n, 0, 1000).n()
-0.205068171626375
sage: sum(1/((n+1)*(2*n-1)), n, 0, 10000).n()
-0.204618542543703
sage: sum(1/((n+1)*(2*n-1)), n, 0, 100000).n()  # seems to be converging
-0.204573546255870
sage: sum(1/((n+1)*(2*n-1)), n, 0, oo).n()      # but not to this number
-1.09345743518226

sage: sum(1/((n+1)*(2*n-1)), n, 0, oo)
2/3*log(2) - 14/9
sage: sum(1/((n+1)*(2*n-1)), n, 0, oo).n() - sum(1/((n+1)*(2*n-1)), n, 0, 10000).n()
-0.888838892638556

Upstream: Completely fixed; Fix reported upstream

Component: calculus

Author: Thierry Monteil

Branch/Commit: 4b525bb

Reviewer: Travis Scrimshaw

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

@kcrisman kcrisman added this to the sage-7.5 milestone Nov 2, 2016
@jhpalmieri
Copy link
Member

comment:1

Tracked upstream at https://sourceforge.net/p/maxima/bugs/3236/.

@jhpalmieri
Copy link
Member

Changed upstream from Not yet reported upstream; Will do shortly. to Reported upstream. No feedback yet.

@kcrisman
Copy link
Member Author

Changed upstream from Reported upstream. No feedback yet. to Fixed upstream, but not in a stable release.

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented May 24, 2017

Branch: u/tmonteil/incorrect_summation

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented May 24, 2017

comment:4

The fix seems to have entered Sage now. So i am just ading a doctest to prevent the bug to reappear.

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented May 24, 2017

Commit: 7a36941

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented May 24, 2017

Author: Thierry Monteil

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented May 24, 2017

Changed upstream from Fixed upstream, but not in a stable release. to Completely fixed; Fix reported upstream

@sagetrac-tmonteil sagetrac-tmonteil mannequin modified the milestones: sage-7.5, sage-8.0 May 24, 2017
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 24, 2017

Branch pushed to git repo; I updated commit sha1. New commits:

4b525bb#21801 : a doctest to prevent the bug to reappear.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 24, 2017

Changed commit from 7a36941 to 4b525bb

@tscrim
Copy link
Collaborator

tscrim commented May 24, 2017

comment:6

LGTM.

@tscrim
Copy link
Collaborator

tscrim commented May 24, 2017

Reviewer: Travis Scrimshaw

@vbraun
Copy link
Member

vbraun commented May 26, 2017

Changed branch from u/tmonteil/incorrect_summation to 4b525bb

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