Color-Coding Graphs To Match Brands Being Benchmarked

With the recent call for comments on improving result graphs, one of the requests that's also been iterated before is for on bar graphs to color them to match the brand of, e.g. the graphics card being tested.

While in the comments I explained how it could be easily done for anyone that wanted to get involved with the open-source Phoronix-Test-Suite on GitHub, I decided to write the small patch myself as an example and to help others get started if they want to make any visual improvements to the graphs that are used not only here on Phoronix.com but also OpenBenchmarking.org, LinuxBenchmarking.com, and wherever else the Phoronix Test Suite is deployed.

This patch that's just a few lines of code adds basic coloring to results that are NVIDIA/GeForce, AMD/Radeon, or Intel. It's a very simple patch and just checks what the supplied result identifier is, though not much more challenging would be to hook it up to look at the reported system hardware/software to see what the real driver/GPU is and to color accordingly, so it would work even if the user running the benchmark labeled the identifier "hello world".

So far it seems to be working fine... How do you like the looks of the colored-by-brand graphs? Comment on this article to let me know. It would roll-out and be deployed to the graphs on Phoronix.com once the fresh code is pulled into OpenBenchmarking.org. Of course, thanks to the design, it would retroactively apply to all existing Phoronix content and OpenBenchmarking.org graphs, etc.

If you're playing around with graph improvements yourself, all of the relevant code is currently under the phx_graph area until I end up merging it back into pts_Graph in the days ahead once I'm done modifying some core parts of the graphing process and ensure no regressions. If you're trying out the Git code yourself as an example you can simply run USE_PHX_GRAPH=1 ./phoronix-test-suite refresh-graphs 1508201-HA-GTX95073337 to look at any coloring/UI differences (the USE_PHX_GRAPH environment variable is needed until I'm done with merging the code back into pts_Graph).