On Mon, Mar 1, 2010 at 8:12 PM, Jae-Joon Lee <lee.j.j...@gmail.com> wrote: > John and T J, > > L1587 at lines.py > > def set_mfc(self, val): > 'alias for set_markerfacecolor' > self.set_markerfacecolor(val, alt=alt) > > "alt" is not defined and it currently raises an exception.
Fixed -- thanks for the catch. > By the way, I noticed that the current approach is to implement > fillstyle for EVERY markers. > An alternative approach would be using a big enough circle for > fillstyle and clip it with the full marker path. > The number, of "draw_markers" call increases but the code will be much > simplified and more easy to maintain. Just a thought. We do some significant optimizations in the agg backend in RendererAgg::draw_markers with cached rendered markers, so some extra machinery would have to be pushed into the backend to continue supporting this optimization if we wanted to follow this suggestion. ------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev _______________________________________________ Matplotlib-users mailing list Matplotlib-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/matplotlib-users
