[CHORE] Ruff + Test Fixtures for Tail tests - #1374
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): 0.0% fully typed (0 / 1)
Patch symbol details
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1374 +/- ##
==========================================
+ Coverage 91.78% 91.86% +0.08%
==========================================
Files 96 97 +1
Lines 5488 5496 +8
Branches 707 707
==========================================
+ Hits 5037 5049 +12
+ Misses 327 325 -2
+ Partials 124 122 -2
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:
|
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.
Reviewed by Cursor Bugbot for commit 69c96d6. Configure here.

Summary of Changes
ruff fixes
adding tail_sample as a test fixture
adding a test for TailClark to cover application of truncation_age
Related GitHub Issue(s)
#1355
#1361
#1277
AI/LLM Usage
Used linter (not AI)
Additional Context for Reviewers
Submitter's Checklist
[FIX],[FEAT],[DOCS],[TST],[CHORE], or[BRK]).Reviewer's Checklist
Note
Low Risk
Formatting and test-only changes; tail estimator behavior is unchanged aside from clearer concatenate kwargs.
Overview
Applies Ruff-driven cleanup across the
chainladder/tailspackage: module/class/method docstrings are reformatted to the multi-line style, and several calls (notablyxp.concatenateinTailClark) use explicitaxis=-1and expanded line breaks without changing tail logic.Test infrastructure adds a shared
tail_samplepytest fixture (with the samenormal_run/sparse_only_runparametrization as other samples) and refactors Bondy, curve, and constant tail tests to use fixtures plusTYPE_CHECKINGannotations instead of inlineload_sample. A newtest_truncation_ageasserts that a very largetruncation_ageonTailClarkmatches omitting truncation on the final CDF.Reviewed by Cursor Bugbot for commit c7433f0. Bugbot is set up for automated code reviews on this repo. Configure here.