Opened 11 years ago
Closed 11 years ago
#11272 closed defect (fixed)
cube(center, size=0.5) does not return a cube centered at center
Reported by: | slabbe | Owned by: | jason, was |
---|---|---|---|
Priority: | major | Milestone: | sage-4.7.1 |
Component: | graphics | Keywords: | cube |
Cc: | Merged in: | sage-4.7.1.alpha1 | |
Authors: | Sébastien Labbé | Reviewers: | Jeroen Demeyer |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
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
Attachments (2)
Change History (12)
Changed 11 years ago by
comment:1 Changed 11 years ago by
- Status changed from new to needs_review
comment:2 Changed 11 years ago by
- Reviewers set to Sébastien Labbé
- Status changed from needs_review to positive_review
comment:3 Changed 11 years ago by
- Status changed from positive_review to needs_work
Needs to be rebased to sage-4.7.rc1
comment:4 Changed 11 years ago by
It seems to work properly in sage-4.7.rc1 without patches. Apparently the bug has already been fixed in another ticket.
comment:5 Changed 11 years ago by
- Milestone changed from sage-4.7.1 to sage-duplicate/invalid/wontfix
- Resolution set to duplicate
- Reviewers changed from Sébastien Labbé to Sébastien Labbé, Johan Bosman
- Status changed from needs_work to closed
Duplicate of #10796.
comment:6 follow-up: ↓ 8 Changed 11 years ago by
I wonder if the doctest could be added to Sage anyway. Can we reopen the ticket for it?
Sébastien
comment:7 Changed 11 years ago by
- Milestone changed from sage-duplicate/invalid/wontfix to sage-4.7.1
- Resolution duplicate deleted
- Status changed from closed to new
comment:8 in reply to: ↑ 6 Changed 11 years ago by
Changed 11 years ago by
comment:9 Changed 11 years ago by
- Description modified (diff)
- Status changed from new to needs_review
comment:10 Changed 11 years ago by
- Merged in set to sage-4.7.1.alpha1
- Resolution set to fixed
- Reviewers changed from Sébastien Labbé, Johan Bosman to Jeroen Demeyer
- Status changed from needs_review to closed
Note: See
TracTickets for help on using
tickets.
Problem is fixed. All test passed in the directory
sage/plot/plot3d
. Documentation builds fine. Positive review. Thanks for the fix!Sébastien