#18004 closed defect (fixed)
Too small relative size of axes labels w.r.t. tick marks in 2D plots
Reported by: | egourgoulhon | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-6.7 |
Component: | graphics | Keywords: | days64 |
Cc: | vdelecroix, kcrisman | Merged in: | |
Authors: | Eric Gourgoulhon | Reviewers: | Vincent Delecroix |
Report Upstream: | N/A | Work issues: | |
Branch: | 69354fc (Commits, GitHub, GitLab) | Commit: | |
Dependencies: | Stopgaps: |
Description
Consider this plot:
sage: plot(sin(x^2), (x, -3, 3), axes_labels=['$x$','$y$'])
The size of the labels x and y is quite small and barely readable when the figure is reproduced in a article with moderate size (cf. the attached figure). In the current settings, the only way to enlarge it is via the optional argument fontsize:
sage: plot(sin(x^2), (x, -3, 3), axes_labels=['$x$','$y$'], fontsize=20)
But then the tick marks are too large! One should be able to set separately the size of the axes labels and that of the tick marks.
Attachments (1)
Change History (26)
Changed 8 years ago by
Attachment: | axes_labels.png added |
---|
comment:1 Changed 8 years ago by
Branch: | → public/18004 |
---|---|
Commit: | → 10b69240dfceaec610564bb26e2ab7eb1a4a5b45 |
comment:2 Changed 8 years ago by
The attached commit introduces an optional parameter for 2D graphics, axes_labels_size
, which sets the size of axes labels relative to that of tick marks. The default value of this parameter has been set to 1.7 (the previous behavior would have corresponded to axes_labels_size=1
.).
comment:3 Changed 8 years ago by
Status: | new → needs_review |
---|
comment:4 Changed 8 years ago by
Cc: | vdelecroix added |
---|
comment:5 Changed 8 years ago by
Cc: | kcrisman added |
---|
comment:6 Changed 8 years ago by
Seems fine on the face of it but I truly have nearly zero Sage reviewing time these days. Wait until school lets out and I'll have a mountain of reviews for you, I'm sure :-)
comment:7 follow-up: 9 Changed 8 years ago by
Status: | needs_review → needs_info |
---|
Hello,
If you set
+ self._axes_labels_size = 1.7
in the class Graphics, why would you need this try/except
block
+ try: + return self._axes_labels_size + except AttributeError: + self._axes_labels_size = 1.7 + return self._axes_labels_size
Vincent
comment:8 Changed 8 years ago by
Reviewers: | → Vincent Delecroix |
---|
comment:9 follow-up: 10 Changed 8 years ago by
Hi Vincent,
You are perfectly right! I am going to change this.
Btw, the same occurs for the attribute _fontsize
(probably I did a copy-and-paste of the latter...): there is
self._fontsize = 10
in Graphics.__init__
and
try: return self._fontsize except AttributeError: self._fontsize = 10 return self._fontsize
in Graphics.fontsize
.
Shall we correct it as well, although this does not pertain to the current ticket ?
Replying to vdelecroix:
Hello,
If you set
+ self._axes_labels_size = 1.7in the class Graphics, why would you need this
try/except
block+ try: + return self._axes_labels_size + except AttributeError: + self._axes_labels_size = 1.7 + return self._axes_labels_sizeVincent
comment:10 follow-up: 11 Changed 8 years ago by
Replying to egourgoulhon:
Hi Vincent,
You are perfectly right! I am going to change this. Btw, the same occurs for the attribute
_fontsize
(probably I did a copy-and-paste of the latter...): there isself._fontsize = 10in
Graphics.__init__
andtry: return self._fontsize except AttributeError: self._fontsize = 10 return self._fontsizein
Graphics.fontsize
. Shall we correct it as well, although this does not pertain to the current ticket ?
Sure. Do it at the same time.
Vincent
comment:11 follow-up: 12 Changed 8 years ago by
Replying to vdelecroix:
Shall we correct it as well, although this does not pertain to the current ticket ?
Sure. Do it at the same time.
A systematic search for AttributeError
in graphics.py reveals that the same issue occurs for the attributes
_show_axes
, _axes_color
, _axes_label_color
, _axes_width
and _tick_label_colors
.
So I guess we shall correct these as well?... The only possible justification is that the current implementation is more robust in case of a derived class of Graphics
, A
let us say, such that A.__init__()
does not call Graphics.__init__()
(but this is bad design IMHO).
Eric.
comment:12 follow-up: 13 Changed 8 years ago by
Replying to egourgoulhon:
Replying to vdelecroix:
Shall we correct it as well, although this does not pertain to the current ticket ?
Sure. Do it at the same time.
A systematic search for
AttributeError
in graphics.py reveals that the same issue occurs for the attributes_show_axes
,_axes_color
,_axes_label_color
,_axes_width
and_tick_label_colors
. So I guess we shall correct these as well?... The only possible justification is that the current implementation is more robust in case of a derived class ofGraphics
,A
let us say, such thatA.__init__()
does not callGraphics.__init__()
(but this is bad design IMHO).
Right. The simplest would be to let it as you initially did... sorry.
Vincent
comment:13 follow-up: 14 Changed 8 years ago by
Replying to vdelecroix:
Replying to egourgoulhon:
Replying to vdelecroix:
Shall we correct it as well, although this does not pertain to the current ticket ?
Sure. Do it at the same time.
A systematic search for
AttributeError
in graphics.py reveals that the same issue occurs for the attributes_show_axes
,_axes_color
,_axes_label_color
,_axes_width
and_tick_label_colors
. So I guess we shall correct these as well?... The only possible justification is that the current implementation is more robust in case of a derived class ofGraphics
,A
let us say, such thatA.__init__()
does not callGraphics.__init__()
(but this is bad design IMHO).Right. The simplest would be to let it as you initially did... sorry.
OK.
Btw note that the case of _axes_width
looks strange: in Graphics.__init__()
, one has
self._axes_width = 0.8
while in Graphics.axes_width()
there is
try: return self._axes_width except AttributeError: self._axes_width = True return self._axes_width
Shouldn't it be there self._axes_width = 0.8
for consistency ?
Eric.
Vincent
comment:14 Changed 8 years ago by
Replying to egourgoulhon:
Shouldn't it be there
self._axes_width = 0.8
for consistency ?
True. But I suggest to not do it here and open a new ticket and get rid of all these try/except.
Vincent
comment:15 Changed 8 years ago by
Milestone: | sage-6.6 → sage-6.7 |
---|
Hi,
Another point that could be discussed is the default value of the new parameter axes_labels_size
: I've set it to 1.7 for it looks good for small labels like $x$ and $y$, but maybe it can be lowered a little bit, to 1.6 or 1.5 say, to accomodate for larger labels. Of course, this can be adjusted by the user (the whole point of this ticket!), but it's always nice if the default value fits most usages.
comment:16 Changed 8 years ago by
Status: | needs_info → needs_review |
---|
comment:17 follow-up: 18 Changed 8 years ago by
Hello Eric,
You added two trailing whitespaces! This is very bad.
The problem with the right size is that it depends if the label is in math mode (like $x$
) or not (like x
). I would say that:
- 1.4 is a good default for non math mode
- 1.7 is a good default for math mode
I just tried
plot(x^2, (x,0,1), axes_labels=('x','$x$'), axes_labels_size=1.7)
There is a big difference in the aspects of both letters!
What do people use, label='x'
or label='$x$'
?
Vincent
comment:18 follow-up: 19 Changed 8 years ago by
Hi Vincent,
Replying to vdelecroix:
You added two trailing whitespaces! This is very bad.
Sorry about this. Yet I've configured my editor to automatically removing trailing whitespaces while saving a file. Apparently two of them went through...
The problem with the right size is that it depends if the label is in math mode (like
$x$
) or not (likex
). I would say that:
- 1.4 is a good default for non math mode
- 1.7 is a good default for math mode
I just tried
plot(x^2, (x,0,1), axes_labels=('x','$x$'), axes_labels_size=1.7)There is a big difference in the aspects of both letters!
You are right!
What do people use,
label='x'
orlabel='$x$'
?
Probably it depends, but if you want to have nice labels (in particular labels that coincide with the symbols used in the main text of your article or labels with subscripts or Greek letters), LaTeX seems mandatory. So I would favor the math mode. Maybe lowered to 1.6 as a compromise?... Eric.
comment:19 follow-up: 21 Changed 8 years ago by
Replying to egourgoulhon:
Replying to vdelecroix: So I would favor the math mode. Maybe lowered to 1.6 as a compromise?...
Looks reasonable to me.
Vincent
comment:20 Changed 8 years ago by
Commit: | 10b69240dfceaec610564bb26e2ab7eb1a4a5b45 → 69354fcb35c701ea3aa4311381acc4f8319cf85a |
---|
Branch pushed to git repo; I updated commit sha1. New commits:
69354fc | Set the default value of axes_labels_size to 1.6
|
comment:21 Changed 8 years ago by
Replying to vdelecroix:
Replying to egourgoulhon:
Replying to vdelecroix: So I would favor the math mode. Maybe lowered to 1.6 as a compromise?...
Looks reasonable to me.
All right; in the new commit, I've set the default value to 1.6.
Eric.
comment:22 Changed 8 years ago by
Status: | needs_review → positive_review |
---|
Good to go then. Thanks.
Vincent
comment:24 Changed 8 years ago by
Branch: | public/18004 → 69354fcb35c701ea3aa4311381acc4f8319cf85a |
---|---|
Resolution: | → fixed |
Status: | positive_review → closed |
comment:25 Changed 8 years ago by
Commit: | 69354fcb35c701ea3aa4311381acc4f8319cf85a |
---|
See #18421 for followup.
New commits:
Add parameter axes_labels_size to Graphics.