#10043: Complete rewrite of LP solver interfaces
----------------------------------+-----------------------------------------
Reporter: ncohen | Owner: ncohen
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-4.6
Component: linear programming | Keywords:
Author: Nathann Cohen | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
----------------------------------+-----------------------------------------
Comment(by ncohen):
Replying to [comment:5 malb]:
Hello !
> I applied the patch to 4.6.alpha3 without problems and doctests pass.
Before anything else, thank you for that `:-)`
> The overall design looks good. However, the documentation is much much
too sparse, especially since this is supposed to be a generic interface.
All functions should get a docstring describing their expected behaviour
and all instantiations should have docstrings testing them (of course,
only indirectly since they are cdef'ed).
Got it. I will document the expected INPUT/OUTPUT, but do you really think
doctests are necessary at this level ? As you say, it will be indirect as
these methods are all cdef (sage -coverage answers 100% on these files
because of that), so as there is a wealth of methods testing them all in
the graph/ and numerical/ directory... That's an honest question, as I do
not see how to write smarter doctests than just a short example of how LP
is used in Sage, and copy/paste it in each function as the same code would
test them all...
> I know it is a lot of work I'm asking for, but I think it is necessary.
Indeed. As long as you ask it knowing it means a lot of time, I do not
mind doing it (as soon as I will have your advise onthe previous question)
Thanks
Nathann
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10043#comment:6>
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.
