Changed Examples for Pep8 Compliance #7298
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
NelleV
approved these changes
Oct 17, 2016
| onto one single plot. Notice how in this example, par1 and | ||
| par2 are both calling twinx meaning both are tied directly to | ||
| the x-axis. From there, each of those two axis can behave | ||
| seperately from the current x-axis as well as the other |
There was a problem hiding this comment.
I think there is a type here: seperately -> separately.
There was a problem hiding this comment.
I don't quite understand. The previous sentence says that both are tied to the x-axis, but this sentence says they can behave separately.
|
Looks like there is some trailing whitespace left... |
QuLogic
reviewed
Oct 18, 2016
|
|
||
| The following code demonstrates how to put a floating | ||
| polar curve within a rectangular box. In order to get | ||
| a better sense of polar curbes, please look at |
| """ | ||
| Parasite Axis Demo | ||
|
|
||
| The following code is an example of of a parasite axis. |
| onto one single plot. Notice how in this example, par1 and | ||
| par2 are both calling twinx meaning both are tied directly to | ||
| the x-axis. From there, each of those two axis can behave | ||
| seperately from the current x-axis as well as the other |
There was a problem hiding this comment.
I don't quite understand. The previous sentence says that both are tied to the x-axis, but this sentence says they can behave separately.
| @@ -1,30 +1,35 @@ | |||
| """ | |||
| An experimental support for curvilinear grid. | |||
| Axis Within Rectangular Frame | |||
There was a problem hiding this comment.
There was a problem hiding this comment.
First word sounds better.
Sorry @sirlittle : do you mind quickly updating this?
NelleV
approved these changes
Oct 18, 2016
|
Oops, missed the whitespace stuff, fixed in 023caec. |
QuLogic
added a commit
to QuLogic/matplotlib
that referenced
this pull request
Oct 18, 2016
Changed Examples for Pep8 Compliance
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changed demo_parasite_axes2.py and demo_floating_axis.py to add pep8 compliance as well a docstring for the sphinx_gallarey.