merge sage's numpy.pxd with the cython numpy.pxd

[jasongrout]

[15:53] <robertwb> yep, we should merge them

CC: @robertwb @sagetrac-dagss

Component: numerical

Author: Robert Bradshaw, Dag Sverre Seljebotn

Reviewer: Robert Bradshaw, Dag Sverre Seljebotn, Minh Van Nguyen

Merged: Sage 4.1.1.alpha1

Issue created by migration from https://trac.sagemath.org/ticket/4571

[robertwb]

comment:1

See http://trac.cython.org/cython_trac/ticket/339

[robertwb]

Attachment: 4571-numpy-pxd.patch.gz

[robertwb]

comment:3

Preliminary patch, need a new Cython spkg for it to work all the way.

[sagetrac-dagss]

comment:4

Attachment: 4571-more-fixes.patch.gz

Here are some more fixes. With this, and the latest state of the

http://hg.cython.org/cython

repo, all the relevant tests seem to pass. I do get two failures (due to Cython upgrade? something else?), see test.log in

/home/dagss/sage-4.0.2-sage.math.washington.edu-x86_64-Linux

Once this works let's tag http://hg.cython.org/cython as 0.11.2.1 for #6438?

[sagetrac-dagss]

comment:5

Note (re: discussion on Cython ML) that shape in numpy.pxd is still a field, long story (well, it is performance critical), and I shouldn't change it.

[sagetrac-dagss]

comment:6

Another quick comment: With this, full Cython/NumPy functionality is available from the notebook and works well.

[robertwb]

comment:7

Those errors seem completely unrelated, I'll see if I get them too. Other than that, it looks really good.

I've used Cython + NumPy from the notebook before, as long as you weren't in sage.ext then "cimport numpy" worked as expected (though there was a point where it didn't).

[robertwb]

comment:8

Hmm... I also got

The following tests failed:

        sage -t -long devel/sage-main/sage/rings/polynomial/polynomial_template.pxi # 1 doctests failed
        sage -t -long devel/sage-main/sage/interfaces/sage0.py # 1 doctests failed

[robertwb]

comment:9

Looks good. The single doctest failure was minor, fixed in #6438.