Improve Ref Docs

.travis.yml

@@ -16,16 +16,17 @@ deploy:
   provider: releases
   api_key: $GITHUB_TOKEN
   file_glob: true
-  file: bundles/*
+  file: $TRAVIS_BUILD_DIR/bundles/*
   skip_cleanup: true
   overwrite: true
   on:
     tags: true
 
 install:
-  - pip install pylint circuitpython-build-tools
+  - pip install pylint circuitpython-build-tools Sphinx sphinx-rtd-theme
 
 script:
   - pylint adafruit_max7219/*.py
   - ([[ ! -d "examples" ]] || pylint --disable=missing-docstring,invalid-name,bad-whitespace examples/*.py)
   - circuitpython-build-bundles --filename_prefix adafruit-circuitpython-max7219 --library_location .
+  - cd docs && sphinx-build -E -W -b html . _build/html

README.rst

@@ -27,9 +27,6 @@ This is easily achieved by downloading
 Usage Example
 =============
 
-Examples
-********
-
 adafruit_max7219.Matrix8x8 Example
 ----------------------------------
 
@@ -87,14 +84,6 @@ adafruit_max7219.BCDDigits Example
       display.show_str(0,'{:9.2f}'.format(-1234.56))
       display.show()
 
-API Reference
-=============
-
-.. toctree::
-   :maxdepth: 2
-
-   api
-
 Contributing
 ============
 
@@ -125,3 +114,26 @@ Then run the build:
 .. code-block:: shell
 
    circuitpython-build-bundles --filename_prefix adafruit-circuitpython-max7219 --library_location .
+
+Sphinx documentation
+-----------------------
+
+Sphinx is used to build the documentation based on rST files and comments in the code. First,
+install dependencies (feel free to reuse the virtual environment from above):
+
+.. code-block:: shell
+
+    python3 -m venv .env
+    source .env/bin/activate
+    pip install Sphinx sphinx-rtd-theme
+
+Now, once you have the virtual environment activated:
+
+.. code-block:: shell
+
+    cd docs
+    sphinx-build -E -W -b html . _build/html
+
+This will output the documentation to ``docs/_build/html``. Open the index.html in your browser to
+view them. It will also (due to -W) error out on any warning like Travis will. This is a good way to
+locally verify it will pass.

adafruit_max7219/bcddigits.py

@@ -22,7 +22,7 @@
 #
 
 """
-:mod:`adafruit_max7219.bcddigits.BCDDigits`
+`adafruit_max7219.bcddigits.BCDDigits`
 ====================================================
 """
 from adafruit_max7219 import max7219
@@ -40,14 +40,12 @@ class BCDDigits(max7219.MAX7219):
     """
     Basic support for display on a 7-Segment BCD display controlled
     by a Max7219 chip using SPI.
+
+    :param object spi: an spi busio or spi bitbangio object
+    :param ~digitalio.DigitalInOut cs: digital in/out to use as chip select signal
+    :param int nDigits: number of led 7-segment digits; default 1; max 8
     """
     def __init__(self, spi, cs, nDigits=1):
-        """
-        :param object spi: an spi busio or spi bitbangio object
-        :param ~digitalio.DigitalInOut cs: digital in/out to use as chip select
-            signal
-        :param int nDigits: number of led 7-segment digits; default 1; max 8
-        """
         self._ndigits = nDigits
         super().__init__(self._ndigits, 8, spi, cs)
 
@@ -66,7 +64,8 @@ def init_display(self):
 
     def set_digit(self, dpos, value):
         """
-        set one digit in the display
+        Set one digit in the display.
+
         :param int dpos: the digit position; zero-based
         :param int value: integer ranging from 0->15
         """
@@ -78,7 +77,8 @@ def set_digit(self, dpos, value):
 
     def set_digits(self, start, values):
         """
-        set the display from a list
+        Set the display from a list.
+
         :param int s: digit to start display zero-based
         :param list ds: list of integer values ranging from 0->15
         """
@@ -89,7 +89,8 @@ def set_digits(self, start, values):
 
     def show_dot(self, dpos, bit_value=None):
         """
-        set the decimal point for a digit
+        Set the decimal point for a digit.
+
         :param int dpos: the digit to set the decimal point zero-based
         :param int value: value > zero lights the decimal point, else unlights the point
         """
@@ -99,15 +100,16 @@ def show_dot(self, dpos, bit_value=None):
 
     def clear_all(self):
         """
-        clear all digits and decimal points
+        Clear all digits and decimal points.
         """
         self.fill(1)
         for i in range(self._ndigits):
             self.show_dot(i)
 
     def show_str(self, start, strg):
         """
-        displays a numeric str in the display.  shows digits 0-9, -, and .
+        Displays a numeric str in the display.  Shows digits ``0-9``, ``-``, and ``.``.
+
         :param int start: start position to show the numeric string
         :param string str: the numeric string
         """
@@ -127,7 +129,8 @@ def show_str(self, start, strg):
 
     def show_help(self, start):
         """
-        display the word HELP in the display
+        Display the word HELP in the display.
+
         :param int start: start position to show HELP
         """
         digits = [12, 11, 13, 14]

adafruit_max7219/matrices.py

@@ -22,7 +22,7 @@
 #
 
 """
-:mod:`adafruit_max7219.matrices.Matrix8x8`
+`adafruit_max7219.matrices.Matrix8x8`
 ====================================================
 """
 from adafruit_max7219 import max7219
@@ -37,12 +37,13 @@
 _DISPLAYTEST = const(15)
 
 class Matrix8x8(max7219.MAX7219):
-    """Driver for a 8x8 LED matrix based on the MAX7219 chip"""
+    """
+    Driver for a 8x8 LED matrix based on the MAX7219 chip.
+
+    :param object spi: an spi busio or spi bitbangio object
+    :param ~digitalio.DigitalInOut cs: digital in/out to use as chip select signal
+    """
     def __init__(self, spi, cs):
-        """
-        :param object spi: an spi busio or spi bitbangio object
-        :param ~digitalio.DigitalInOut cs: digital in/out to use as chip select signal
-        """
         super().__init__(8, 8, spi, cs)
 
     def init_display(self):
@@ -59,7 +60,7 @@ def init_display(self):
 
     def text(self, strg, xpos, ypos, bit_value=1):
         """
-        draw text in the 8x8 matrix.
+        Draw text in the 8x8 matrix.
 
         :param int xpos: x position of LED in matrix
         :param int ypos: y position of LED in matrix
@@ -70,6 +71,6 @@ def text(self, strg, xpos, ypos, bit_value=1):
 
     def clear_all(self):
         """
-        unlights all matrix leds
+        Clears all matrix leds.
         """
         self.fill(0)

adafruit_max7219/max7219.py

@@ -20,30 +20,36 @@
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 # THE SOFTWARE.
 """
-`adafruit_max7219.MAX7219` - MAX7219 LED Matrix/Digit Display Driver
+`adafruit_max7219.max7219` - MAX7219 LED Matrix/Digit Display Driver
 ========================================================================
 CircuitPython library to support MAX7219 LED Matrix/Digit Display Driver.
 This library supports the use of the MAX7219-based display in CircuitPython,
 either an 8x8 matrix or a 8 digit 7-segment numeric display.
 
-see also
-========
+See Also
+=========
 * matrices.Maxtrix8x8 is a class support an 8x8 led matrix display
 * bcddigits.BCDDigits is a class that support the 8 digit 7-segment display
 
 Beware that most CircuitPython compatible hardware are 3.3v logic level! Make
 sure that the input pin is 5v tolerant.
+
 * Author(s): Michael McWethy
 
 Implementation Notes
 --------------------
 **Hardware:**
+
 * Adafruit `MAX7219CNG LED Matrix/Digit Display Driver -
-MAX7219 <https://www.adafruit.com/product/453>`_ (Product ID: 453)
+  MAX7219 <https://www.adafruit.com/product/453>`_ (Product ID: 453)
+
 **Software and Dependencies:**
-* Adafruit CircuitPython firmware (1.0.0+) for the ESP8622 and M0-based boards:
-https://github.com/adafruit/circuitpython/releases
+
+* Adafruit CircuitPython firmware for the ESP8622 and M0-based boards:
+  https://github.com/adafruit/circuitpython/releases
+
 * Adafruit's Bus Device library: https://github.com/adafruit/Adafruit_CircuitPython_BusDevice
+
 **Notes:**
 #.  Datasheet: https://cdn-shop.adafruit.com/datasheets/MAX7219.pdf
 """
@@ -63,18 +69,19 @@
 
 
 class MAX7219:
-    """ MAX2719 - driver for displays based on max719 chip_select"""
+    """
+    MAX2719 - driver for displays based on max719 chip_select
+
+    :param int width: the number of pixels wide
+    :param int height: the number of pixels high
+    :param object spi: an spi busio or spi bitbangio object
+    :param ~digitalio.DigitalInOut chip_select: digital in/out to use as chip select signal
+    :param baudrate: for SPIDevice baudrate (default 8000000)
+    :param polarity: for SPIDevice polarity (default 0)
+    :param phase: for SPIDevice phase (default 0)
+    """
     def __init__(self, width, height, spi, cs, *,
                  baudrate=8000000, polarity=0, phase=0):
-        """
-        :param int width: the number of pixels wide
-        :param int height: the number of pixels high
-        :param object spi: an spi busio or spi bitbangio object
-        :param ~digitalio.DigitalInOut chip_select: digital in/out to use as chip select signal
-        :param baudrate: for SPIDevice baudrate (default 8000000)
-        :param polarity: for SPIDevice polarity (default 0)
-        :param phase: for SPIDevice phase (default 0)
-        """
 
         self._chip_select = cs
         self._chip_select.direction = digitalio.Direction.OUTPUT
@@ -91,12 +98,13 @@ def __init__(self, width, height, spi, cs, *,
         self.init_display()
 
     def init_display(self):
-        """must be implement by derived class"""
+        """Must be implemented by derived class (``matrices``, ``bcddigits``)"""
         pass
 
     def brightness(self, value):
         """
-        control the brightness of the display
+        Controls the brightness of the display.
+
         :param int value: 0->15 dimmest to brightest
         """
         if not 0 <= value <= 15:
@@ -105,21 +113,23 @@ def brightness(self, value):
 
     def show(self):
         """
-        update the display with recent changes in buffer
+        Updates the display with recent changes in buffer.
         """
         for ypos in range(8):
             self.write_cmd(_DIGIT0 + ypos, self._buffer[ypos])
 
     def fill(self, bit_value):
         """
-        set all buffer bits to a col
+        Sets all buffer bits to a col
+
         :param bit_value: int value > 0 set the buffer bit, else clears the buffer bit
         """
         self.framebuf.fill(bit_value)
 
     def pixel(self, xpos, ypos, bit_value=None):
         """
-        set one buffer bit
+        Set one buffer bit
+
         :param xpos: x position to set bit
         :param ypos: y position to set bit
         :param bit_value: int value > 0 sets the buffer bit, else clears the buffer bit
@@ -128,11 +138,11 @@ def pixel(self, xpos, ypos, bit_value=None):
         self.framebuf.pixel(xpos, ypos, bit_value)
 
     def scroll(self, delta_x, delta_y):
-        """scolls the display using delta_x,delta_y"""
+        """Scolls the display using delta_x,delta_y."""
         self.framebuf.scroll(delta_x, delta_y)
 
     def write_cmd(self, cmd, data):
-        """writes a command to spi device"""
+        """Writes a command to spi device."""
         #print('cmd {} data {}'.format(cmd,data))
         self._chip_select.value = False
         with self._spi_device as my_spi_device:

docs/api.rst

@@ -0,0 +1,11 @@
+
+.. If you created a package, create one automodule per module in the package.
+
+.. automodule:: adafruit_max7219.max7219
+   :members:
+
+.. automodule:: adafruit_max7219.matrices
+   :members:
+
+.. automodule:: adafruit_max7219.bcddigits
+   :members: