-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
ENH: Add the ability to have a separate title for each subplot when plotting #14753
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 12 commits
3f133ac
55f4667
8d1b598
ecb9453
d6d1b0c
94ea2d3
eb43f25
5586a96
aa5bb98
2059339
7a293ef
301cc7d
5b88951
c206daf
59ab880
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1217,8 +1217,25 @@ def _adorn_subplots(self): | |
|
||
if self.title: | ||
if self.subplots: | ||
self.fig.suptitle(self.title) | ||
if is_list_like(self.title): | ||
if len(self.title) != self.nseries: | ||
msg = 'The length of `title` must equal the number ' \ | ||
'of columns if using `title` of type `list` ' \ | ||
'and `subplots=True`.\n' \ | ||
'length of title = {}\n' \ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you use parentheses around the full string instead of |
||
'number of columns = {}'.format(len(self.title), | ||
self.nseries) | ||
raise ValueError(msg) | ||
|
||
for (ax, title) in zip(self.axes, self.title): | ||
ax.set_title(title) | ||
else: | ||
self.fig.suptitle(self.title) | ||
else: | ||
if is_list_like(self.title): | ||
msg = 'Using `title` of type `list` is not supported ' \ | ||
'unless `subplots=True` is passed' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same here |
||
raise ValueError(msg) | ||
self.axes[0].set_title(self.title) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. should raise if |
||
|
||
def _apply_axis_properties(self, axis, rot=None, fontsize=None): | ||
|
@@ -2555,8 +2572,10 @@ def _plot(data, x=None, y=None, subplots=False, | |
figsize : a tuple (width, height) in inches | ||
use_index : boolean, default True | ||
Use index as ticks for x axis | ||
title : string | ||
Title to use for the plot | ||
title : string or list | ||
If a string is passed, print the string at the top of the figure. If a | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you leave the starting sentence "Title to use for the plot" ? |
||
list is passed and subplots is True, print each item in the | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. single backticks ( |
||
list above the corresponding subplot. | ||
grid : boolean, default None (matlab style default) | ||
Axis grid lines | ||
legend : False/True/'reverse' | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It'd be nice to report the length of
title
and the expected number of columns in the error message.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done :)