#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.

Reply via email to