#17920: Reimplement IntegerLists using Polyhedron.integral_points()
-------------------------------------+-------------------------------------
Reporter: jdemeyer | Owner:
Type: enhancement | Status: new
Priority: major | Milestone: sage-6.6
Component: combinatorics | Resolution:
Keywords: | Merged in:
Authors: Jeroen Demeyer | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/jdemeyer/ticket/17920 | 755b67a1c9516bcd44f59aab6a8b6032d257591d
Dependencies: #17937 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by ncohen):
Replying to [comment:12 jdemeyer]:
> This now seems to work reasonably well. Not yet ready, but good enough
for example to compare with the existing implementation. That's how I
found all the bugs at #17548.
>
> Due to the polyhedra overhead, it is generally (a lot) slower than the
existing code.
Is it worth changing this branch so that it changes the existing code
instead of adding a new file ? As you said: let's be correct first, *then*
fast.
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/17920#comment:13>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
