#4571: [with patch, needs work] merge sage's numpy.pxd with the cython numpy.pxd
---------------------------------------------------+------------------------
Reporter: jason | Owner: mabshoff
Type: defect | Status: new
Priority: major | Milestone: sage-4.1.1
Component: numerical | Keywords:
Reviewer: Robert Bradshaw, Dag Sverre Seljebotn | Author: Robert
Bradshaw, Dag Sverre Seljebotn
Merged: |
---------------------------------------------------+------------------------
Comment(by mvngu):
With both patches applied in the following order:
1. {{{4571-numpy-pxd.patch}}}
1. {{{4571-more-fixes.patch}}}
I see the following build failure:
{{{
[mv...@sage sage-4.1.1.alpha0]$ ./sage -br main
----------------------------------------------------------
sage: Building and installing modified Sage library files.
Installing c_lib
scons: `install' is up to date.
Updating Cython code....
Building modified file sage/finance/time_series.pyx.
Building modified file sage/matrix/change_ring.pyx.
Building modified file sage/matrix/matrix_complex_double_dense.pyx.
Building modified file sage/matrix/matrix_double_dense.pyx.
Building modified file sage/matrix/matrix_real_double_dense.pyx.
Building modified file sage/modules/vector_complex_double_dense.pyx.
Building modified file sage/modules/vector_double_dense.pyx.
Building modified file sage/modules/vector_real_double_dense.pyx.
Building sage/rings/polynomial/real_roots.pyx because it depends on
sage/modules/vector_double_dense.pxd.
Building sage/stats/hmm/chmm.pyx because it depends on
sage/matrix/matrix_double_dense.pxd.
Building sage/stats/hmm/hmm.pyx because it depends on
sage/matrix/matrix_double_dense.pxd.
python `which cython` --embed-positions --incref-local-binop
-I/scratch/mvngu/release/sage-4.1.1.alpha0/devel/sage-main -o
sage/finance/time_series.c sage/finance/time_series.pyx
warning: /scratch/mvngu/release/sage-4.1.1.alpha0/devel/sage-
main/sage/finance/time_series.pyx:1722:24: cdef variable 'j' declared
after it is used
Error converting Pyrex file to C:
------------------------------------------------------------
...
[20.0000, -3.0000, 4.5000, -2.0000]
"""
cnumpy.import_array() #This must be called before using the numpy
C/api or you will get segfault
cdef cnumpy.npy_intp dims[1]
dims[0] = self._length
cdef cnumpy.ndarray n = cnumpy.PyArray_SimpleNewFromData(1, dims,
cnumpy.NPY_DOUBLE, self._values)
^
------------------------------------------------------------
/scratch/mvngu/release/sage-4.1.1.alpha0/devel/sage-
main/sage/finance/time_series.pyx:1862:72: Cannot convert 'numpy.npy_intp
[1]' to Python object
Error converting Pyrex file to C:
------------------------------------------------------------
...
[20.0000, -3.0000, 4.5000, -2.0000]
"""
cnumpy.import_array() #This must be called before using the numpy
C/api or you will get segfault
cdef cnumpy.npy_intp dims[1]
dims[0] = self._length
cdef cnumpy.ndarray n = cnumpy.PyArray_SimpleNewFromData(1, dims,
cnumpy.NPY_DOUBLE, self._values)
^
------------------------------------------------------------
/scratch/mvngu/release/sage-4.1.1.alpha0/devel/sage-
main/sage/finance/time_series.pyx:1862:97: Cannot convert 'double *' to
Python object
Error running command, failed with status 256.
sage: There was an error installing modified sage library code.
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4571#comment:12>
Sage <http://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 post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
