ggsave() should take a size parameter about ggpy HOT 2 CLOSED

Should be pretty easy to add this. Almost positive matplitlib fig.save also accepts a size parameter. If not then we can just hijack te fig size from the theme right before the plot is rendered.

from ggpy.

So, here is the ggsave documentation from http://docs.ggplot2.org/0.9.3.1/ggsave.html

ggsave(filename = default_name(plot), plot = last_plot(), device = default_device(filename), 
  path = NULL, scale = 1, width = par("din")[1], height = par("din")[2], units = c("in", 
          "cm", "mm"), dpi = 300, limitsize = TRUE, ...)

and here is the one for savefig(...):

savefig(fname, dpi=None, facecolor='w', edgecolor='w',
        orientation='portrait', papertype=None, format=None,
        transparent=False, bbox_inches=None, pad_inches=0.1,
        frameon=None)

So, what is possible from the ggsave parameter:

• filename: file name/filename of plot
• plot: plot to save, defaults to last plot displayed -> see below
• device: device to use, automatically extract from file name extension -> Would need a translation to format
• path: path to save plot to (if you just want to set path and not filename)
• scale: scaling factor -> No idea
• width: width (defaults to the width of current plotting window) -> bbox_inches
• height: height (defaults to the height of current plotting window) -> bbox_inches
• units: units for width and height when either one is explicitly specified (in, cm, or mm) -> would need a translation into bbox_inches
• dpi: dpi to use for raster graphics
• limitsize: when TRUE (the default), ggsave will not save images larger than 50x50 inches, to prevent the common error of specifying dimensions in pixels.

The only interesting thing I can see is the plot argument: if there is a plot argument we would call __repr__() on it an then save the last plt.gcf(). But that would mean that __repr__() would NOT print the plot in interactive mode.

So what would you say to the following proposal:

• add a 'draw_plot()' which would 1) set interactive to false 2) draw the plot 3) reset interactive to whatever the value was before
• change __repr__() to call draw_plot() (and afterwards plt.show()?)

from ggpy.