-
-
Notifications
You must be signed in to change notification settings - Fork 125
Open
Description
Is the explanation of the histogram here correct?
website/content/src/content/docs/docs/observability/metrics.mdx
Lines 352 to 363 in d848c89
buckets: [ | |
[ 0, 0 ], // No values in the 0-10 range | |
[ 10, 7 ], // 7 values in the 10-20 range | |
[ 20, 11 ], // 11 values in the 20-30 range | |
[ 30, 20 ], // 20 values in the 30-40 range | |
[ 40, 27 ], // and so on... | |
[ 50, 38 ], | |
[ 60, 53 ], | |
[ 70, 64 ], | |
[ 80, 73 ], | |
[ 90, 84 ], | |
[ Infinity, 100 ] // All 100 values have been recorded |
The initial comments on the buckets seem to contradict the final comment on the amount of values. Given the initial comments say more or less that it's the values in the range the total number of values based on that explanation would be 0+7+11+20+27 [...]
. Based on the end result I assume the initial comments would rather need to be:
buckets: [
[ 0, 0 ], // No values lower then 0
[ 10, 7 ], // 7 values lower equal 10. As there were 0 values in the previous range up to equal 0 there are 7 values between 1 and 10
[ 20, 11 ], // 11 values lower equal 20. As there were 7 values in the previous range up to equal 10 there are 4 values between 11 and 20
[...]
Metadata
Metadata
Assignees
Labels
No labels