addressing comments

choldgraf · choldgraf · commit 2b00840a4eef · 2017-05-16T10:21:09.000-07:00
diff --git a/doc/api/toolkits/axes_grid.rst b/doc/api/toolkits/axes_grid.rst
@@ -13,10 +13,11 @@ Axes Grid
 ---------
 .. note::
    
-   There is an older version of the AxesGrid toolkit, simply called ``axes_grid`` instead of
-   ``axes_grid1``.  the toolkit had a single namespace of axes_grid. This toolkit was broken
-   into the two modules below. For the documentation on the original ``axes_grid``
-   module, see the `previous version of the docs 
+   There is an older version of the AxesGrid toolkit, ``axes_grid`` (instead of
+   ``axes_grid1``).  The old version had a single namespace for all axes_grid
+   objects, and in the new version this toolkit was broken
+   into the two modules below. For the documentation on ``axes_grid``,
+   see the `previous version of the docs 
    <http://matplotlib.org/2.0.1/mpl_toolkits/axes_grid/index.html#toolkit-axesgrid-index>`_.
 
 .. autosummary::
diff --git a/doc/api/toolkits/mplot3d.rst b/doc/api/toolkits/mplot3d.rst
@@ -33,13 +33,13 @@ mplot3d API
 ===================================
 
 .. note::
-   Historically, axis3d has suffered from having hard-coded constants
-   controlling the look and feel of the 3D plot. This precluded user
-   level adjustments such as label spacing, font colors and panel colors.
-   For version 1.1.0, these constants have been consolidated into a single
-   private member dictionary, `self._axinfo`, for the axis object. This is
-   intended only as a stop-gap measure to allow user-level customization,
-   but it is not intended to be permanent.
+   See :attr:`mpl_toolkits.mplot3d.axis3d._axinfo` for a dictionary containing
+   constants that may be modified for controlling the look and feel
+   of mplot3d axes (e.g., label spacing, font colors and panel colors).
+    Historically, axis3d has suffered from having hard-coded constants
+    that precluded user adjustments, and this dictionary was implemented
+    in version 1.1 as a stop-gap measure.
+
 
 .. autosummary::
    :toctree: ../_as_gen
diff --git a/doc/mpl_toolkits/axisartist/index.rst b/doc/mpl_toolkits/axisartist/index.rst
@@ -3,11 +3,11 @@
 Matplotlib axisartist Toolkit
 =============================
 
-The *axisartist* namespace includes a derived Axes implementation. The
-biggest difference is that the artists responsible to draw axis line,
-ticks, ticklabel and axis labels are separated out from the mpl's Axis
-class, which are much more than artists in the original mpl. This
-change was strongly motivated to support curvilinear grid.
+The *axisartist* namespace includes a derived Axes implementation (
+:class:`mpl_toolkits.axisartist.Axes`). The
+biggest difference is that the artists that are responsible for drawing
+axis lines, ticks, ticklabels, and axis labels are separated out from the
+mpl's Axis class. This change was strongly motivated to support curvilinear grid.
 
 You can find a tutorial describing usage of axisartist at
 :ref:`axisartist_users-guide-index`.
diff --git a/doc/mpl_toolkits/index.rst b/doc/mpl_toolkits/index.rst
@@ -18,8 +18,8 @@ mplot3d
 =======
 
 :mod:`mpl_toolkits.mplot3d` provides some basic 3D
-plotting (scatter, surf, line, mesh) tools.  Not the fastest or feature
-complete 3D library out there, but ships with Matplotlib and thus may be a
+plotting (scatter, surf, line, mesh) tools.  Not the fastest or most feature
+complete 3D library out there, but it ships with Matplotlib and thus may be a
 lighter weight solution for some use cases. Check out the
 :ref:`mplot3d tutorial <sphx_glr_tutorials_toolkits_mplot3d.py>` for more
 information.
@@ -47,7 +47,7 @@ axes_grid1
 ==========
 
 The :mod:`mpl_toolkits.axes_grid1` toolkit is a
-collection of helper classes to ease displaying multiple axes in Matplotlib.
+collection of helper classes for displaying multiple axes in Matplotlib.
 
 .. image:: /_static/demo_axes_grid.png
 
@@ -63,7 +63,7 @@ axisartist
 ==========
 
 The :mod:`mpl_toolkits.axisartist` toolkit contains
-a custom Axes class that is meant to support for curvilinear grids.
+a custom Axes class that is meant to support curvilinear grids.
 
 .. toctree::
     :maxdepth: 2
