adaptive-machine-learning / CapyMOA / 34086228048

81%

Build Type

push

github

Committed by tachyonicClock

Commit Message

fix: resolve remaining ruff 0.16 findings and lint-driven regressions

Handles the rule categories that had no safe autofix, following up on
the mechanical formatting commit:

- RUF012: convert mutable class-level constants (dataset mean/std/
  shape, drift detector TEST_TYPES) to tuples instead of annotating
  with ClassVar.
- B008: replace mutable/stateful default arguments (ADWIN(),
  torch.Generator(), DecisionTree*(), get_download_dir() which reads
  an env var) with None + lazy construction. Several of these were
  latent bugs: a shared ADWIN/Generator instance was reused across
  every learner/buffer instance created with the default.
- TRY004: raise TypeError instead of ValueError at isinstance-based
  type-check sites, matching Python convention. Updated the one test
  that asserted the old exception type.
- BLE001: inline `# noqa: BLE001` with a justification at each of the
  9 deliberate catch-all sites (benchmark machine-info collection,
  Sphinx doc-build introspection, AutoML candidate-skip logic).
- DTZ005: project-wide ignore in pyproject.toml -- all call sites stamp
  human-readable local timestamps, not timezone-sensitive logic.
- PLW1510, B026, UP007, PLC0206, B023, SIM102/113/115/117, PYI045,
  PLR1704, PLW1508: fixed individually.

Also fixes two regressions introduced by the prior commit's safe
`ruff check --fix`:
- The isort autofix alphabetized imports in capymoa/core/__init__.py,
  capymoa/stream/__init__.py, and capymoa/classifier/__init__.py,
  which broke their circular-import-avoiding order. Restored the
  correct order with `# isort: skip` and an explanatory comment so
  future autofixes don't reintroduce it.
- The unsafe `--fix` pass deleted `_AnyInstance = TypeVar(...)` in
  core/_instance.py as an "unused private TypeVar", but it's
  re-exported and used across stream/_stream.py and
  stream/_csv_stream.py. Restored it with `# noqa: PYI018`.

Adds .git-blame-ignore-revs pointing at the prior mechanical commit.

Coverage Stats

130 of 277 new or added lines in 61 files covered. (46.93%)

864 existing lines in 70 files now uncovered.

9565 of 11732 relevant lines covered (81.53%)

0.82 hits per line