Skip to content

Commit

Permalink
fix: don't allow point savings with min_size = None in CAPA
Browse files Browse the repository at this point in the history
None means fitting is required, and min_size == 1 will never require fitting
  • Loading branch information
Tveten committed Jan 4, 2025
1 parent c89134f commit 2c546b6
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion skchange/anomaly_detectors/capa.py
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,7 @@ def __init__(
self._segment_saving = to_saving(_segment_saving)

_point_saving = L2Saving() if point_saving is None else point_saving
if _point_saving.min_size is not None and _point_saving.min_size > 1:
if _point_saving.min_size != 1:
raise ValueError("Point saving must have a minimum size of 1.")
self._point_saving = to_saving(_point_saving)

Expand Down

0 comments on commit 2c546b6

Please sign in to comment.