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

wrong evaluation of infinite sum #13712

Closed
burcin opened this issue Nov 15, 2012 · 22 comments
Closed

wrong evaluation of infinite sum #13712

burcin opened this issue Nov 15, 2012 · 22 comments

Comments

@burcin
Copy link

burcin commented Nov 15, 2012

From this ask.sagemath.org question:

sage: n = var('n')
sage: sum(1/((2*n-1)^2*(2*n+1)^2*(2*n+3)^2), n, 0, oo)
3/256*pi^2 - 1/32

The correct answer is 3/256*pi^2.

We should report this to maxima:

(%i10) load(simplify_sum);
(%o10) /tmpbig/burcin/sage-5.4/local/share/maxima/5.26.0/share/contrib/solve_r\
ec/simplify_sum.mac
(%i11) simplify_sum(sum(1/((2*n-1)^2*(2*n+1)^2*(2*n+3)^2), n, 0, inf));

                                          2
                                       %pi
                                   2   ---- - 4
                                %pi     2
(%o11)                          ---- + --------
                                128      128

Depends on #13973

Component: symbolics

Keywords: maxima

Work Issues: add doctest

Author: Travis Scrimshaw

Branch/Commit: 6e7166a

Reviewer: Jeroen Demeyer, Karl-Dieter Crisman

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

@burcin burcin added this to the sage-5.10 milestone Nov 15, 2012
@burcin burcin self-assigned this Nov 15, 2012
@kcrisman
Copy link
Member

Upstream: Reported upstream. No feedback yet.

@kcrisman
Copy link
Member

comment:1

I can confirm this is true in Maxima 5.28.0 as well. This is now Maxima artifact 3587514.

@jdemeyer
Copy link

comment:2

Fixed in Maxima 5.30.0: #13973.

@jdemeyer
Copy link

Reviewer: Jeroen Demeyer

@jdemeyer
Copy link

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

@jdemeyer
Copy link

comment:5

Should add doctest.

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@pjbruin
Copy link
Contributor

pjbruin commented May 20, 2014

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

@pjbruin
Copy link
Contributor

pjbruin commented May 20, 2014

Dependencies: #13973

@pjbruin
Copy link
Contributor

pjbruin commented May 20, 2014

Work Issues: add doctest

@pjbruin
Copy link
Contributor

pjbruin commented May 20, 2014

comment:9

After #13973:

sage: n = var('n')
sage: sum(1/((2*n-1)^2*(2*n+1)^2*(2*n+3)^2), n, 0, oo)
3/256*pi^2

@tscrim
Copy link
Collaborator

tscrim commented May 29, 2014

Author: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented May 29, 2014

comment:10

Done (at least in someplace where I think it's natural...)


New commits:

6e7166aAdded doctest for ticket.

@tscrim
Copy link
Collaborator

tscrim commented May 29, 2014

Commit: 6e7166a

@tscrim
Copy link
Collaborator

tscrim commented May 29, 2014

Branch: public/ticket/inf_sum_doctest-13712

@kcrisman
Copy link
Member

comment:11

This is not a useful commit, unfortunately, because this depends on #13973 which isn't in 6.3.beta2, as far as I can tell. I just wasted a lot of time discovering this, and I don't have enough git prowess to figure out how to "merge" this into that ticket's tree.

@tscrim
Copy link
Collaborator

tscrim commented May 29, 2014

comment:12

git merge trac u/pbruin/13973-maxima_update or whatever your remote for trac is (the other likely one is origin).

@kcrisman
Copy link
Member

comment:13

git merge trac u/pbruin/13973-maxima_update or whatever your remote for trac is (the other likely one is origin).

Apparently my remote is indeed trac but this command doesn't work, nor do I have any idea what the branch is since that ticket seems to have a branch called "a130eed".

Okay, the following sequence worked:

$ git fetch trac refs/heads/u/pbruin/13973-maxima_update:refs/remotes/trac/u/pbruin/13973-maxima_update
$ git merge trac/u/pbruin/13973-maxima_update

This is so frustrating compared to the old system.

@tscrim
Copy link
Collaborator

tscrim commented May 29, 2014

comment:14

Ack sorry, I meant git pull trac u/pbruin/13973-maxima_update (which, in effect, combines those two) (also in case you are unaware, you can also delete the temp branch by git branch -D name/of/branch).

@kcrisman
Copy link
Member

comment:15

Okay, finally I got it all working and positive review!

@kcrisman
Copy link
Member

Changed reviewer from Jeroen Demeyer to Jeroen Demeyer, Karl-Dieter Crisman

@tscrim
Copy link
Collaborator

tscrim commented May 29, 2014

comment:16

Thanks. Sorry about mixing my git commands up!

@vbraun
Copy link
Member

vbraun commented May 30, 2014

Changed branch from public/ticket/inf_sum_doctest-13712 to 6e7166a

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

6 participants