diff --git a/lib/mpl_toolkits/axisartist/axislines.py b/lib/mpl_toolkits/axisartist/axislines.py
@@ -1,10 +1,9 @@
 """
 Axislines includes modified implementation of the Axes class. The
-biggest difference is that the artists responsible to draw axis line,
-ticks, ticklabel and axis labels are separated out from the mpl's Axis
-class, which are much more than artists in the original
-mpl. Originally, this change was motivated to support curvilinear
-grid. Here are a few reasons that I came up with new axes class.
+biggest difference is that the artists responsible for drawing the axis spine,
+ticks, ticklabels and axis labels are separated out from mpl's Axis
+class. Originally, this change was motivated to support curvilinear
+grid. Here are a few reasons that I came up with a new axes class:
 
 
  * "top" and "bottom" x-axis (or "left" and "right" y-axis) can have
diff --git a/tutorials/toolkits/axes_grid.py b/tutorials/toolkits/axes_grid.py
@@ -14,7 +14,7 @@
 (multiple) images with matplotlib.  In matplotlib, the axes location
 (and size) is specified in the normalized figure coordinates, which
 may not be ideal for displaying images that needs to have a given
-aspect ratio.  For example, it helps you to have a colorbar whose
+aspect ratio.  For example, it helps if you have a colorbar whose
 height always matches that of the image.  `ImageGrid`_, `RGB Axes`_ and
 `AxesDivider`_ are helper classes that deals with adjusting the
 location of (multiple) Axes.  They provides a framework to adjust the
@@ -276,7 +276,7 @@
 Example 2. twin
 ~~~~~~~~~~~~~~~
 
-*twin* without a transform argument treat the parasite axes to have a
+*twin* without a transform argument assumes that the parasite axes has the 
 same data transform as the host. This can be useful when you want the
 top(or right)-axis to have different tick-locations, tick-labels, or
 tick-formatter for bottom(or left)-axis. ::
@@ -407,23 +407,24 @@
 AxesDivider
 ===========
 
-The axes_divider module provide helper classes to adjust the axes
-positions of set of images in the drawing time.
+The axes_divider module provides helper classes to adjust the axes
+positions of a set of images at drawing time.
 
-* :mod:`~mpl_toolkits.axes_grid1.axes_size` provides a classes of
-  units that the size of each axes will be determined. For example,
-  you can specify a fixed size
+* :mod:`~mpl_toolkits.axes_grid1.axes_size` provides a class of
+  units that are used to determine the size of each axes. For example,
+  you can specify a fixed size.
 
-* :class:`~mpl_toolkits.axes_grid1.axes_size.Divider` this is the class
-  that is used calculates the axes position. It divides the given
-  rectangular area into several areas. You initialize the divider by
-  setting the horizontal and vertical list of sizes that the division
-  will be based on. You then use the new_locator method, whose return
-  value is a callable object that can be used to set the axes_locator
-  of the axes.
+* :class:`~mpl_toolkits.axes_grid1.axes_size.Divider` is the class
+  that calculates the axes position. It divides the given
+  rectangular area into several areas. The divider is initialized by
+  setting the lists of horizontal and vertical sizes on which the division
+  will be based. Then use
+  :meth:`~mpl_toolkits.axes_grid1.axes_size.Divider.new_locator`,
+  which returns a callable object that can be used to set the
+  axes_locator of the axes.
 
 
-You first initialize the divider by specifying its grids, i.e.,
+First, initialize the divider by specifying its grids, i.e.,
 horizontal and vertical.
 
 for example,::
@@ -454,37 +455,37 @@
 * v2 => 3, 0
 
 The height of the bottom row is always 2 (axes_divider internally
-assumes that the unit is inch). The first and the second rows with
-height ratio of 2:3. For example, if the total height of the grid 6,
+assumes that the unit is inches). The first and the second rows have a
+height ratio of 2:3. For example, if the total height of the grid is 6,
 then the first and second row will each occupy 2/(2+3) and 3/(2+3) of
-(6-1) inches. The widths of columns (horiz) will be similarly
-determined. When aspect ratio is set, the total height (or width) will
+(6-1) inches. The widths of the horizontal columns will be similarly
+determined. When the aspect ratio is set, the total height (or width) will
 be adjusted accordingly.
 
 
 The :mod:`mpl_toolkits.axes_grid1.axes_size` contains several classes
 that can be used to set the horizontal and vertical configurations. For
-example, for the vertical configuration above will be::
+example, for vertical configuration one could use::
 
   from mpl_toolkits.axes_grid1.axes_size import Fixed, Scaled
   vert = [Fixed(2), Scaled(2), Scaled(3)]
 
 After you set up the divider object, then you create a locator
-instance which will be given to the axes.::
+instance that will be given to the axes object.::
 
      locator = divider.new_locator(nx=0, ny=1)
      ax.set_axes_locator(locator)
 
-The return value of the new_locator method is a instance of the
+The return value of the new_locator method is an instance of the
 AxesLocator class. It is a callable object that returns the
 location and size of the cell at the first column and the second row.
 You may create a locator that spans over multiple cells.::
 
      locator = divider.new_locator(nx=0, nx=2, ny=1)
 
 The above locator, when called, will return the position and size of
-the cells spanning the first and second column and the first row. You
-may consider it as [0:2, 1].
+the cells spanning the first and second column and the first row. In
+this example, it will return [0:2, 1].
 
 See the example,
 
@@ -495,8 +496,8 @@
 
    Simple Axes Divider2
 
-You can adjust the size of the each axes according to their x or y
-data limits (AxesX and AxesY), similar to the axes aspect parameter.
+You can adjust the size of each axes according to its x or y
+data limits (AxesX and AxesY).
 
 .. figure:: ../../gallery/axes_grid1/images/sphx_glr_simple_axes_divider3_001.png
    :target: ../../gallery/axes_grid1/simple_axes_divider3.html
diff --git a/tutorials/toolkits/axisartist.py b/tutorials/toolkits/axisartist.py
@@ -10,11 +10,11 @@
    (derived from the mpl's original Axes class).
    As a side effect, some commands (mostly tick-related) do not work.
 
-The *axisartist* contains custom Axes class that is meant to support for
+The *axisartist* contains a custom Axes class that is meant to support
 curvilinear grids (e.g., the world coordinate system in astronomy).
 Unlike mpl's original Axes class which uses Axes.xaxis and Axes.yaxis
-to draw ticks, ticklines and etc., Axes in axisartist uses special
-artist (AxisArtist) which can handle tick, ticklines and etc. for
+to draw ticks, ticklines, etc., axisartist uses a special
+artist (AxisArtist) that can handle ticks, ticklines, etc. for
 curved coordinate systems.
 
 .. figure:: ../../gallery/axisartist/images/sphx_glr_demo_floating_axis_001.png
@@ -24,29 +24,29 @@
 
    Demo Floating Axis
 
-Since it uses special artists, some mpl commands that work on
+Since it uses special artists, some Matplotlib commands that work on
 Axes.xaxis and Axes.yaxis may not work.
 
 .. _axisartist_users-guide-index:
 
 axisartist
 ==========
 
-*axisartist* module provides a custom (and very experimental) Axes
-class, where each axis (left, right, top and bottom) have a separate
-associated artist which is responsible to draw axis-line, ticks,
-ticklabels, label.  Also, you can create your own axis, which can pass
+The *axisartist* module provides a custom (and very experimental) Axes
+class, where each axis (left, right, top, and bottom) have a separate
+associated artist which is responsible for drawing the axis-line, ticks,
+ticklabels, and labels.  You can also create your own axis, which can pass
 through a fixed position in the axes coordinate, or a fixed position
 in the data coordinate (i.e., the axis floats around when viewlimit
 changes).
 
 The axes class, by default, has its xaxis and yaxis invisible, and
-has 4 additional artists which are responsible for drawing the 4 axis sides in
-"left","right","bottom" and "top".  They are accessed as
+has 4 additional artists which are responsible for drawing the 4 axis spines in
+"left", "right", "bottom", and "top".  They are accessed as
 ax.axis["left"], ax.axis["right"], and so on, i.e., ax.axis is a
 dictionary that contains artists (note that ax.axis is still a
-callable methods and it behaves as an original Axes.axis method in
-mpl).
+callable method and it behaves as an original Axes.axis method in
+Matplotlib).
 
 To create an axes, ::
 
@@ -60,7 +60,7 @@
   ax = AA.Subplot(fig, 111)
   fig.add_subplot(ax)
 
-For example, you can hide the right, and top axis by ::
+For example, you can hide the right and top spines using::
 
   ax.axis["right"].set_visible(False)
   ax.axis["top"].set_visible(False)
@@ -74,7 +74,7 @@
    Simple Axisline3
 
 
-It is also possible to add an extra axis. For example, you may have an
+It is also possible to add a horizontal axis. For example, you may have an
 horizontal axis at y=0 (in data coordinate). ::
 
     ax.axis["y=0"] = ax.new_floating_axis(nth_coord=0, value=0)
@@ -98,8 +98,8 @@
 axisartist with ParasiteAxes
 ----------------------------
 
-Most commands in the axes_grid1 toolkit can take a axes_class keyword
-argument, and the commands creates an axes of the given class. For example,
+Most commands in the axes_grid1 toolkit can take an axes_class keyword
+argument, and the commands create an axes of the given class. For example,
 to create a host subplot with axisartist.Axes, ::
 
   import mpl_toolkits.axisartist as AA
@@ -108,7 +108,7 @@
   host = host_subplot(111, axes_class=AA.Axes)
 
 
-Here is an example that uses  parasiteAxes.
+Here is an example that uses parasiteAxes.
 
 
 .. figure:: ../../gallery/axisartist/images/sphx_glr_demo_parasite_axes2_001.png
@@ -123,7 +123,7 @@
 Curvilinear Grid
 ----------------
 
-The motivation behind the AxisArtist module is to support curvilinear grid
+The motivation behind the AxisArtist module is to support a curvilinear grid
 and ticks.
 
 .. figure:: ../../gallery/axisartist/images/sphx_glr_demo_curvelinear_grid_001.png
@@ -137,7 +137,7 @@
 Floating Axes
 -------------
 
-This also support a Floating Axes whose outer axis are defined as
+AxisArtist also supports a Floating Axes whose outer axes are defined as
 floating axis.
 
 .. figure:: ../../gallery/axisartist/images/sphx_glr_demo_floating_axes_001.png
