Opened 15 years ago
Closed 15 years ago
#1833 closed defect (fixed)
[with patch; positive review] plot3d and parametric_plot3d can be very slow on some inputs
| Reported by: | William Stein | Owned by: | William Stein |
|---|---|---|---|
| Priority: | major | Milestone: | sage-2.10.1 |
| Component: | graphics | Keywords: | |
| Cc: | Merged in: | ||
| Authors: | Reviewers: | ||
| Report Upstream: | N/A | Work issues: | |
| Branch: | Commit: | ||
| Dependencies: | Stopgaps: |
Status badges
Description
sage: x,y = var('x,y')
sage: plot3d(x*y, (x,-1,1), (y,-1,1))
is fast, but
sage: x,y = var('x,y')
sage: plot3d(x*y, (-1,1), (-1,1))
is shockingly slow (and similar remarks for parametric plots). The attached patch fixes this problem.
This also fixes trac #1737.
Attachments (1)
Change History (5)
comment:1 Changed 15 years ago by
| Component: | algebraic geometry → graphics |
|---|
comment:2 Changed 15 years ago by
Changed 15 years ago by
| Attachment: | trac-1833.patch added |
|---|
comment:3 Changed 15 years ago by
| Summary: | [with patch; needs review] plot3d and parametric_plot3d can be very slow on some inputs → [with patch; positive review] plot3d and parametric_plot3d can be very slow on some inputs |
|---|
Looks good to me.
comment:4 Changed 15 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Merged in Sage 2.10.1.alpha1
Note: See
TracTickets for help on using
tickets.
This patch also moves plot3d_adaptive into plot3d (i.e., as an option), and deprecates globally exposing plot3d_adaptive. This is natural to do in the context of this patch.