-
Notifications
You must be signed in to change notification settings - Fork 53
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
Suggestion for line plot in Example 9.8 #106
Comments
Thanks for the note! I agree, those two lines look very similar in color (even for those without colorblindness). As mentioned in the acknowledgment section, we optimized the general color scheme to be colorblind friendly, but this line will need some work. I'll assign Tim to this issue since he has the keenest sense of aesthetics and might have some thoughts on the best correction. |
Please let me know if you would like to be acknowledged, in which case I would need your first and last name. |
@wuzh07, could you please let us know what you think of this use of orange instead of the offending purple line? This plot would let us keep our consistent style of only using circular markers. |
@tawheeler The latest one works pretty well for me and I can tell different lines easily. My name is Zhen Wu and thank you again for your consideration. |
Perfect, we'll go with that then. |
The colors of "random rollout" (102,110,185) and "lookahead" (25,117,185) in the line plot in Example 9.8 are confusing for individuals with red-green color blindness (like me). It is greatly appreciated if a color blind-friendly palette is adopted or legends are diversified in different aspects (e.g. color, marker ...). Thank you very much.
The text was updated successfully, but these errors were encountered: