#17570: ToricRationalDivisorClass: Integer/Rational mixed up
------------------------+----------------------------
Reporter: jdemeyer | Owner:
Type: defect | Status: new
Priority: major | Milestone: sage-6.5
Component: geometry | Keywords:
Merged in: | Authors:
Reviewers: | Report Upstream: N/A
Work issues: | Branch:
Commit: | Dependencies:
Stopgaps: |
------------------------+----------------------------
In `src/sage/schemes/toric/divisor.py`, there is:
{{{
class
ToricRationalDivisorClassGroup_basis_lattice(FreeModule_ambient_pid):
def __init__(self, group):
# ...
super(ToricRationalDivisorClassGroup_basis_lattice,
self).__init__(
ZZ, group.dimension())
}}}
(note the `ZZ`)
However, in `src/sage/schemes/toric/divisor_class.pyx`, there is
{{{
cdef class ToricRationalDivisorClass(Vector_rational_dense):
}}}
So, do we want rationals or integers? This causes segfaults when improving
vectors in #17562.
--
Ticket URL: <http://trac.sagemath.org/ticket/17570>
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.