#15260: Floor (integer) division of reals and rationals
-------------------------------------+-------------------------------------
       Reporter:  ncohen             |        Owner:
           Type:  defect             |       Status:  needs_info
       Priority:  major              |    Milestone:  sage-7.1
      Component:  basic arithmetic   |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Jan Keitel,        |    Reviewers:  Jeroen Demeyer,
  Jeroen Demeyer                     |  Thierry Monteil
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/jdemeyer/ticket/15260            |  616f88c009990bd9d778970ac2bb3496e500b1a1
   Dependencies:  #2034              |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by bruno):

 I used `QQ` as an example, but the same holds for other fields. In the
 current branch, `__floordiv__` always returns an integer, while it should
 to my mind always be internal. Note that I am strongly against raising an
 exception since it would be a useless regression. (By this I mean that it
 would break some existing code, and bring nothing interesting.) I guess
 that my viewpoint is the same as (close to?) the one expressed by Vincent
 in comment:28 and comment:30.

 Replying to [comment:31 vbraun]:
 > The sage-devel thread is in favor of throwing an exception.

 I do not see any such consensus in the sage-devel thread! You are the only
 one proposing to raise an exception... William says that it is a
 reasonable option, as is Jeroen's proposal.

--
Ticket URL: <http://trac.sagemath.org/ticket/15260#comment:35>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to