bug: On timeseries titles, we display the unit with SI prefix

grafana / explore-profiles

Explore Profiles is a native Grafana application designed to integrate seamlessly with Pyroscope, the open-source continuous profiling platform, providing a smooth, query-less experience for browsing and analyzing profiling data.

GNU Affero General Public License v3.0

17 stars 1 forks source link

bug: On timeseries titles, we display the unit with SI prefix #115

Open simonswine opened 1 month ago

simonswine commented 1 month ago

Reported by @bboreham

We should not use the SI prefix, as the axis scales it to the fitting SI prefix and that's confusing
Total might be not accurate, as it shows it over time

Unsure if this is the best alternative:

"Seconds of CPU time consumed"
"Bytes allocated in the heap"

grafakus commented 3 weeks ago

In some cases, we could even get rid of the unit completely, e.g. "CPU time consumed".

We will review all the descriptions and make a suggestion.