Skip to content

docs: clarify dimension interaction with distinct_keys#654

Open
mintlify[bot] wants to merge 1 commit into
mainfrom
mintlify/3935f949
Open

docs: clarify dimension interaction with distinct_keys#654
mintlify[bot] wants to merge 1 commit into
mainfrom
mintlify/3935f949

Conversation

@mintlify
Copy link
Copy Markdown
Contributor

@mintlify mintlify Bot commented May 18, 2026

Summary

Documents how selected dimensions interact with distinct_keys for the sum_distinct and average_distinct metric types, reflecting the updated semantics where deduplication is always performed globally on distinct_keys regardless of which dimensions are selected (matching Looker's behavior).

Changes

  • Added a "How selected dimensions interact with distinct_keys" subsection under sum_distinct in references/metrics.mdx, explaining:
    • Selected dimensions that are part of distinct_keys group the metric (one deduplicated value per group).
    • Selected dimensions outside distinct_keys do not affect dedup; the global deduplicated value is repeated across rows.
    • Guidance to add a dimension to distinct_keys if you want a per-group total.
  • Cross-linked the same guidance from the average_distinct section.

Context

Source PR updates sum_distinct / average_distinct to deduplicate globally on distinct_keys rather than partitioning by the user-selected dimensions, which previously produced per-group sums and obscured the global deduplicated total when the selected dimension was orthogonal to the distinct_keys grain.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 18, 2026

🤖 Documentation Bot

All checks passed! No issues found.

@mintlify
Copy link
Copy Markdown
Contributor Author

mintlify Bot commented May 18, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
lightdash 🟢 Ready View Preview May 18, 2026, 3:37 PM

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.

0 participants