Opened 15 years ago

Closed 15 years ago

#1845 closed defect (fixed)

[with patch, spkg, with positive review] list_plot3d should be able to accept lists of points in arbitrary positions

Reported by: jkantor 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

Attachments (1)

trac-1845-fix-doctest.patch (870 bytes) - added by Carl Witty 15 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 15 years ago by Michael Abshoff

Milestone: sage-2.10.1

comment:2 Changed 15 years ago by Carl Witty

Description: modified (diff)

Changed 15 years ago by Carl Witty

Attachment: trac-1845-fix-doctest.patch added

comment:3 Changed 15 years ago by Carl Witty

Summary: [with patch, spkg] list_plot3d should be able to accept lists of points in arbitrary positions[with patch, spkg, with positive review] list_plot3d should be able to accept lists of points in arbitrary positions

Code looks good; doctests pass in sage/plot. I went through all the doctests by hand and they all work and look pretty. One of the doctests was malformed (syntax error); the attached trac-1845-fix-doctest.patch fixes that.

I did not look at differences between the old and new scipy_sandbox, or check old functionality of that spkg.

To apply: apply list_plot3d.hg, then trac-1845-fix-doctest.patch; and install the new scipy_sandbox spkg.

comment:4 Changed 15 years ago by Michael Abshoff

Resolution: fixed
Status: newclosed

Merged in Sage 2.10.1.rc1: the bundle and updated scipy_sandbox as well as cwitty's doctest fix patch.

Note: See TracTickets for help on using tickets.