On Monday, July 23, 2012, Eric Firing wrote:
> On 2012/07/23 11:43 AM, Damon McDougall wrote:
> > Hello all,
> >
> > So, as per Philip's suggestion
> > (https://github.com/matplotlib/matplotlib/pull/737) I've started
> > encapsulating fplot functionality into a class. The point of this is so
> > that the user can call either of the following:
> >
> > FPlot_instance = ax.fplot(f, [x0, y0, x1, y1])
> > ax.fplot(FPlot_instance, ...)
>
> The second of these seems odd to me; I would expect FPlot_instance to
> have a __call__ method, so the normal use of an existing instance would be
>
> FPlot_instance(...)
>
> Also, regarding the second argument in the first form: I would think it
> more natural to split it up into a required [x0, x1] and an optional
> [y0, y1], with autoscaling if it is not provided.
>
> Eric
Agreed, it is a bit odd/awkward, and I also agree about autoscaling.
With the whole viewlims callbacks, make sure you have the class disconnect
itself upon removal, such as through cla().
l <https://lists.sourceforge.net/lists/listinfo/matplotlib-devel>
Cheers!
Ben Root
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
