Opened 20 months ago
Closed 10 months ago
#16176 closed defect (fixed)
doctest a fixed sum simplification
Reported by: | kcrisman | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-6.5 |
Component: | calculus | Keywords: | |
Cc: | Merged in: | ||
Authors: | Ralf Stephan | Reviewers: | Karl-Dieter Crisman |
Report Upstream: | Fixed upstream, in a later stable release. | Work issues: | |
Branch: | 851f961 (Commits) | Commit: | 851f9617d0a3ecd9f27bdf1f20c8a9fd455f0ce2 |
Dependencies: | #9635 | Stopgaps: |
Description (last modified by rws)
This is now fixed and needs a doctest:
See this sage-support thread.
n = var('n') sum(log(1-1/n^2),n,2,oo)
The error is in Maxima's simplify_sum.
Change History (15)
comment:1 Changed 20 months ago by kcrisman
- Report Upstream changed from Not yet reported upstream; Will do shortly. to Reported upstream. Developers acknowledge bug.
comment:2 Changed 20 months ago by kcrisman
- 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 19 months ago by vbraun_spam
- Milestone changed from sage-6.2 to sage-6.3
comment:4 Changed 16 months ago by vbraun_spam
- Milestone changed from sage-6.3 to sage-6.4
comment:5 Changed 10 months ago by rws
- Description modified (diff)
- Summary changed from Incorrect sum simplification leads to error to doctest a fixed sum simplification
comment:6 Changed 10 months ago by rws
- Branch set to u/rws/doctest_a_fixed_sum_simplification
comment:7 Changed 10 months ago by rws
- Commit set to d27f8497dcd19d70ec08155888e6fec9c74b839a
- Milestone changed from sage-6.4 to sage-6.5
- Status changed from new to needs_review
comment:8 Changed 10 months ago by git
- Commit changed from d27f8497dcd19d70ec08155888e6fec9c74b839a to d0d7abe91d6a30f77f2a6dee471811d1747cafce
Branch pushed to git repo; I updated commit sha1. New commits:
d0d7abe | 16176: doctest |
comment:9 Changed 10 months ago by kcrisman
+ Check that :trac:`16176` is fixed:: + + n = var('n') + sum(log(1-1/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 10 months ago by kcrisman
- Report Upstream changed from Fixed upstream, but not in a stable release. to Fixed upstream, in a later stable release.
- Reviewers set to Karl-Dieter Crisman
- Status changed from needs_review to needs_work
- Work issues set to use sage prompt and give answer
comment:11 Changed 10 months ago by kcrisman
Also, might as well rebase to #9635 which presumably is all set but would conflict... (?)
comment:12 Changed 10 months ago by rws
- Branch changed from u/rws/doctest_a_fixed_sum_simplification to u/rws/16176
comment:13 Changed 10 months ago by rws
- Commit changed from d0d7abe91d6a30f77f2a6dee471811d1747cafce to 851f9617d0a3ecd9f27bdf1f20c8a9fd455f0ce2
- Dependencies set to #9635
- Status changed from needs_work to positive_review
comment:14 Changed 10 months ago by kcrisman
- Work issues use sage prompt and give answer deleted
Thanks!
comment:15 Changed 10 months ago by vbraun
- 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.