id summary reporter owner description type status priority milestone component resolution keywords cc merged author reviewer upstream work_issues branch commit dependencies stopgaps
12857 Split off Graphics class from plot.py jdemeyer tbd "The file `sage/plot/plot.py` is getting huge. One consequence is that doctesting this file (without `--long`) takes double the time of any other file. '''blocker''' because it regularly causes doctest timeouts.
The following will be moved to a new file `graphics.py`:
1. class `Graphics`
2. class `GraphicsArray`
3. def `show_default`
4. def `is_Graphics`
A few doctests consistently taking at least 2 seconds on `sage.math` have been marked `# long time`.
Before this patch on `sage.math`:
{{{
sage -t ""devel/sage/sage/plot/plot.py""
[133.6 s]
}}}
After this patch on `sage.math`:
{{{
sage -t ""devel/sage/sage/plot/graphics.py""
[51.5 s]
sage -t ""devel/sage/sage/plot/plot.py""
[67.9 s]
}}}
The patch was created by first ''copying'' `plot.py` to `graphics.py` and then
removing the duplicate code in either `plot.py` or `graphics.py`.
'''Apply :'''
- attachment:12857_split_graphics.patch
- attachment:trac_12857-split_graphics_review-fh.patch
" enhancement closed blocker sage-5.0 graphics fixed kcrisman sage-5.0.rc0 Jeroen Demeyer Benjamin Jones, Florent Hivert N/A