#6805: [with patch, needs review] Integrality check in LatticePolytope
----------------------+-----------------------------------------------------
Reporter: novoselt | Owner: mhampton
Type: defect | Status: new
Priority: major | Milestone: sage-4.1.2
Component: geometry | Keywords:
Reviewer: | Author:
Merged: |
----------------------+-----------------------------------------------------
It is possible now to create a lattice polytope with rational vertices,
which allows things to work, but causes wrong answers:
{{{
sage: m = matrix([1/2, 3/2])
sage: m
[1/2 3/2]
sage: LatticePolytope(m).points()
[0 1]
}}}
This patch adds an extra check/conversion to the constructor to prevent
this:
{{{
sage: m = matrix([1/2, 3/2])
sage: m
[1/2 3/2]
sage: LatticePolytope(m).points()
Traceback (most recent call last):
...
ValueError: Points must be integral!
Given:
[1/2 3/2]
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6805>
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
-~----------~----~----~----~------~----~------~--~---
