Skip to content

fix: relax censor_min_count lower bound from 10 to 0 (#182)#188

Open
d-koding wants to merge 2 commits intoepic-open-source:mainfrom
d-koding:fix/182-relax-censor-min-count
Open

fix: relax censor_min_count lower bound from 10 to 0 (#182)#188
d-koding wants to merge 2 commits intoepic-open-source:mainfrom
d-koding:fix/182-relax-censor-min-count

Conversation

@d-koding
Copy link
Copy Markdown

Overview

Closes #182

Description of changes

Relaxes the censor_min_count lower bound from ge=10 to ge=0 in DataUsage.

The previous hard minimum of 10 blocked legitimate workflows where privacy constraints
do not apply, such as synthetic data, debugging, or LLM feedback visualizations
(e.g. Likert plots). The default value remains 10 so no existing behavior is changed.

  • Changed Field(10, ge=10) to Field(10, ge=0) in src/seismometer/configuration/model.py
  • Updated docstring with .. versionchanged:: NEXT_VERSION_PLACEHOLDER directive
  • Added tests covering values below 10 and rejection of negative values
  • Added changelog fragment changelog/182.bugfix.rst

Author Checklist

  • Linting passes; run early with pre-commit hook.
  • Tests added for new code and issue being fixed.
  • Added type annotations and full numpy-style docstrings for new methods.
  • For breaking API changes, added .. versionchanged:: NEXT_VERSION_PLACEHOLDER directive to affected docstrings.
  • Draft your news fragment in new changelog/ISSUE.TYPE.rst files; see changelog/README.md.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 24, 2026

CLA assistant check
All committers have signed the CLA.

@diehlbw diehlbw requested a review from MahmoodEtedadi April 16, 2026 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Relax censor_min_count Constraint to Allow Thresholds Below 10 for Small/Synthetic Datasets

2 participants