|
From: Phil E. <pel...@gm...> - 2013-08-09 12:25:07
|
Hi Martin, Thanks for this - we are really interested in speeding up the scatter and barchart plotting with large data sets. In fact, we've done some work ( https://github.com/matplotlib/matplotlib/pull/2156) recently to make the situation better. I'd really like to review these changes (against matplotlib master), and the best possible solution to doing this is if you were to submit a pull request. If the changes you have made are logically seperable, then I'd encourage you to make a few PRs, but otherwise, a single PR with all of these changes would be great. Would you mind turning these patches into PR(s)? ( https://github.com/matplotlib/matplotlib/compare/) Thanks! Phil On 9 August 2013 12:53, Martin Mokrejs <mmo...@fo...> wrote: > Hi, > I am drawing some barcharts and scatter plot and the speed for rendering > is awful once you have > 100 000 of dots. I ran python profiler which lead me to .startswith() > calls and some for loops > which append do a list repeatedly. This parts could be still sped up I > think but a first attempt > is here: > > > UNPATCHED 1.2.1 > > real 23m17.764s > user 13m25.880s > sys 3m37.180s > > > PATCHED: > > real 6m59.831s > user 5m18.000s > sys 1m40.360s > > > > The patches are simple and because I see elsewhere in the code list > expansions I do not see any > problems with backwards compatibility (new new python language features > are required). > > Hope this helps, > Martin > > > ------------------------------------------------------------------------------ > Get 100% visibility into Java/.NET code with AppDynamics Lite! > It's a free troubleshooting tool designed for production. > Get down to code-level detail for bottlenecks, with <2% overhead. > Download for free and get started troubleshooting in minutes. > http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk > _______________________________________________ > Matplotlib-users mailing list > Mat...@li... > https://lists.sourceforge.net/lists/listinfo/matplotlib-users > > |