#2100: sensible defaults for aspect ratio
--------------------------------------+-------------------------------------
Reporter: jason | Owner: was
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-4.6.2
Component: graphics | Keywords:
Author: Jason Grout | Upstream: N/A
Reviewer: Andrey Novoseltsev | Merged:
Work_issues: complete documentation |
--------------------------------------+-------------------------------------
Comment(by kcrisman):
Replying to [comment:26 kcrisman]:
> I had to change a few things based on the new save() but otherwise
nearly all is the same. It also turned out that fig_tight was never used
in .matplotlib(), and in fact only can be passed to matplotlib's
savefig(), so I treated it basically the same as transparent in .save().
>
> I can also remove 'equal' if you want, that's fine.
Done as well.
> Before I go ahead and update this, though, I have a question. The two
functions not documented (`pad_for_tick_labels` and
`adjust_figure_to_contain_bbox`) only are used in a commented-out line
> {{{
> # this messes up the aspect ratio!
> #figure.canvas.mpl_connect('draw_event',
pad_for_tick_labels)
> }}}
> So... should they even be included at this time?
Quote from Jason: "Feel free to move those functions to another ticket if
they are really not called from anywhere. I think they are leftovers from
experimentation, so probably not strictly needed (especially if they
really aren't called or used from anywhere)."
New patch coming up relatively soon.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/2100#comment:28>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
