#6456: Upgrade cvxopt in sage from 0.9 to 1.1.2
--------------------------------+-------------------------------------------
Reporter: was | Owner: mabshoff
Type: defect | Status: needs_work
Priority: major | Milestone: sage-4.5.2
Component: packages | Keywords:
Author: schilly, dimpase | Upstream: Completely fixed; Fix reported
upstream
Reviewer: | Merged:
Work_issues: |
--------------------------------+-------------------------------------------
Changes (by dimpase):
* upstream: Reported upstream. Developers acknowledge bug. => Completely
fixed; Fix reported upstream
Comment:
Replying to [comment:80 dimpase]:
> Replying to [comment:50 pjeremy]:
[...]
> the following 2-byte change appears to cure the problem. Semantically,
this
> restricts the dense matrices that are dealt with to the sizes 2^31 by
2^31, but this
> is OK for all the practical purposes.
> Hopefully CVXOPT people will come up with a better fix, but for the time
being
> this should suffice and cure this particular headache.
>
>
{{{
--- a/patches/cvxopt.h Mon Jul 26 18:45:42 2010 +0300
+++ b/patches/cvxopt.h Thu Jul 29 09:12:21 2010 -0700
@@ -61,7 +61,7 @@
typedef struct {
PyObject_HEAD
void *buffer; /* in column-major-mode array of type 'id' */
- int_t nrows, ncols; /* number of rows and columns */
+ int nrows, ncols; /* number of rows and columns -- was int_t */
int id; /* DOUBLE, INT, COMPLEX */
} matrix;
}}}
One of cvxopt developers has acknowledged this as a valid fix. Further, he
says that it will get into a new cvxopt release, 1.1.3, due shortly.
I am inclined to wait for 1.1.3, while preparing an spkg-check starting
off from Dave's version, and eventually sorting out OSX and Cygwin.
(and eventually hooking up the cvxopt's documentation to Sage's
documentation)
Further, the Sun complex.h related issue, I looked at the
/usr/include/complex.h over on t2 and mark. It just does not make sense
how _Imaginary_I is (not) defined there,
something like
{{{
#define _Imaginary_I _Imaginary_I
}}}
---no wonder it does not work. While it does make sense in the supplied
patches/sun_complex.h, which differs from the former at this and few other
places.
Dima
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6456#comment:82>
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 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.
