docs: add Triangle and estimator I/O examples (#704) - #1209
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
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 #1209 +/- ##
==========================================
- Coverage 91.26% 91.25% -0.01%
==========================================
Files 91 91
Lines 5401 5400 -1
Branches 691 691
==========================================
- Hits 4929 4928 -1
Misses 338 338
Partials 134 134
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'm good with the docstring changes. do you mind addressing the ruff beef while you are here? |
The ruff workflow clears per-file ignores on touched files, so the grandfathered lambda assignment failed CI. Co-authored-by: Cursor <cursoragent@cursor.com>
|
The ruff workflow runs with Replaced it with an inline dict comprehension and dropped the |
Summary of Changes
ExamplesforTriangle.to_pickle/Triangle.to_jsonand the estimator mixinsto_pickle/to_jsoninchainladder/core/io.py.read_pickle/read_jsonso the write methods are shown in the same pattern as the already-documented readers.Related GitHub Issue(s)
Additional Context for Reviewers
Bite-sized follow-up to the remaining core-module work on #704. Companion PR for slicing: #1208.
Checklist
Made with Cursor