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.

Sign up for GitHub

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

Jump to bottom

Update ratio-metrics.md to reflect use of metrics as inputs #3880

Merged
merged 2 commits into from
Aug 7, 2023
Merged

Update ratio-metrics.md to reflect use of metrics as inputs #3880

merged 2 commits into from
Aug 7, 2023

Conversation

tlento
Copy link
Contributor

@tlento tlento commented Aug 7, 2023

Ratio metrics take metric inputs for the numerator and denominator, rather than measures.

This change makes the relevant updates to the ratio metrics details page.

What are you changing in this pull request and why?

Checklist

  • Review the Content style guide and About versioning so my content adheres to these guidelines.
  • Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

@tlento
Update ratio-metrics.md to reflect use of metrics as inputs
a0c331d 
Ratio metrics take metric inputs for the numerator and denominator, rather than measures.
@tlento tlento requested a review from a team as a code owner August 7, 2023 21:39
@github-actions github-actions bot added the content Improvements or additions to content label Aug 7, 2023
@netlify
Copy link

netlify bot commented Aug 7, 2023 •
edited
Loading

✅ Deploy Preview for docs-getdbt-com ready!

Name Link
🔨 Latest commit f982dfa
🔍 Latest deploy log https://app.netlify.com/sites/docs-getdbt-com/deploys/64d16611170a410008045dc8
😎 Deploy Preview https://deploy-preview-3880--docs-getdbt-com.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 site configuration.

@github-actions github-actions bot added the size: small This change will take 1 to 2 days to address label Aug 7, 2023
@tlento tlento linked an issue Aug 7, 2023 that may be closed by this pull request
Semantic Layer ratio metric documentation is confusing #3877
Closed
1 task
@Jstein77 Jstein77 enabled auto-merge August 7, 2023 21:44
@Jstein77 Jstein77 disabled auto-merge August 7, 2023 21:45
@Jstein77 Jstein77 enabled auto-merge August 7, 2023 21:45
@Jstein77 Jstein77 merged commit 10ee507 into current Aug 7, 2023
10 checks passed
@Jstein77 Jstein77 deleted the tlento-ratio-metrics-detail-fix branch August 7, 2023 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jstein77 Jstein77 Jstein77 approved these changes

Assignees
No one assigned
Labels
content Improvements or additions to content size: small This change will take 1 to 2 days to address
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Semantic Layer ratio metric documentation is confusing
2 participants
@tlento @Jstein77