FIX: Fix doctest#1091
Conversation
Pyright Type CompletenessView the full Project (full
Other symbols referenced but not exported by
Symbols without documentation:
Patch (exported symbols added or changed by this PR): 23.3% fully typed (7 / 30); 1 no longer exported
Patch symbol details
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1091 +/- ##
==========================================
+ Coverage 90.06% 90.76% +0.69%
==========================================
Files 91 91
Lines 5287 5287
Branches 671 671
==========================================
+ Hits 4762 4799 +37
+ Misses 370 346 -24
+ Partials 155 142 -13
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
I see 252 tests here, I think this is good. |
|
i'm going to use this branch to fix all the failing doctests |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 4b4eb71. Configure here.
* Update common.py * Update bornferg.py * Update clark.py * Update capecod.py * Update benktander.py * Update chainladder.py * Update chainladder.py * Update bornferg.py * Update bornferg.py * Update benktander.py * Update benktander.py * Update clark.py * Update bornferg.py


Summary of Changes
This is just a test (don't intend to merge) to see if the docstrings get picked up. Should be 252 tests instead of 109.
Related GitHub Issue(s)
1089
Additional Context for Reviewers
uv run pytest) and documentation changes (uv run jb build docs --builder=custom --custom-builder=doctest)Note
Medium Risk
The
WeightedRegressionchange tosigma_affects Development/Mack stochastic outputs; new paper-reconciliation tests reduce but do not eliminate regression risk outside tested triangles.Overview
Fixes documentation testing so CI generates Sphinx autosummary stubs before running Jupyter Book doctests, which lets many more docstring examples (including API stubs) execute in CI.
Docstring and doctest cleanup across
Common(pipe,_validate_assumption), IBNR methods,ClarkLDF, andWeightedRegression—mostly expected outputs,testsetup/testoutputformatting, and clearer parameter docs so examples pass reliably.WeightedRegression(origin-forced OLS path) now documentsfit/averageexplicitly and changes howsigma_is derived (sqrt(mse)instead of the prior Mack-style scaling). New Mack (1994/1997) hardcoded regression tests anchorsigma_/ IBNR SE behavior;test_sigma1in development tests was removed.Test coverage additions exercise
Common._validate_assumption,set_backend(including nestedCommon/ldf_), andhas_zeta/cum_zeta_.Reviewed by Cursor Bugbot for commit b45ab45. Bugbot is set up for automated code reviews on this repo. Configure here.