Hi Ken,

Thanks for your comments.

Ken McIvor wrote:

>1. It appears that as_sizer_element() uses the _axes_sizer_elements  
>dictionary to cache MplAxesSizerElement instances.  Using a  
>WeakKeyDictionary from the "weakref" module instead of a regular  
>dictionary may be necessary to allow the garbage collection of the  
>MplAxesSizerElements when their associated Axes gets GC'd.
>  
>
I'll check into this.

>2. Convenience MplBoxSizer subclasses that let you omit the "orient"  
>keyword might be nice:
>
>     class MplHBoxSizer(MplBoxSizer):
>         def __init__(self):
>             MplBoxSizer.__init__(self, orientation='vertical')
>  
>
I was thinking I want an exact a replica of the wx API as possible,
simply so my feeble mind doesn't get (more) confused. Is there anything
like this in stock wx? I think it's a decent enough idea, but I'd rather
not cause namespace bloat. But, if you're really into it, let's do it.

>3. Couldn't you just drop mplsizer.py into the "matplotlib.toolkits"  
>virtual package?  Maybe you can't -- I'm pretty new to applied python- 
>eggery.
>  
>
I think this is possible, and I hope for not too much effort. This is
the result of historic accident and personal time-allocation issues.
Mplsizer started life as a private setuptools-ized package, and I'm not
spending my time to de-setuptools-ize software -- I see setuptools as
the way forward, not something (I will spend my time) to remove. I
understand that some people aren't too fond of setuptools, and to them,
I say, "patches welcome".

>4. I feel we should avoid the whole European/American spelling  
>problem that WX has.  Why not make both 'align_centre' and  
>'align_center' do the same thing?
>  
>
Same answer as to point 2...

>5. Why not use shorter names, with less redundancy? (e.g.  
>"matplotlib.toolkits.sizer", FigureSizer, Box, HBox, Grid, etc)
>  
>
Same answer as to point 2... (I'd be happy to drop the "Mpl" prefix,
though.)

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel

Reply via email to