Add load density chart

[holly-cummins]

I was going to explain the chart, then realised I shouldn't, to see if it's self-explanatory. :)

Money version coming in follow-on PR.

This builds on the commits from #40, so those commits don't need to be reviewed here.

[github-actions]

🚀 This is what some of the generated images would look like with this change. You can browse the full set on the preview branch.

-

-

-

-

[holly-cummins]

Whoops, because I build on the other PR, the image preview is showing that. Here's what I've built:

• https://github.com/quarkusio/benchmarks/blob/cc0e013dc520e2accb6e12d7f8b9ae7d23e8ed48/images/tuned/spring-quarkus-perf-comparison-latest-tuned-load-density-for-all-dark.svg (a bit of a salad, for that many frameworks)
• https://github.com/quarkusio/benchmarks/blob/cc0e013dc520e2accb6e12d7f8b9ae7d23e8ed48/images/tuned/spring-quarkus-perf-comparison-latest-tuned-load-density-for-main-comparison-light.svg (sensible, IMO)

[edeandrea]

Should the number of instances lines be in front of the frameworks? As you get to the right I find it hard to correlate to number of instances.

[holly-cummins]

Should the number of instances lines be in front of the frameworks? As you get to the right I find it hard to correlate to number of instances.

Ah, yes, I see what you mean - or translucent might look quite smart, will try that.

[holly-cummins]

Updated to give the effect of transparent fill with visible gridlines. WDYT?

[edeandrea]

Updated to give the effect of transparent fill with visible gridlines. WDYT?

I think its better. What about about re-labeling the Y axis on the far right too?

[insectengine]

Updated to give the effect of transparent fill with visible gridlines. WDYT?

I think its better. What about about re-labeling the Y axis on the far right too?

I think that's an appropriate solution.

[holly-cummins]

Updated to give the effect of transparent fill with visible gridlines. WDYT?

I think its better.

I do too. It was fiddlier than I expected, but as soon as I saw it working, I could see it was way better than the first pass. Yay for team collaboration!

What about about re-labeling the Y axis on the far right too?

Ooh, tricky. I have another PR that I'll raise once this one has settled that does cost, and on that one I use left and right. Do you think having the right-axis be instances is confusing on this one, if we have another one where the right axis is money? I'm genuinely not sure, and can't judge because I'm too close to it.

The good news is I have the code if we do want to do it...

[edeandrea]

Ooh, tricky. I have another PR that I'll raise once this one has settled that does cost, and on that one I use left and right. Do you think having the right-axis be instances is confusing on this one, if we have another one where the right axis is money? I'm genuinely not sure, and can't judge because I'm too close to it.

I go back and forth. I was suggesting the re-labelling because even though the foreground is transparent, its still hard to read. And as I get older my eyesight isn't as good and I find that I'm correlating with the wrong label on the left-side Y axis. Thats where my suggestion for re-labelling the Y axis on the right came from. Note - i meant the same label as on the left.

But now that I see both visualizations for # instances & cost I do like that. What about combining # instances & cost in a single label and putting on both the left & right? IDK if thats a good idea (I'm certainly not a good UX person).