Changes between Initial Version and Version 32 of Ticket #11316


Ignore:
Timestamp:
06/01/11 15:21:53 (10 years ago)
Author:
SimonKing
Comment:

I was reading the patch, and it seems ok. Meanwhile I think that __setstate__ is the best way to solve the unpickling problem for old pickles, given the fact that both name and meaning of several attributes have changed. I can confirm that old pickles can be correctly unpickled, including a block order (the old attribute blocks does not override the new method blocks()).

Having weighted degree orders is a good thing. There are some limitations inherited from Singular: The degree weights have to be positive integers.

I give a positive review, and add a reviewer patch, which I hope is fine for you.

The reviewer patch adds to the docs that the degree weights must be positive integers, and it lets an error be raised if any weight is non-positive. It is attempted to convert any non-integral weight to an integer. In particular, a weight such as 1.1 will silently be converted into 1. There are doctests for that behaviour.

Also, I add a comment in the doc of the new method blocks(), stating that back in the old days some orders had an attribute of the same name, so that we have a backward incompatible (but apparently not problematic) change.

So, if you don't oppose against my reviewer patch:

Apply trac_11316.4.patch trac11316_reviewer.patch

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #11316

    • Property Status changed from new to positive_review
    • Property Authors changed from to Kwankyu Lee
    • Property Cc burcin added
    • Property Priority changed from minor to major
    • Property Reviewers changed from to Simon King
  • Ticket #11316 – Description

    initial v32  
    22
    33New term orders as well as matrix term orders can be used in block term orders.
     4
     5Apply:
     6
     7 1. [attachment:trac_11316.4.patch]
     8
     9 2. [attachment:trac11316_reviewer.patch]