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
