-
Notifications
You must be signed in to change notification settings - Fork 61
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
Colour vision deficiency issues #107
Comments
I know some colors (all of them?) are hard-coded in https://github.com/vlsi/ksar/blob/master/src/main/resources/Config.xml I know there's http://colorbrewer2.org/ @06nenglish , do you think it is relevant? |
Yes, its hard-coded, but only few graphs are covered. The ksar_all.log file lists all missing colour code (you need to open each charts)
Feel free to create a "colour-blind" Config.xml and we make this available in the options dialog |
I was playing with this colours and adjusted the CPU chart, looks like this now @06nenglish what do you think? |
Apologies for the slow responses both! @Pitterling that is a lot easier for me to understand. I really appreciate you taking the time to look into it for me. Was this done via the config.xml or was it actually code that required changing? |
@Pitterling , Diverging scale might suit better here. |
@06nenglish yes, i only changed config.xml, i just picked the 8 proposed colours and distributed them across the stats ...
@vlsi vlsi I'm open to any change .. finally the colour code scheme should come from colour blind people ... In general we could offer, several sets .. default, colour blind ... etc |
@06nenglish are you still looking into this? I expected an adjusted config.xml file, coming from color-blind affected people ... |
Hi,
Really love the tool, going to help a great deal when performing tests. However I have one slight issue, maybe a quick way round maybe not.
I'm colour blind with Red, Green and Blue and struggle with similar colours quite badly and as you can imagine the graphs are very hard to decipher which colour matches the line on the graph.
If there is no option (I can't see anything set anywhere), it's something I will look into contributing as I know it'll help out many users.
Thanks!
The text was updated successfully, but these errors were encountered: