cube(center, size=0.5) does not return a cube centered at center

[seblabbe]

The center of the cube is affected by the argument size :

sage: c = cube(center=(10,10,10), size=0.5)              
sage: c.bounding_box()                            
((4.75, 4.75, 4.75), (5.25, 5.25, 5.25))          

The intended behavior is that the center is independant of the size:

sage: c = cube((10,10,10), size=0.5)              
sage: c.bounding_box()  
((9.75, 9.75, 9.75), (10.25, 10.25, 10.25))  

For the patchbot : Apply trac_11272-doctest.patch

Component: graphics

Keywords: cube

Author: Sébastien Labbé

Reviewer: Jeroen Demeyer

Merged: sage-4.7.1.alpha1

Issue created by migration from https://trac.sagemath.org/ticket/11272

[sagetrac-johanbosman]

Attachment: trac_11272_cube_centre.patch.gz

[seblabbe]

Author: Johan Bosman

[seblabbe]

comment:2

Problem is fixed. All test passed in the directory sage/plot/plot3d. Documentation builds fine. Positive review. Thanks for the fix!

Sébastien

[seblabbe]

Reviewer: Sébastien Labbé

[jdemeyer]

comment:3

Needs to be rebased to sage-4.7.rc1

[sagetrac-johanbosman]

comment:4

It seems to work properly in sage-4.7.rc1 without patches. Apparently the bug has already been fixed in another ticket.

[jdemeyer]

Changed author from Johan Bosman to none

[jdemeyer]

Changed reviewer from Sébastien Labbé to Sébastien Labbé, Johan Bosman

[jdemeyer]

comment:5

Duplicate of #10796.

[seblabbe]

comment:6

I wonder if the doctest could be added to Sage anyway. Can we reopen the ticket for it?

Sébastien

[jdemeyer]

comment:8

Replying to @seblabbe:

I wonder if the doctest could be added to Sage anyway.

Good idea!

[seblabbe]

comment:9

Attachment: trac_11272-doctest.patch.gz

[jdemeyer]

Changed reviewer from Sébastien Labbé, Johan Bosman to Jeroen Demeyer

[jdemeyer]

Author: Sébastien Labbé

[jdemeyer]

Merged: sage-4.7.1.alpha1