Ticket #2899 (new defect)

Opened 9 months ago

Last modified 1 month ago

[with patch, needs rebase] Make RDF round and friends return Integers

Reported by: jason Assigned to: robertwb
Priority: major Milestone: sage-3.4
Component: coercion Keywords:
Cc:

Description 

> Also, round(RR(3.0)) returns an Integer...should RDF behave the same
> > way? (currently round(RDF(3.0)) returns an RDF).

We recently changed round, floor, ceiling, and trunc on RR to return
integers; yes, I think the corresponding RDF methods should change as
well.

Carl

Attachments

2899-ncalexan-RIF-floor-ceil-1.patch (4.4 kB) - added by ncalexan on 08/13/2008 05:36:01 PM.

Change History

04/12/2008 09:39:18 PM changed by jason

The fix for #2898 will fix this.

08/13/2008 05:36:01 PM changed by ncalexan

  • attachment 2899-ncalexan-RIF-floor-ceil-1.patch added.

08/13/2008 05:38:58 PM changed by ncalexan

  • summary changed from Make RDF round and friends return Integers to [with patch, needs review] Make RDF round and friends return Integers.

This makes floor and ceil do what I expect for RIF. I believe that #2898 does make RDF work.

11/27/2008 12:11:11 AM changed by craigcitro

  • summary changed from [with patch, needs review] Make RDF round and friends return Integers to [with patch, needs rebase] Make RDF round and friends return Integers.

I just tried to apply this against 3.2, and it fails. It's just a matter of moving the doctests around, I think.