Changes between Version 22 and Version 80 of Ticket #10682


Ignore:
Timestamp:
03/06/12 18:58:26 (10 years ago)
Author:
jpflori
Comment:

I usually do not do it, especially so many lines at a time, but my reviewer patch was all about removing some white spaces reintroduced by Dima, and split some overlongish lines (that's another tricky choice, I can leave with long lines, but it's much more readable in a terminal without them and according to our coding guidelines we should stick to 79 chars at max...).

So I chose to suppress them this time.

IIRC there was a discussion some time ago on sae-devel about make a one time huge patch for suppressing spaces but some people, especially among sage-combinat, raised the problem that it would break patch queues and that we should rather ensure that no such new lines where added and fix for previous spaces made locally.

I hope my choice won't break anything.

Anyway, good that this ticket is finally positively reviewed.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #10682

    • Property Status changed from needs_review to positive_review
    • Property Work issues changed from several doctests need to be patched due to changes in output format/term order to
    • Property Keywords maxima 5.26.0 binomial sum added
    • Property Authors changed from to Dima Pasechnik
    • Property Reviewers changed from to Jean-Pierre Flori, Nils Bruin
  • Ticket #10682 – Description

    v22 v80  
    11I test
     2
    23{{{
    34sage: var ('n,k')
     
    67sage: sum (binomial(n,k)*k^2, k, 2, n)  # is false : I get 0
    78}}}
     9This works correctly on Maxima 5.26 - we need to upgrade! The new spkg is at
    810
    9 This works correctly on Maxima 5.26 - we need to upgrade!
    10 The new spkg is [http://boxen.math.washington.edu/home/dima/packages/maxima-5.26.0.spkg here].
     11http://perso.telecom-paristech.fr/~flori/sage/maxima-5.26.0.p0.spkg
    1112
    12 Install the spkg and apply the patch.
     13Install the spkg and apply:
     14
     15 *   [attachment:trac10682_1.patch]
     16 *  [attachment:trac_10682-reviewer.patch]