Opened 13 months ago

Closed 3 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 13 months ago by kcrisman

  • Report Upstream changed from Not yet reported upstream; Will do shortly. to Reported upstream. Developers acknowledge bug.

Upstream report at Maxima bug 2722.

comment:2 Changed 12 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 12 months ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:4 Changed 9 months ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:5 Changed 3 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 3 months ago by rws

  • Branch set to u/rws/doctest_a_fixed_sum_simplification

comment:7 Changed 3 months ago by rws

  • Authors set to Ralf Stephan
  • Commit set to d27f8497dcd19d70ec08155888e6fec9c74b839a
  • Milestone changed from sage-6.4 to sage-6.5
  • Status changed from new to needs_review

comment:8 Changed 3 months ago by git

  • Commit changed from d27f8497dcd19d70ec08155888e6fec9c74b839a to d0d7abe91d6a30f77f2a6dee471811d1747cafce

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

d0d7abe16176: doctest

comment:9 Changed 3 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 3 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 3 months ago by kcrisman

Also, might as well rebase to #9635 which presumably is all set but would conflict... (?)

comment:12 Changed 3 months ago by rws

  • Branch changed from u/rws/doctest_a_fixed_sum_simplification to u/rws/16176

comment:13 Changed 3 months ago by rws

  • Commit changed from d0d7abe91d6a30f77f2a6dee471811d1747cafce to 851f9617d0a3ecd9f27bdf1f20c8a9fd455f0ce2
  • Dependencies set to #9635
  • Status changed from needs_work to positive_review

New commits:

5479dd19635: doctest
b8569feAdd full doctest
851f96116176: doctest

comment:14 Changed 3 months ago by kcrisman

  • Work issues use sage prompt and give answer deleted

Thanks!

comment:15 Changed 3 months ago by vbraun

  • Branch changed from u/rws/16176 to 851f9617d0a3ecd9f27bdf1f20c8a9fd455f0ce2
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.