Sphinx improvements

Makefile

@@ -0,0 +1,20 @@
+POETRY     = poetry run
+SOURCEDIR  = docs/source
+BUILDDIR   = docs/build
+PACKAGEDIR = client
+
+.PHONY: docs docs-clean docs-live
+
+# Build documentation
+docs:
+	$(POETRY) sphinx-build -a "$(SOURCEDIR)" "$(BUILDDIR)/html"
+
+# Remove documentation outputs
+# This includes the api dir as it's autogenerated with sphinx-build
+docs-clean:
+	rm -r "$(BUILDDIR)" "$(SOURCEDIR)/api"
+
+# Spin up a local server to serve documentation pages
+# Auto-reloads when code changes in PACKAGEDIR are made
+docs-live:
+	$(POETRY) sphinx-autobuild --open-browser --watch "$(PACKAGEDIR)" -a "$(SOURCEDIR)" "$(BUILDDIR)/html"

README.md

@@ -86,15 +86,13 @@ poetry run black client/models/pose_detection/classification.py
 
 ## Documentation
 
-To build the docs locally, run from the root of the repo:
-
+We use [Sphinx](https://www.sphinx-doc.org/) for documentation. To view the documentation locally, run the following command:
 ```bash
-poetry run sphinx-apidoc -f -o docs/source/generated client &&
-cd docs &&
-poetry run make html
+make docs-live
 ```
+This spins up a local server which serves the documentation pages, and also hot-reloads and auto-rebuilds whenever code changes are made.
 
-The documentation is generated automatically from the source code, so it isn't stored in the repo (hence why we need to run `sphinx-apidoc`).
+You can build the documentation (without spinning up a server) with `make docs`, and clean the documentation output with `make docs-clean`.
 
 ## Downloading ML Models
 From top-level directory.

client/models/pose_detection/camera.py

@@ -11,7 +11,14 @@
 
 def is_camera_aligned(pose_landmark_result: PoseLandmarkerResult) -> np.bool_:
     """
-    Returns whether the camera is aligned to capture the person's side view.
+    Checks whether the camera is aligned to capture the person's side view.
+
+    Args:
+        pose_landmarker_result: Landmarker result as returned by a
+          mediapipe.tasks.vision.PoseLandmarker
+
+    Returns:
+        True if the camera is aligned, False otherwise
     """
     landmarks: list[list[Landmark]] = pose_landmark_result.pose_world_landmarks
 

client/models/pose_detection/classification.py

@@ -21,10 +21,15 @@ def posture_angle(p1: Landmark, p2: Landmark) -> np.float64:
     The y coordinate of P3 is irrelevant but for simplicity we set it to zero.
 
     For a neck inclination calculation, take P1 to be the shoulder location and pivot
-    point, and P2 to be the ear location.
+    point, and P2 to be the ear location. For a torso inclination calculation, take P1
+    to be the hip location and pivot point, and P2 to be the hip location.
 
-    For a torso inclination calculation, take P1 to be the hip location and pivot
-    point, and P2 to be the hip location.
+    Parameters:
+        p1: Landmark for P1 as described above
+        p2: Landmark for P2 as described above
+
+    Returns:
+        Angle (in degrees) between P2 and P3
     """
     x1, y1 = p1.x, p1.y
     x2, y2 = p2.x, p2.y
@@ -34,12 +39,16 @@ def posture_angle(p1: Landmark, p2: Landmark) -> np.float64:
 
 def posture_classify(pose_landmark_result: PoseLandmarkerResult) -> np.bool_:
     """
-    Returns whether the pose in the image has good (True) or bad (False) posture.
+    Returns whether the pose in the image has good or bad posture.
 
     Note: The camera should be aligned to capture the person's side view; the output
     may not be accurate otherwise. See `is_camera_aligned()`.
 
     REF: https://learnopencv.com/building-a-body-posture-analysis-system-using-mediapipe
+
+    Parameters:
+        pose_landmarker_result: Landmarker result as returned by a
+          mediapipe.tasks.vision.PoseLandmarker
     """
     landmarks: list[list[Landmark]] = pose_landmark_result.pose_world_landmarks
 

client/models/pose_detection/landmarking.py

@@ -14,8 +14,9 @@
 
 @dataclass
 class AnnotatedImage:
-    """Represents mutable annoted image through data attribute. Can be used to set annotated image
-    within a callback asynchronously without raising an error.
+    """
+    Represents mutable annoted image through data attribute. Can be used to set
+    annotated image within a callback asynchronously without raising an error.
     """
 
     data: Optional[np.ndarray] = None
@@ -26,11 +27,11 @@ def draw_landmarks_on_image(
 ) -> np.ndarray:
     """
     Args:
-        bgr_image (np.ndarray): CxWxH image array where channels are sorted in BGR.
-        detection_result (PoseLandmarkerResult): Landmarker result as returned by a
-            mediapipe.tasks.vision.PoseLandmarker
+        bgr_image: CxWxH image array where channels are sorted in BGR.
+        detection_result: Landmarker result as returned by a
+            mediapipe.tasks.vision.PoseLandmarker.
     Returns:
-        (np.ndarray): Image with landmarks annotated.
+        Image with landmarks annotated.
     """
     pose_landmarks_list = detection_result.pose_landmarks
     annotated_image = np.copy(bgr_image)
@@ -58,17 +59,16 @@ def display_landmarking(
     result: PoseLandmarkerResult,
     output_image: mp.Image,
     timestamp: int,
-    annotated_image=AnnotatedImage,
+    annotated_image: AnnotatedImage,
 ) -> None:
     """Mutates annotated image to contain visualization of detected landmarks.
     Also prints debugging info to the standard output.
 
     Args:
-        result (PoseLandmarkerResult):  Landmarker result as returned by a
-            mediapipe.tasks.vision.PoseLandmarker
-        output_image (mp.Image): Raw image used for landmarking.
-        timestamp (int): Video timestamp in milliseconds.
-        annotated_image (AnnotatedImage): Image to mutate.
+        result: Landmarker result as returned by a mediapipe.tasks.vision.PoseLandmarker
+        output_image: Raw image used for landmarking.
+        timestamp: Video timestamp in milliseconds.
+        annotated_image: Image to mutate.
     """
     annotated_image.data = draw_landmarks_on_image(output_image.numpy_view(), result)
     print(

client/models/pose_detection/routines.py

@@ -38,8 +38,9 @@ def __init__(
         self.video_capture = cv2.VideoCapture(0)
 
     def track_posture(self) -> None:
-        """Get frame from video capture device and process with pose model, then posture algorithm.
-        Print debugging info and display landmark annotated frame.
+        """
+        Get frame from video capture device and process with pose model, then posture
+        algorithm. Print debugging info and display landmark annotated frame.
         """
         success, frame = self.video_capture.read()
         if not success:
@@ -60,9 +61,11 @@ def __exit__(self, unused_exc_type, unused_exc_value, unused_traceback) -> None:
 
 
 def create_debug_posture_tracker() -> DebugPostureTracker:
-    """Handles config of livestreamed input and model loading.
+    """
+    Handles config of livestreamed input and model loading.
+
     Returns:
-        (DebugPostureTracker): Tracker object which acts as context manager.
+        Tracker object which acts as context manager.
     """
     annotated_image = AnnotatedImage()
     options = PoseLandmarkerOptions(

docs/source/conf.py

@@ -3,6 +3,9 @@
 # For the full list of built-in configuration values, see the documentation:
 # https://www.sphinx-doc.org/en/master/usage/configuration.html
 
+import os
+import sys
+
 # -- Project information -----------------------------------------------------
 # https://www.sphinx-doc.org/en/master/usage/configuration.html#project-information
 
@@ -15,20 +18,27 @@
 # -- General configuration ---------------------------------------------------
 # https://www.sphinx-doc.org/en/master/usage/configuration.html#general-configuration
 
+
+sys.path.insert(0, os.path.abspath("../.."))
+
 extensions = [
     "sphinx.ext.duration",
-    "sphinx.ext.autodoc",
-    "sphinx.ext.autosummary",
     "sphinx.ext.napoleon",
+    "sphinxcontrib.apidoc",
 ]
 
 templates_path = ["_templates"]
 exclude_patterns = []
 
+apidoc_module_dir = "../../client"
+apidoc_output_dir = "api"
+
+napoleon_google_docstring = True
+napoleon_numpy_docstring = False
 
 # -- Options for HTML output -------------------------------------------------
 # https://www.sphinx-doc.org/en/master/usage/configuration.html#options-for-html-output
 
 
-html_theme = "alabaster"
+html_theme = "piccolo_theme"
 html_static_path = ["_static"]

docs/source/index.rst

@@ -17,4 +17,4 @@ Contents
 
 .. toctree::
 
-   generated/modules
+   api/modules