-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Several small What's New fixes #9259
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
Conversation
@@ -31,27 +31,27 @@ The polar axes transforms have been greatly re-factored to allow for more | |||
customization of view limits and tick labelling. Additional options for view | |||
limits allow for creating an annulus, a sector, or some combination of the two. | |||
|
|||
The :meth:`~matplotlib.axes.projections.polar.PolarAxes.set_rorigin` method may | |||
The :meth:`~matplotlib.projections.polar.PolarAxes.set_rorigin` method may |
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.
I think you can get away (here and below) with omitting the :meth: because the current default role is :py:obj:.
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.
Quite likely; I was a bit lazy and didn't change the existing ones, but generally didn't use any roles when adding the new links. There might be a slight difference with adding ()
for :func:
and :meth:
, though?
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.
http://www.sphinx-doc.org/en/stable/domains.html#role-py:func
http://www.sphinx-doc.org/en/stable/config.html#confval-add_function_parentheses
Not sure whether it is set in conf.py right now, or whether py:obj respecsts that setting. If it doesn't we can just set add_function_parentheses to False.
Of course it's fine if you decide you don't want to mess with the markup, this is already an improvement...
EDIT: sphinx-doc/sphinx#171 suggests :py:obj: respects add_function_parentheses.
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.
When I tried the two options you gave below, the one with the :meth:
role has parentheses while the one without doesn't.
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.
Probably worth reporting as a Sphinx bug then?
Calling :func:`figure.legend` can now be done with no arguments. In this case a | ||
legend will be created that contains all the artists on all the axes contained | ||
within the figure. | ||
Calling ``Figure.``:func:`~matplotlib.figure.Figure.legend` can now be |
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.
Does
`.Figure.legend`
or
:meth:`.Figure.legend`
work?
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.
Ah, I wasn't sure if we could use the leading dot, but it does work in this case.
I am going to merge this as-is so I can rebase some re-organization I have been doing on top of it. |
Oh, I was about to push corrections... |
Sorry! |
Fix cross-links to other parts of the documentation as well as add links in almost every entry. Change code bits from cross-references to code-styling.