Skip to content

Migrate docstrings from NumPy to Google convention#1140

Merged
hmgaudecker merged 3 commits intomainfrom
migrate-docstrings-google
Mar 19, 2026
Merged

Migrate docstrings from NumPy to Google convention#1140
hmgaudecker merged 3 commits intomainfrom
migrate-docstrings-google

Conversation

@hmgaudecker
Copy link
Copy Markdown
Collaborator

Summary

  • Switch pydocstyle.convention from "numpy" to "google" in pyproject.toml
  • Convert docstrings in src/gettsim/plot/dag/__init__.py (the only gettsim file with NumPy-style sections)
  • Policy functions already had summary-only docstrings — no changes needed there

Closes #1139

Test plan

  • pixi run ty — all checks passed
  • pixi run ty-jax — all checks passed
  • prek run --all-files — all passed
  • pixi run -e py314 tests -n 7 — 2471 passed
  • grep -rn "^ ----------" src/ — no remaining NumPy-style dashes

🤖 Generated with Claude Code

Switch pydocstyle.convention from "numpy" to "google" in pyproject.toml
and convert docstrings in plot/dag/__init__.py.

Closes #1139

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@read-the-docs-community
Copy link
Copy Markdown

read-the-docs-community bot commented Mar 19, 2026

Documentation build overview

📚 gettsim | 🛠️ Build #31877364 | 📁 Comparing f928ccb against latest (539c421)


🔍 Preview build

Show files changed (2 files in total): 📝 2 modified | ➕ 0 added | ➖ 0 deleted
File Status
how_to_guides/modifications_of_policy_environments.html 📝 modified
tutorials/visualizing_the_system.html 📝 modified

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@hmgaudecker
Copy link
Copy Markdown
Collaborator Author

Convert another docstring to google style.

Good catch, thanks!

@hmgaudecker hmgaudecker merged commit 11bdb00 into main Mar 19, 2026
14 checks passed
@hmgaudecker hmgaudecker deleted the migrate-docstrings-google branch March 19, 2026 11:08
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.

Migrate docstrings from NumPy to Google convention

2 participants