Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cut out any cube result less than 10 by default. #69

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Conversation

mikix
Copy link
Contributor

@mikix mikix commented Jan 8, 2025

With an option (--option min-bucket-size:3) to change it.

Fixes #65

@mikix mikix force-pushed the mikix/min-bucket branch from de3765d to d5577e1 Compare January 8, 2025 16:59
With an option (--option min-bucket-size:3) to change it.
@mikix mikix force-pushed the mikix/min-bucket branch from d5577e1 to 950931e Compare January 8, 2025 17:00
@mikix mikix marked this pull request as ready for review January 8, 2025 17:01
Copy link

@dogversioning dogversioning left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should consider making this a library level cli arg at some point, honestly

@mikix mikix merged commit bdbe8ba into main Jan 8, 2025
2 checks passed
@mikix mikix deleted the mikix/min-bucket branch January 8, 2025 18:33
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.

Allow bucket limits for deid / privacy
2 participants