Skip to content

[WIP] DOC Updated documentation of arrow function to numpy docs format. #7029

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

Merged
merged 6 commits into from
Sep 24, 2016

Conversation

mobando
Copy link
Contributor

@mobando mobando commented Sep 5, 2016

Hi @NelleV,
we tried to update the doc of arrow().
Bye,
Miguel and Bo

@NelleV

dy : float
Length of arrow in y-coordinate

Return
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a small mistake in the title of this section. It should be Returns (with an s)

@NelleV
Copy link
Member

NelleV commented Sep 5, 2016

Apart from my two comments, this is great!
Thanks,
N


**Example:**
Examples
--------

.. plot:: mpl_examples/pylab_examples/arrow_demo.py
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please add a blank line after this? I am not entirely sure why, but python3.5 is unhappy…

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I wasn't clear: I meant after the ".. plot::" directive.

@NelleV
Copy link
Member

NelleV commented Sep 5, 2016

The rendering is now fixed and it looks great!

@QuLogic
Copy link
Member

QuLogic commented Sep 6, 2016

It appears you had not set the config for your git client initially. Your first commit is attributed to "Your Name you@example.com", which is probably not what you want.

y : float
Y-coordinate of the arrow base
dx : float
Length of arrow in x-coordinate
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"in" -> "along"

@NelleV NelleV changed the title DOC Updated documentation of arrow function to numpy docs format. [WIP] DOC Updated documentation of arrow function to numpy docs format. Sep 14, 2016
@NelleV
Copy link
Member

NelleV commented Sep 19, 2016

Hi Miguel and Bo,

This patch still needs a bit of work. Do you want to finish it or should I take over?

Cheers,
N

@tacaswell tacaswell added this to the 2.0.1 (next bug fix release) milestone Sep 20, 2016
@mobando
Copy link
Contributor Author

mobando commented Sep 21, 2016

I repaired the wording (in > along) and the whitespace as per comments, but I am at a loss about the FancyArrow properties issue. I would appreciate suggestions here.


Draws arrow on specified axis from (*x*, *y*) to (*x* + *dx*,
*y* + *dy*). Uses FancyArrow patch to construct the arrow.
%(FancyArrow)s : Parent object type.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not put anything after %(FancyArrow)s, that code get templated at import time to be a big tables.

@QuLogic
Copy link
Member

QuLogic commented Sep 22, 2016

CI's not done, but I expect it will pass as the doc build is already done. The first commit still needs rebasing to give it a unique author (which could be done by one of us if given the name/email to use), but modulo that it should be ready.

@mobando
Copy link
Contributor Author

mobando commented Sep 22, 2016

So now the commits have been rebased to have a unique author and Travis passed, but one of the appveyor builds failed. Can anyone make sense of the .tex error messages?

There seems to be a small difference in the generated images of the test_backend_pgf, but I do not see how that relates to the documentation changes.

@Kojoley
Copy link
Member

Kojoley commented Sep 22, 2016

There seems to be a small difference in the generated images of the test_backend_pgf, but I do not see how that relates to the documentation changes.

The fail in pgf_pdflatex is not related. (it is failing for all build for a while now #7160)

@tacaswell tacaswell merged commit 8ff57bb into matplotlib:master Sep 24, 2016
@tacaswell
Copy link
Member

Thanks everyone!

tacaswell added a commit that referenced this pull request Sep 24, 2016
DOC: Updated documentation of arrow function to numpy docs format.
@tacaswell
Copy link
Member

backported to v2.x as 0370181

@QuLogic QuLogic modified the milestones: 2.0.1 (next bug fix release), 2.0 (style change major release) Dec 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants