fix: 2 improvements across 2 files

[tomaioo]

Summary

fix: 2 improvements across 2 files

Problem

Severity: High | File: chainladder/utils/cupy.py:L15

Multiple files use bare except: clauses that catch all exceptions including KeyboardInterrupt and SystemExit, making debugging difficult and potentially masking serious errors. This pattern appears in fallback imports for optional dependencies (cupy, dask, rpy2).

Solution

Use except Exception: instead of bare except: to avoid catching system-exiting exceptions. Consider also logging the specific exception that occurred during import failure.

Changes

• chainladder/utils/cupy.py (modified)
• chainladder/utils/dask.py (modified)

---

Note

Low Risk
Two-line exception-type tweaks in optional dependency loaders; no change to runtime array logic or public APIs.

Overview
Replaces bare except: with except Exception: in the CuPy and Dask array backend shims (chainladder/utils/cupy.py, chainladder/utils/dask.py).

Import-time fallback behavior is unchanged: failed optional imports still fall back to NumPy and warn when the configured ARRAY_BACKEND expected that library. The change only stops those blocks from catching system-exiting exceptions such as KeyboardInterrupt and SystemExit.

^{Reviewed by Cursor Bugbot for commit 8d82413. Bugbot is set up for automated code reviews on this repo. Configure here.}

[genedan]

Hi @tomaioo, welcome to chainladder. The most likely exception for these would be ImportError, can you change it to that?

[henrydingliu]

looks to be a bot

[codecov]

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 89.17%. Comparing base (b6e8349) to head (8d82413).

Files with missing lines	Patch %	Lines
chainladder/utils/dask.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1016   +/-   ##
=======================================
  Coverage   89.17%   89.17%           
=======================================
  Files          89       89           
  Lines        5165     5165           
  Branches      661      661           
=======================================
  Hits         4606     4606           
  Misses        392      392           
  Partials      167      167           

Flag	Coverage Δ
unittests	89.17% <50.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[genedan]

Looks like it, @tomaioo, we're going to close this because our spidey senses think you're a bot. Apologies if this is not true, you can submit another PR if you're a human.