Opened 10 years ago

Closed 13 months ago

#12854 closed defect (invalid)

Severe slow-down in plotting

Reported by: jdemeyer Owned by: tbd
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: performance Keywords:
Cc: Merged in:
Authors: Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

Plotting has severely slowed down since sage-4.8.

For example the following command (inspired on a doctest from devel/sage/sage/plot/plot.py):

sage: timeit('''plot(sin(x),(x,0,2*pi),ticks=pi/3,tick_formatter=pi).save("/tmp/1.png")''', number=1, repeat=20)

It's annoying to measure this as the timing has quite a large standard deviation. But on boxen.math, with sage-4.8, the best case timing is about 1 second, while with sage-5.0.beta13, the best case timing is about 1.5 seconds.

Change History (10)

comment:1 Changed 10 years ago by kcrisman

This is due to the change in attachment:trac-11915-matplotlib-1.1.0-upgrade.patch:ticket:11915 that reads

figure.tight_layout() 

Comment it out and see the times go down. See #11915 for the rationale behind adding this, which I will point out was not arbitrary.

comment:2 Changed 10 years ago by jdemeyer

So should we simply keep the tight_layout() and live with it?

comment:3 Changed 10 years ago by jdemeyer

  • Priority changed from blocker to critical

comment:4 Changed 8 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:5 Changed 8 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:6 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:7 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:8 Changed 14 months ago by mkoeppe

  • Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
  • Priority changed from critical to major
  • Status changed from new to needs_review

Outdated, should be closed

comment:9 Changed 14 months ago by tscrim

  • Reviewers set to Travis Scrimshaw
  • Status changed from needs_review to positive_review

I concur.

comment:10 Changed 13 months ago by chapoton

  • Resolution set to invalid
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.