Skip to content
This repository was archived by the owner on Jan 26, 2026. It is now read-only.

docs: add doctests to core modules#404

Merged
tschm merged 4 commits into
mainfrom
doctests
Jan 26, 2026
Merged

docs: add doctests to core modules#404
tschm merged 4 commits into
mainfrom
doctests

Conversation

@tschm

@tschm tschm commented Jan 26, 2026

Copy link
Copy Markdown
Owner

Add executable examples to docstrings for:

  • Circle class and contains() method
  • perpendicular_slope() function
  • min_circle_cvx() function

Summary by CodeRabbit

  • Documentation
    • Enhanced documentation with practical examples demonstrating usage and expected outputs across the codebase.

✏️ Tip: You can customize this high-level summary in your review settings.

tschm and others added 2 commits January 26, 2026 19:58
Add executable examples to docstrings for:
- Circle class and contains() method
- perpendicular_slope() function
- min_circle_cvx() function

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jan 26, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

Three source files received documentation enhancements through the addition of example sections to existing docstrings. The min_circle_cvx function, Circle class, and perpendicular_slope function now include usage examples demonstrating expected behavior. No functional code modifications were made.

Changes

Cohort / File(s) Summary
Documentation Examples
src/min_circle/cvx.py, src/min_circle/utils/circle.py, src/min_circle/welzl.py
Added Examples sections to function and class docstrings: min_circle_cvx demonstrates solver usage with a 2-point case; Circle class and contains method show instantiation and membership testing; perpendicular_slope illustrates behavior for horizontal and non-horizontal point pairs.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Three docstrings bloomed with examples so bright,
Circle slopes and convex hulls taking flight,
No logic touched, just wisdom displayed,
Documentation's a garden where examples are made! 📚✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding documentation examples (doctests) to core modules. It is concise, specific, and directly relates to the primary objective of the changeset.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@tschm
tschm merged commit 79026ea into main Jan 26, 2026
16 checks passed
@tschm
tschm deleted the doctests branch January 26, 2026 19:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant