Skip to content

Conversation

williamjameshandley
Copy link

Summary

This PR updates docstrings throughout the Nested Sampling and Slice Sampling modules to match BlackJAX's established documentation style.

Changes

  • Module docstrings: Simplified to one-line "Public API for X" format
  • Class docstrings: Shortened attribute descriptions to be more concise
  • Function docstrings: Simplified parameter and return descriptions
  • Consistency: Ensured consistent use of "a" before acronyms (e.g., "a NS kernel")
  • Tone: Made more technical and concise, removed verbose explanations
  • Length: Significantly reduced docstring length while maintaining essential information

Motivation

The NS and SS module docstrings were more verbose than the rest of BlackJAX. This PR brings them in line with the established style used in modules like , , , and .

Testing

No functional changes were made - only documentation updates.

- Simplify module docstrings to one-line 'Public API for X' format
- Shorten class attribute descriptions to be more concise
- Simplify function parameter and return descriptions
- Follow BlackJAX conventions: brief descriptions, minimal explanations
- Ensure consistent terminology (use 'a' before acronyms)
- Add brief docstrings to inner kernel functions
- Match the concise, technical tone of other BlackJAX modules
@yallup
Copy link
Collaborator

yallup commented Sep 11, 2025

Closing as API still in flux, so exercise will likely need to be repeated

@yallup yallup closed this Sep 11, 2025
@yallup yallup deleted the update-ns-docstrings branch September 11, 2025 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants