Ticket #6098 (closed enhancement: fixed)

Opened 16 months ago

Last modified 14 months ago

[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 Author(s): Emily Kirkman
Report Upstream: Reviewer(s): Robert Miller
Merged in: sage-4.1.1.alpha0 Work issues:

Description


Attachments

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

Change History

Changed 16 months ago by ekirkman

  • summary changed from 3d bezier path plotting to [with patch, needs review] 3d bezier path plotting

Changed 16 months ago by ekirkman

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

Changed 16 months ago by rlm

Referee edit

Changed 16 months ago by rlm

  • summary changed from [with patch, needs review] 3d bezier path plotting to [with patch, positive review] 3d bezier path plotting

Changed 15 months ago by ncalexan

  • reviewer set to Robert Miller
  • summary changed from [with patch, positive review] 3d bezier path plotting to [with patch, needs work] 3d bezier path plotting
  • author set to Emily Kirkman

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 months ago by rlm

Apply only this patch

Changed 14 months ago by rlm

  • summary changed from [with patch, needs work] 3d bezier path plotting to [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! :)

Changed 14 months 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 :-(

Changed 14 months ago by mvngu

  • status changed from new to closed
  • resolution set to fixed
  • merged set to sage-4.1.1.alpha0
Note: See TracTickets for help on using tickets.