[DiscordArchive] I wanted to ask if the next suggestion makes sense before implementing it. I'd like to make a slight
[DiscordArchive] I wanted to ask if the next suggestion makes sense before implementing it. I'd like to make a slight
Original source
I wanted to ask if the next suggestion makes sense before implementing it. I'd like to make a slight improvement to our diff measurement.
Currently, we display the following diff measurements:
`Update time diff: 10ms, average: 15ms.`
However, when it comes to measuring performance, just knowing the average doesn't always tell the whole story. Sometimes we've got these spikes or delays that can affect how the system performs overall.
So, I'm thinking we should spice it up a bit and start showing not just the average, but also some other stats like percentiles.
So my suggestion to output something like that:
`Update time diff: 10ms. Last 500 diffs summary: mean - 15ms, 50% - 12ms, 95% - 30ms, 99% - 60ms, max - 70ms.`
This way, we gain a better sense of how performance truly varies and ensure we don't overlook any important details.
Is there a chance to merge this?
rektbyfaith
02-23-2024, 10:35 AM #1Archived author: walkline • Posted: 2024-02-23T10:35:14.151000+00:00
Original source
I wanted to ask if the next suggestion makes sense before implementing it. I'd like to make a slight improvement to our diff measurement.
Currently, we display the following diff measurements:
`Update time diff: 10ms, average: 15ms.`
However, when it comes to measuring performance, just knowing the average doesn't always tell the whole story. Sometimes we've got these spikes or delays that can affect how the system performs overall.
So, I'm thinking we should spice it up a bit and start showing not just the average, but also some other stats like percentiles.
So my suggestion to output something like that:
`Update time diff: 10ms. Last 500 diffs summary: mean - 15ms, 50% - 12ms, 95% - 30ms, 99% - 60ms, max - 70ms.`
This way, we gain a better sense of how performance truly varies and ensure we don't overlook any important details.
Is there a chance to merge this?
Original source
Maybe that should be added under a special config option or something.
rektbyfaith
02-23-2024, 01:16 PM #2Archived author: Natrist • Posted: 2024-02-23T13:16:58.506000+00:00
Original source
Maybe that should be added under a special config option or something.
1 Guest(s)
1 Guest(s)