Counter charts should not display a unit

[mcrumm]

The unit label on the y-axis and in the legend is confusing for a counter, since the values are simply positive whole numbers.

[leeoniya]

from #30 (comment) it sounds like you guys ditched donut charts for x/y plots? that's quite the change, considering how differently they show data. unless you're needing to show the counter progression over time, you might want to just DIY the donut charts via a sprinkle of SVG:

https://akzhy.com/blog/create-animated-donut-chart-using-svg-and-javascript/

uPlot is definitely not the hammer for every nail :P

although without seeing what you're referring to, i'm possibly inferring a lot of wrong stuff, heh.

[mcrumm]

Hi @leeoniya!

For more context, we're using uPlot to visualize data from Telemetry.Metrics. We're still in the very early stages of this implementation, so starting with a generic timeseries chart for all metrics types, given the performance improvements, seemed preferable for the time being.

That said, thank you very much for the link! ❤️ It's given me some new ideas :)

[josevalim]

And for completeness, we should also ignore the value and keep a counter on the client.

[mcrumm]

we should also ignore the value and keep a counter on the client

That's how it's done now - counter increments by one for each event and sum increments by the y value.