#19758: Remove deprecated "property foo:" Cython syntax
-------------------------------------+-------------------------------------
       Reporter:  jdemeyer           |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-7.0
      Component:  cython             |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Jeroen Demeyer     |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/jdemeyer/remove_deprecated__property_foo___cython_syntax|  
f6cc4629a2d8ee3dcab5a0ab6bdd4b59311d4c9b
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Description changed by jdemeyer:

Old description:

> Replace `property foo:` by `@property`.
>
> The relevant lines are:
> {{{
> src/sage/rings/rational.pyx:    property __array_interface__:
> src/sage/rings/real_mpfr.pyx:    property __array_interface__:
> src/sage/rings/integer.pyx:    property __array_interface__:
> src/sage/rings/complex_number.pyx:    property __array_interface__:
> src/sage/matrix/matrix2.pyx:    property T:
> src/sage/matrix/matrix2.pyx:    property C:
> src/sage/matrix/matrix2.pyx:    property H:
> src/sage/matrix/matrix2.pyx:    property I:
> src/sage/misc/lazy_string.pyx:    property value:
> src/sage/symbolic/expression.pyx:    property op:
> }}}
>
> I made one minor functional change: for conversion of `RR` and `CC` to
> NumPy, `double` is used for precisions <= 53 (instead of 57 earlier). The
> 57 doesn't make sense to me, as a `double` has a precision of 53 bits. I
> asked at #6506 in case somebody remembers.

New description:

 Replace `property foo:` by `@property`.

 The relevant lines are:
 {{{
 src/sage/rings/rational.pyx:    property __array_interface__:
 src/sage/rings/real_mpfr.pyx:    property __array_interface__:
 src/sage/rings/integer.pyx:    property __array_interface__:
 src/sage/rings/complex_number.pyx:    property __array_interface__:
 src/sage/matrix/matrix2.pyx:    property T:
 src/sage/matrix/matrix2.pyx:    property C:
 src/sage/matrix/matrix2.pyx:    property H:
 src/sage/matrix/matrix2.pyx:    property I:
 src/sage/misc/lazy_string.pyx:    property value:
 src/sage/symbolic/expression.pyx:    property op:
 }}}

 I made one minor functional change: for conversion of `RR` and `CC` to
 !NumPy, `double` is used for precisions <= 53 (instead of 57 earlier). The
 57 doesn't make sense to me, as a `double` has a precision of 53 bits. I
 asked at #6506 in case somebody remembers.

--

--
Ticket URL: <http://trac.sagemath.org/ticket/19758#comment:5>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to