Skip to content

Fix winsorization constraints for count_distinct in eppo-schema docs#726

Open
bertilhatt wants to merge 1 commit into
mainfrom
docs/fix-count-distinct-winsorization-constraint
Open

Fix winsorization constraints for count_distinct in eppo-schema docs#726
bertilhatt wants to merge 1 commit into
mainfrom
docs/fix-count-distinct-winsorization-constraint

Conversation

@bertilhatt

@bertilhatt bertilhatt commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • A customer reported that our eppo-schema docs incorrectly state that count_distinct metrics cannot use winsorization.
  • Product behavior and eppo-metrics-sync both support winsorization on count_distinct.
  • Updated the aggregation table, winsorization constraints section, and invalid-config example to match.

Test plan

  • Confirm the Winsorization Constraints section lists count_distinct as supported
  • Confirm the invalid example no longer claims count_distinct + winsorization is an error
  • Cross-check against eppo-metrics-sync README constraints

The certified metrics schema incorrectly listed count_distinct as unsupported
for winsorization, contradicting product behavior and eppo-metrics-sync.

Co-authored-by: Cursor <cursoragent@cursor.com>
@netlify

netlify Bot commented Jun 5, 2026

Copy link
Copy Markdown

Deploy Preview for eppo-data-docs ready!

Name Link
🔨 Latest commit 3135d9d
🔍 Latest deploy log https://app.netlify.com/projects/eppo-data-docs/deploys/6a22b09ddb092f0009f97851
😎 Deploy Preview https://deploy-preview-726--eppo-data-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@bertilhatt bertilhatt requested a review from emielver June 5, 2026 11:30
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.

1 participant