#7012: clean up sage/numerical/mip.pyx
-------------------------+--------------------------------------------------
Reporter: mvngu | Owner: jkantor
Type: enhancement | Status: new
Priority: major | Milestone: sage-4.1.2
Component: numerical | Keywords:
Reviewer: | Author:
Merged: |
-------------------------+--------------------------------------------------
As the subject says. This is a follow up to #6869 to address mhansen's
suggestions:
{{{
After going through this patch, I think it would be best to revert it
before 4.1.2 is released. There is still a lot of things that need to be
done to get it cleaned up. Some of the things,
1. Almost all of the docstrings are incorrectly formatted.
1. This module should _definitely_ not be a Cython module as it does not
gain any benefit from Cython. It just makes Sage slower to compile and
things harder to debug.
1. Some of the naming conventions do not match Sage's conventions.
(isbinary vs. is_binary). Also, names like the more explicit
MixedIntegerProgram? are preferred over the shortened MIP.
1. The optional spkgs should not install modules into the sage.* namespace
(sage.numerical.mipGlpk). This is not the right way to do things and will
eventually break. I think it also makes sense to use (and contribute to)
something like ctypes-glpk to allow greater functionality and not reinvent
the wheel.
I have some code that addresses some of these things that I'll put up
shortly.
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7012>
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
-~----------~----~----~----~------~----~------~--~---
