Opened 3 years ago
Closed 3 years ago
#16176 closed defect (fixed)
doctest a fixed sum simplification
Reported by:  kcrisman  Owned by:  

Priority:  major  Milestone:  sage6.5 
Component:  calculus  Keywords:  
Cc:  Merged in:  
Authors:  Ralf Stephan  Reviewers:  KarlDieter Crisman 
Report Upstream:  Fixed upstream, in a later stable release.  Work issues:  
Branch:  851f961 (Commits)  Commit:  851f9617d0a3ecd9f27bdf1f20c8a9fd455f0ce2 
Dependencies:  #9635  Stopgaps: 
Description (last modified by )
This is now fixed and needs a doctest:
See this sagesupport thread.
n = var('n') sum(log(11/n^2),n,2,oo)
The error is in Maxima's simplify_sum
.
Change History (15)
comment:1 Changed 3 years ago by
 Report Upstream changed from Not yet reported upstream; Will do shortly. to Reported upstream. Developers acknowledge bug.
comment:2 Changed 3 years ago by
 Report Upstream changed from Reported upstream. Developers acknowledge bug. to Fixed upstream, but not in a stable release.
Gotta love distributed work. Already committed in upstream. (I haven't checked it works but anyway there is a commit in the right file.)
So this (eventually) needs a new Maxima package and a doctest.
comment:3 Changed 3 years ago by
 Milestone changed from sage6.2 to sage6.3
comment:4 Changed 3 years ago by
 Milestone changed from sage6.3 to sage6.4
comment:5 Changed 3 years ago by
 Description modified (diff)
 Summary changed from Incorrect sum simplification leads to error to doctest a fixed sum simplification
comment:6 Changed 3 years ago by
 Branch set to u/rws/doctest_a_fixed_sum_simplification
comment:7 Changed 3 years ago by
 Commit set to d27f8497dcd19d70ec08155888e6fec9c74b839a
 Milestone changed from sage6.4 to sage6.5
 Status changed from new to needs_review
comment:8 Changed 3 years ago by
 Commit changed from d27f8497dcd19d70ec08155888e6fec9c74b839a to d0d7abe91d6a30f77f2a6dee471811d1747cafce
Branch pushed to git repo; I updated commit sha1. New commits:
d0d7abe  16176: doctest

comment:9 Changed 3 years ago by
+ Check that :trac:`16176` is fixed:: + + n = var('n') + sum(log(11/n^2),n,2,oo)
I think you might have done this at a weird time of day :) but this is correct, anyway, so I'll be sure to give this positive review once the branch looks right. Thanks!
comment:10 Changed 3 years ago by
 Report Upstream changed from Fixed upstream, but not in a stable release. to Fixed upstream, in a later stable release.
 Reviewers set to KarlDieter Crisman
 Status changed from needs_review to needs_work
 Work issues set to use sage prompt and give answer
comment:11 Changed 3 years ago by
Also, might as well rebase to #9635 which presumably is all set but would conflict... (?)
comment:12 Changed 3 years ago by
 Branch changed from u/rws/doctest_a_fixed_sum_simplification to u/rws/16176
comment:13 Changed 3 years ago by
 Commit changed from d0d7abe91d6a30f77f2a6dee471811d1747cafce to 851f9617d0a3ecd9f27bdf1f20c8a9fd455f0ce2
 Dependencies set to #9635
 Status changed from needs_work to positive_review
comment:15 Changed 3 years ago by
 Branch changed from u/rws/16176 to 851f9617d0a3ecd9f27bdf1f20c8a9fd455f0ce2
 Resolution set to fixed
 Status changed from positive_review to closed
Upstream report at Maxima bug 2722.