Opened 14 years ago

Closed 14 years ago

#6098 closed enhancement (fixed)

[with patch, positive review] 3d bezier path plotting

Reported by: ekirkman Owned by: ekirkman
Priority: major Milestone: sage-4.1.1
Component: graphics Keywords:
Cc: rlm, mhansen Merged in: sage-4.1.1.alpha0
Authors: Emily Kirkman Reviewers: Robert Miller
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description


Attachments (3)

curve.jpg​ (23.1 KB) - added by ekirkman 14 years ago.
This picture was created with input sage: bezier3d((0,0,0),(1,0,0),(0,1,0),(0,1,1)?)
trac6098_bezier3d.patch​ (9.7 KB) - added by rlm 14 years ago.
Referee edit
trac_6098-rebased.patch​ (9.1 KB) - added by rlm 14 years ago.
Apply only this patch

Download all attachments as: .zip

Change History (9)

comment:1 Changed 14 years ago by ekirkman

Summary: 3d bezier path plotting → [with patch, needs review] 3d bezier path plotting

Changed 14 years ago by ekirkman

Attachment: curve.jpg​ added

This picture was created with input sage: bezier3d((0,0,0),(1,0,0),(0,1,0),(0,1,1)?)

Changed 14 years ago by rlm

Attachment: trac6098_bezier3d.patch​ added

Referee edit

comment:2 Changed 14 years ago by rlm

Summary: [with patch, needs review] 3d bezier path plotting → [with patch, positive review] 3d bezier path plotting

comment:3 Changed 14 years ago by ncalexan

Authors: → Emily Kirkman
Reviewers: → Robert Miller
Summary: [with patch, positive review] 3d bezier path plotting → [with patch, needs work] 3d bezier path plotting

Unfortunately, against 4.0.2.alpha0: 

----------------------------------------------------------------------

The following tests failed:

        sage -t -long devel/sage/sage/plot/plot3d/shapes2.py # 4 doctests failed
        sage -t -long devel/sage/sage/plot/plot3d/shapes2.py # 4 doctests failed
        sage -t -long devel/sage/sage/plot/bezier_path.py # 2 doctests failed
----------------------------------------------------------------------

Changed 14 years ago by rlm

Attachment: trac_6098-rebased.patch​ added

Apply only this patch

comment:4 Changed 14 years ago by rlm

Summary: [with patch, needs work] 3d bezier path plotting → [with patch, positive review] 3d bezier path plotting

It was completely trivial to get this patch working again. I did the work while Emily watched over my shoulder. If one positive review isn't enough, then you have two! :)

comment:5 Changed 14 years ago by mvngu

Oops... ignore the patch trac_6098-reviewer.patch. Just to let people know, only the patch trac_6098-rebased.patch has been merged in sage-4.1.1-alpha0. I can't close this ticket because I don't have the privilege to do so. Sorry, folks :-(

comment:6 Changed 14 years ago by mvngu

Merged in: → sage-4.1.1.alpha0
Resolution: → fixed
Status: new → closed
Note: See TracTickets for help on using tickets.