docs(friedland): add chapter 9 exhibit IV, V, VI and complete exhibits I-III - #1197
docs(friedland): add chapter 9 exhibit IV, V, VI and complete exhibits I-III#1197priyam0k wants to merge 36 commits into
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): no exported symbol type-completeness changes detected. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1197 +/- ##
=======================================
Coverage 91.31% 91.31%
=======================================
Files 91 91
Lines 5434 5434
Branches 700 700
=======================================
Hits 4962 4962
Misses 336 336
Partials 136 136
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:
|
0d582d1 to
645d24d
Compare
645d24d to
4d26f54
Compare
…xhibit VI steady state
…bot steady state fix
…all scenarios and bases
…e for Scenarios 3 & 4
|
I think the problem is that Friedland's footnote for col 6 and 7 is wrong. The patterns are from volume-weighted average, not simple. |
|
Spot on🔥. I actually ran this test and yes you're totally right. |
|
Chapter 7 is pretty close. Feel free to wait till it merges. then we can go in and persist estimators. |
Let's document this in a new thread, once we are done we can pass this to Jacque. Opened #1215 |
|
FYI Chapter 7 is in. |
|
yep, i'll update ch9 to load the persisted estimators.. |
|
i think there's also scope to add this in ch10 (capecod) too in a follow-up pr and make it more consistent like ch9.. |
… patched CDF in Chapter 9
ebb6ca8 to
ce60441
Compare
…riyam0k/chainladder-python into friedland/chapter-9-exhibit-iv
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit e1d23b3. Configure here.
…ge-48 paid CDF override
|
while working on Chapter 9 Exhibit I, I came across another small rounding nuance and was wondering if this is worth adding to the error tracking list #1215 ? . I can find similar in other exhibits also:
|
|
Column 7 and 8 doesn't actually say they are using the rounded numbers though? Where do you see this? |
|
@priyam0k we are trying to use chainladder constructs to fill out these exhibits. doesn't mean we need to follow the column notes exxactly. i would use |
|
Ya unfortunately when the text is printed we lose all this info. I think if it's a typical usage of round/non-round number, we just document in the recreation work and it won't need to go on #1215. Now, if the numbers aren't tying with either rounded nor non-rounded numbers, then we should let the author know. My point is if we include this specific example, we would be including a bunch of instances and will clog up "real errors". |
Got it @henrydingliu . Will use native chainladder properties
Yes i agree, keeping #1215 issue strictly for real textbook errors and keep rounding notes inline in the notebook. |




Summary of Changes
friedland_us_autoimplicit_axis=Truefor age extraction and clean.to_frame()series calls per feedbackRelated GitHub Issue(s)
#588
Additional Context for Reviewers
Checklist
uv run pytest) and documentation changes (uv run --directory docs jb build . --builder=custom --custom-builder=doctest)Note
Low Risk
Documentation-only notebook changes with numeric reconciliation asserts; no production library or runtime behavior changes.
Overview
Extends
docs/friedland/chapter_9.ipynbso Chapter 9 matches the full Friedland exhibit set: Exhibit IV (U.S. Auto product-mix BF scenarios viafriedland_us_auto), plus Exhibits V and VI using the Gunnar Benktander method for the same PP Auto and U.S. Auto cases (two sheets each).Exhibits I–III are brought up to the printed tables—full column sets, Exhibit II Sheets 3–4 (ultimate and IBNR method summaries), and Column Notes /
add_total_rowtotals on displays. Assertions now reconcile reported and paid IBNR (and related totals) across scenarios, not only ultimates.Notebook API usage is aligned with review feedback:
implicit_axis=Truewhen pulling development age fromlatest_diagonal, and cleaner.to_frame()/ series handling in scenario helpers.Reviewed by Cursor Bugbot for commit 1ff72b5. Bugbot is set up for automated code reviews on this repo. Configure here.