Fernando Perez wrote the following on 04/17/2008 06:31 PM:
> ps - for now, using the deprecated 'faceted=False' seems to still
> work, so I'll go with that.

I noticed this a few days ago as well...using edgecolors='None' seemed 
to have the same effect as faceted=False.  This brings up a couple issues.

1) I find the shading=* in the docstring to be a bit confusing.  The 
equal sign makes me think of keyword arguments. Since 'shading' is not a 
valid keyword, additional clarity might result if we do not use the 
equal sign.

2) If we remove the equal signs, we might have something like this:

      This kwarg is deprecated;
      please use the edgecolors kwarg instead:
         flat shading     --> edgecolors='None'
         faceted shading  --> edgecolors=None

But even this is confusing. Reading

http://www.physnet.uni-hamburg.de/physnet/matlab/help/techdoc/ref/shading.html

makes me think we should have:

      This kwarg is deprecated;
      please use the edgecolors kwarg instead:
         flat shading     --> edgecolors=None
      For faceted shading, edgecolors can be any mpl color or sequence 
of colors.


---

So it seems like the code is working, but that the docstring is wrong. 
Perhaps the behavior should be changed so that edgecolors must be set to 
None rather than 'None' when requesting flat shading.

~C


-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel

Reply via email to