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:
- class
Graphics
- class
GraphicsArray
- def
show_default
- 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 :
CC: @kcrisman
Component: graphics
Author: Jeroen Demeyer
Reviewer: Benjamin Jones, Florent Hivert
Merged: sage-5.0.rc0
Issue created by migration from https://trac.sagemath.org/ticket/12857
The file
sage/plot/plot.pyis 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:GraphicsGraphicsArrayshow_defaultis_GraphicsA few doctests consistently taking at least 2 seconds on
sage.mathhave been marked# long time.Before this patch on
sage.math:After this patch on
sage.math:The patch was created by first copying
plot.pytographics.pyand thenremoving the duplicate code in either
plot.pyorgraphics.py.Apply :
CC: @kcrisman
Component: graphics
Author: Jeroen Demeyer
Reviewer: Benjamin Jones, Florent Hivert
Merged: sage-5.0.rc0
Issue created by migration from https://trac.sagemath.org/ticket/12857