Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

[docs] Add docs explaining metrics. #3498

Merged
merged 2 commits into from
Mar 8, 2021
Merged

[docs] Add docs explaining metrics. #3498

merged 2 commits into from
Mar 8, 2021

Conversation

stephenroller
Copy link
Contributor

Patch description
Add explanations for a bunch of our metrics, in case someone is trying to figure out what they mean.

Testing steps
Screen Shot 2021-03-08 at 11 01 40 AM

@stephenroller stephenroller requested a review from moyapchen March 8, 2021 16:12
moyapchen
moyapchen approved these changes Mar 8, 2021
View reviewed changes
Copy link
Contributor

@moyapchen moyapchen left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Pretty documentation, lol. :D

Seems possibly worthwhile to get this page to autogenerate, but definitely lower-priority.

@stephenroller stephenroller merged commit 7224486 into master Mar 8, 2021
@stephenroller stephenroller deleted the metrics_names branch March 8, 2021 17:09
@stephenroller stephenroller mentioned this pull request Mar 8, 2021
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@moyapchen moyapchen moyapchen approved these changes

Assignees
No one assigned
Labels
CLA Signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@stephenroller @moyapchen @facebook-github-bot