Skip to content

Split off Graphics class from plot.py #12857

@jdemeyer

Description

@jdemeyer

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 :

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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions