#5230: [with patch, positive review] implement boundary_style parameter for
parametric 3d plots
-------------------------+--------------------------------------------------
 Reporter:  was          |       Owner:  wcauchois 
     Type:  enhancement  |      Status:  assigned  
 Priority:  major        |   Milestone:  sage-3.4.1
Component:  graphics     |    Keywords:            
-------------------------+--------------------------------------------------

Comment(by was):

 REFEREE REPORT:

 It works perfectly.

  * Add to "        boundary_style -- (default: None, no boundary) a dict
 that describes
                           how to draw boundaries of regions"
 that the dict just gives the options that are passed to the line3d
 command.

  * Put a space after the doctest:
 {{{
         105             sage: parametric_plot3d((cos(u), sin(u) + cos(v),
 sin(v)), (u, 0, pi), (v, 0, pi), boundary_style={"color": "black",
 "thickness": 2})
         106         Any options you would normally use to specify the
 appearance of a curve are valid
         107         as entries in the boundary_style dict.
 }}}

 I attached a patch that makes these trivial changes.  With this, positive
 review.

 NOTE: This patch will have to be rebased for 3.4.alpha because of the ReST
 transition.  It applies fine to 3.3 though.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5230#comment:3>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of 
Reinventing the Wheel

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to