Skip to content

Dashboard should use log-scale and have a smaller aspect ratio #1901

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
the8472 opened this issue Apr 27, 2024 · 12 comments · Fixed by #1990
Closed

Dashboard should use log-scale and have a smaller aspect ratio #1901

the8472 opened this issue Apr 27, 2024 · 12 comments · Fixed by #1990

Comments

@the8472
Copy link
Member

the8472 commented Apr 27, 2024

Currently on my desktop the dashboard charts look like this

image

Due to the exponential downwards trend each incremental change becomes more more difficult to distinguish. Changing the aspect ratio to make the Y-axis taller would help to some extent. But I think we should use a log-scale, for the same reasons as @scottmcm already argued in #192 (comment)

If it's easy to toggle the charting library between linear and log then adding some toggle for that would be fine too. Then people who know how to read log scales can choose that.

@Kobzol
Copy link
Contributor

Kobzol commented Apr 27, 2024

It's not much better if we just switch to a log-scale:
image

Maybe it would be better to normalize it vs the 1.28.0 baseline instead, as discussed here.

@the8472
Copy link
Member Author

the8472 commented Apr 27, 2024

Well that seems to make the effective aspect ratio worse due to the y-axis offset wasting a lot of vertical space.

@nnethercote
Copy link
Contributor

When looking at those graphs I always massively shrink the width of my browser window, which makes it more readable. Perhaps we could just restrict the graph width to achieve the same effect automatically?

@s7tya
Copy link
Contributor

s7tya commented Jul 12, 2024

Due to #1816, the graphs now always have a smaller width. Therefore, it might be okay to close this issue.

@Kobzol
Copy link
Contributor

Kobzol commented Jul 12, 2024

Now with two charts per row it's indeed much better. I wonder how would it look like with the log scale now, could you please try it?

@s7tya
Copy link
Contributor

s7tya commented Jul 12, 2024

Alright! Will try

@s7tya
Copy link
Contributor

s7tya commented Oct 9, 2024

@Kobzol I took too much time to work on this. Here's the log scale version of the dashboard. How do you think?

Image
Image

@s7tya
Copy link
Contributor

s7tya commented Oct 9, 2024

the screenshot of current linear scale for convenience:

Image

@the8472
Copy link
Member Author

the8472 commented Oct 9, 2024

They're wasting a lot of space which vertically compresses the chart which eats up any gains we might get from it being a log chart.

Image

@s7tya
Copy link
Contributor

s7tya commented Oct 9, 2024

Ah, I see. Then, how about this one?
I set the minimum of the chart range to the data's lowest value and use a log scale. ( or we could just remove min maybe)

Image

@Kobzol
Copy link
Contributor

Kobzol commented Oct 9, 2024

Yeah, that looks much better, good idea! I'd still vote for a simple switch at the top of the page to allow selecting linear/log scale axes :)

@s7tya
Copy link
Contributor

s7tya commented Oct 9, 2024

Which option should be the default: log scale or linear scale?
I've created PR that use linear scale as a default

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants